michael-o commented on a change in pull request #382:
URL: https://github.com/apache/tomcat/pull/382#discussion_r540877133
##########
File path: webapps/docs/config/http.xml
##########
@@ -208,6 +208,24 @@
The default is <code>POST</code></p>
</attribute>
+ <attribute name="path" required="false">
Review comment:
Having both enabled might lead to confusion as it does now with
dual-stack socket. Most don't even know that one can pass sockopt IPV6_ONLY. I
would avoid such situation unless the admin does exactly know what is happening
here. Now the misfortune here is that we don't have an address field, but
rather a host/port which is tied to TCP sockets. Maybe it is worth a discussion
on the dev ML to move to `address` instead of `host`/`port` combination. So do
all decent socket-based services like sshd, httpd, etc. WDYT?
----------------------------------------------------------------
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
