Am 2019-10-11 um 11:51 schrieb Rémy Maucherat:
On Fri, Oct 11, 2019 at 10:30 AM <micha...@apache.org> wrote:
This is an automated email from the ASF dual-hosted git repository.
michaelo pushed a commit to branch BZ-63835/8.5.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit 6ff2233cbbd27c9c2c649208a21931e5f3e132a6
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Fri Oct 11 10:30:08 2019 +0200
First draft
+ if (keepAliveTimeout > 0) {
+ String value = "timeout=" +
TimeUnit.MILLISECONDS.toSeconds(keepAliveTimeout);
+
+ if (maxKeepAliveRequests > 0) {
+ value += ", max=" + maxKeepAliveRequests;
+ }
StringBuilder ?
StringBuilder does not make any sense because:
* The compiler will replace the static code automatically with a
StringBuilder
* StringBuilder gains benefit when you concat strings in a for/while/do
loop. The above code is purely static.
Can you add a new flag controlling the feature ? The information may not be
very useful in many cases such as when proxying, so it would be better to
skip generating it by default.
This is -- as documented -- a first draft.
As mentioned on the ticket. This is hop-by-hop and writetn only if the
client requests this piece of information. We can surely discuss a flag
for this on the connector.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
