rmaucher commented on issue #277:
URL: https://github.com/apache/tomcat/pull/277#issuecomment-617661025


   The initial post says SHOULD, but after actually checking the spec it is a 
MUST.
   https://tools.ietf.org/html/rfc7540#section-8.1.2.2
   
   It is really odd the specification made such a choice given how many 
applications toy with that header in an attempt to do something useful, and 
given they have no idea if they're using HTTP/1.1 or HTTP/2. So this looks to 
me like a spec mistake [for compatibility, they should have said it has to be 
ignored and can be freely removed, whatever], and it is not very surprising 
most HTTP/2 clients would not check this requirement.


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to