Partial revert of rev.14638

It did not catch all cases of the SP it was intended to and the chunked
encoding parse will need significantly different changes pending IETF WG
discussions.

Keep the violationLevel() member, which will be useful in general.

diff --git a/src/http/one/TeChunkedParser.cc b/src/http/one/TeChunkedParser.cc
index 682432ef671c12ba755d03b156468639dab9fd86..d49c516e5f13a704766effaec5b07e4beb273318 100644 (file)
--- a/src/http/one/TeChunkedParser.cc
+++ b/src/http/one/TeChunkedParser.cc
@@ -91,14 +91,6 @@ Http::One::TeChunkedParser::parseChunkSize(Http1::Tokenizer &tok)
 
         theChunkSize = theLeftBodySize = size;
         debugs(94,7, "found chunk: " << theChunkSize);
-
-#if USE_HTTP_VIOLATIONS
-        // Bug 4492: IBM_HTTP_Server pads out to 4 bytes with SP characters
-        if (Config.onoff.relaxed_header_parser && tok.skipAll(CharacterSet::SP)) {
-            debugs(94, violationLevel(), "WARNING: skipped invalid whitespace in chunk size");
-        }
-#endif
-
         buf_ = tok.remaining(); // parse checkpoint
         parsingStage_ = Http1::HTTP_PARSE_CHUNK_EXT;
         return true;