bneradt commented on code in PR #12146:
URL: https://github.com/apache/trafficserver/pull/12146#discussion_r2023071089


##########
doc/admin-guide/configuration/proxy-protocol.en.rst:
##########
@@ -31,7 +31,7 @@ TLS connections.
 
 .. note::
 
-    The current implementation doesn't support TLV fields of Version 2.
+    The current implementation doesn't support sending TLV fields of Version 2.

Review Comment:
   Actually, it might be good to state explicitly that ATS does support 
receiving TLV version 2 now that you added that.  Maybe:
   
   ```
   While |TS| supports receiving and processing TLV Version 2 formatted 
payloads,
   it does not support sending them.



-- 
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: github-unsubscr...@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to