From: Amos Jeffries <squid3@treenet.co.nz>
Date: Thu, 14 Apr 2016 12:01:02 +0000 (+1200)
Subject: Partial revert of rev.14638
X-Git-Tag: SQUID_4_0_9~9
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=24ddaa3474202addb2db7b3db9fd1fa0e306f7fc;p=thirdparty%2Fsquid.git

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 682432ef67..d49c516e5f 100644
--- 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;