RFR: 8230694: http.keepAlive system property is inconsistently/incorrectly documented

Chris Hegarty chris.hegarty at oracle.com
Fri Sep 6 12:59:17 UTC 2019


> On 6 Sep 2019, at 13:52, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
> 
> Hi,
> 
> Please find below a trivial fix for:
> 
> 8230694: http.keepAlive system property is inconsistently/incorrectly
>         documented
> https://bugs.openjdk.java.net/browse/JDK-8230694
> 
> The net-properties.html has a typo in the name of the http.keepAlive
> network property: it should read `http.keepAlive` not `http.keepalive`.
> 
> best regards,
> 
> -- daniel
> 
> patch:
> 
> diff --git a/src/java.base/share/classes/java/net/doc-files/net-properties.html b/src/java.base/share/classes/java/net/doc-files/net-properties.html
> --- a/src/java.base/share/classes/java/net/doc-files/net-properties.html
> +++ b/src/java.base/share/classes/java/net/doc-files/net-properties.html
> @@ -165,7 +165,7 @@
> 	-Dhttp.agent=”foobar” is used, the User-Agent header will
> 	contain “foobar Java/1.5.0” if the version of the VM is
> 	1.5.0). This property is checked only once at startup.</P>
> -	<LI><P><B>{@systemProperty http.keepalive}</B> (default: true)<BR>
> +	<LI><P><B>{@systemProperty http.keepAlive}</B> (default: true)<BR>
> 	Indicates if persistent connections should be supported. They improve
> 	performance by allowing the underlying socket connection to be reused
> 	for multiple http requests. If this is set to true then persistent

LGTM.

-Chris.




More information about the net-dev mailing list