https://bz.apache.org/bugzilla/show_bug.cgi?id=61088
Bug ID: 61088
Summary: Documentation upper/lower case mismach for
SSLHostConfig doc and SSLHostConfig java source
Product: Tomcat 8
Version: 8.5.x-trunk
Hardware: PC
Status: NEW
Severity: normal
Priority: P2
Component: Documentation
Assignee: dev@tomcat.apache.org
Reporter: didier.wir...@mesr.etat.lu
Target Milestone: ----
There is an upper/lower case mismatch between SSLHostConfig code and the
documentation of SSLHostConfig:
https://tomcat.apache.org/tomcat-8.5-doc/config/http.html#SSL_Support_-_SSLHostConfig
Doc says quote:
"The token all is an alias for SSLv2Hello,TLSv1,TLSv1.1,TLSv1.2."
and "all" is listed as an option:
The source code uses with "A" uppercase:
static {
/* Default used if protocols is not configured, also
used if protocols="All" */
/* If protocols is configured to be empty, the effective
value comes from
org.apache.tomcat.util.net.jsse.JSSESocketFactory.defaultServerProtocols
(JSSE) resp. org.apache.tomcat.jni.SSL.SSL_PROTOCOL_ALL (OpenSSL)*/
SSL_PROTO_ALL_SET.add(Constants.SSL_PROTO_SSLv2Hello);
SSL_PROTO_ALL_SET.add(Constants.SSL_PROTO_TLSv1);
SSL_PROTO_ALL_SET.add(Constants.SSL_PROTO_TLSv1_1);
SSL_PROTO_ALL_SET.add(Constants.SSL_PROTO_TLSv1_2);
}
Documentation should use the correct case:
"The token All is an alias for SSLv2Hello,TLSv1,TLSv1.1,TLSv1.2."
And the listing should use "All" everywhere.
Thx!
Didier
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
