shinrich commented on pull request #7905: URL: https://github.com/apache/trafficserver/pull/7905#issuecomment-852477343
The argument to fail in this case is concern about framing errors pre-HTTP/2. If there is a body, and we don't fail, then we need to either forward on the body or drain the body. The current code just passes along the trace without the body and then ATS processes the body separately. Which will probably fail, but anytime you break framing you are opening the door for someone to try to be clever. I think it would be cleaner and safer to fail and close the connection. -- 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]
