masaori335 opened a new pull request #7332:
URL: https://github.com/apache/trafficserver/pull/7332


   Based on #7331 change. Please look at the second commit.
   I opened separated PR because this adds unit tests and also changes the 
behavior of the parser.
   
   1). "UNKNOWN" protocol short form support
   ```
   "PROXY UNKNOWN\r\n"
   ```
   The spec 
([2020/03/05](http://www.haproxy.org/download/1.8/doc/proxy-protocol.txt)) 
Section 2.1. says
   > For "UNKNOWN", the rest of the line before the
       CRLF may be omitted by the sender, and the receiver must ignore anything
       presented before the CRLF is found.
   
   and
   
   > If the announced transport protocol is "UNKNOWN", then the receiver knows 
that
   the sender speaks the correct PROXY protocol with the appropriate version, 
and
   SHOULD accept the connection and use the real connection's parameters as if
   there were no PROXY protocol header on the wire.
   
   2). Strict malformed headers checks ( like invalid prefaces, protocols, and 
spaces )


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to