trafficserver
trafficserver copied to clipboard
Add TCP info from server socket
Current tcpinfo condition from header_rewrite plugin only provides tcp info from the client socket.
- Adding a CLIENT and SERVER qualifier within this condition will provide tcp info from both client and server sockets. This could be beneficial in getting more info on the server connection behavior
Please add to the existing documentation as well. thanks.
I thought we were shifting to using the terms "INBOUND" and "OUTBOUND" instead of "CLIENT" and "SERVER".
As per amc's question, I wonder if this should be changed to e.g.
%{TCP-INFO:INBOUND}
%{TCP-INFO:OUTBOUND}
or
%{INBOUND:TCP-INFO}
%{OUTBOUND:TCP-INFO}
? Also, this patch (or the above) would both be incompatible with current version of ATS 9.x right? As such, we can't put such a change into 9.x, unless you keep the old variation and add the new as an addition.
New changes should still be compatible. Without qualifiers, it defaults to previous behavior, using the client (inbound) socket.
[approve ci]
[approve ci autest]
[approve ci]
This pull request has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.