maskit commented on issue #12191:
URL: 
https://github.com/apache/trafficserver/issues/12191#issuecomment-2807346508

   > Both formats are designed to fit in the smallest TCP segment that any 
TCP/IP
   host is required to support (576 - 40 = 536 bytes). This ensures that the 
whole
   header will always be delivered at once when the socket buffers are still 
empty
   at the beginning of a connection.
   
   An easy mitigation would be to increase the buffer size to 536 bytes. But 
that's still not enough if many TLV fields follow the header.
   
   There is another issue that is slightly related to the buffer size. Current 
code assumes everything for PROXY protocol (both the header and TLV fields) 
fits in one buffer block. If the header and/or TLV fields are stored over 
multiple buffers, ATS fails to parse them.
   
   The trampoline needs a major change if we have to support those cases.


-- 
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.

To unsubscribe, e-mail: [email protected]

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

Reply via email to