diff options
author | James Tucker <jftucker@gmail.com> | 2012-03-17 12:50:59 -0700 |
---|---|---|
committer | James Tucker <raggi@google.com> | 2012-12-29 23:30:28 -0500 |
commit | 3623d04526b953a63bfb3e72de2d6920a042563f (patch) | |
tree | 637ff270774111e52196631577837eb6c774da31 /lib/rack/lint.rb | |
parent | 1fa9108657280d80486fb2b2653df6725f2c14b3 (diff) | |
download | rack-3623d04526b953a63bfb3e72de2d6920a042563f.tar.gz |
Correct some of the mime type issues. References #316 and #366.
HTTP 1.0 and 1.1 do not have MUST for Content-Type requirements, they have "should" (not SHOULD). They also have text describing how clients should handle this header being missing.
Diffstat (limited to 'lib/rack/lint.rb')
-rw-r--r-- | lib/rack/lint.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/rack/lint.rb b/lib/rack/lint.rb index 3290be06..a5569bae 100644 --- a/lib/rack/lint.rb +++ b/lib/rack/lint.rb @@ -474,9 +474,10 @@ module Rack return end } - assert("No Content-Type header found") { - Rack::Utils::STATUS_WITH_NO_ENTITY_BODY.include? status.to_i - } + # This is a SHOULD in HTTP 1.0 and 1.1: + # assert("No Content-Type header found") { + # Rack::Utils::STATUS_WITH_NO_ENTITY_BODY.include? status.to_i + # } end ## === The Content-Length |