RFR: 8296804: Document HttpClient configuration properties in java.net.http module-info [v7]
Daniel Fuchs
dfuchs at openjdk.org
Fri Nov 25 12:23:48 UTC 2022
On Thu, 24 Nov 2022 11:18:33 GMT, Michael McMahon <michaelm at openjdk.org> wrote:
>> Hi,
>>
>> Could I get the following doc change reviewed please? It documents the system/networking properties used by the java.net.http HTTP client in its module-info. (CSR to follow)
>>
>> Thanks,
>> Michael
>
> Michael McMahon has updated the pull request incrementally with one additional commit since the last revision:
>
> update to be compatible with JDK-8297030
Changes requested by dfuchs (Reviewer).
src/java.net.http/share/classes/module-info.java line 76:
> 74: * <li><p><b>{@systemProperty jdk.httpclient.HttpClient.log}</b> (default: none)<br>
> 75: * Enables high-level logging of various events through the Java Logging API (which is contained in the
> 76: * package java.util.logging). The value contains a comma-separated list of any of the following items:
java.util.logging is actually only the default backend, so I'd suggest to reword as:
* Enables high-level logging of various events through the {@linkplain java.lang.System.Logger Platform Logging API}.
* The value contains ...
```
src/java.net.http/share/classes/module-info.java line 94:
> 92: * </ul><br>
> 93: * Specifying an item adds it to the HTTP client's log. For example, if you specify the following value,
> 94: * then the Java Logging API logs all possible HTTP Client events:<br>
Same here - use `Platform Logging API` rather than `Java Logging API`.
src/java.net.http/share/classes/module-info.java line 102:
> 100: * The number of seconds to keep idle HTTP connections alive in the keep alive cache. This property
> 101: * applies to both HTTP/1.1 and HTTP/2. The value for HTTP/2 can be overridden with the
> 102: * jdk.httpclient.keepalive.timeout.h2 property.
`{@code jdk.httpclient.keepalive.timeout.h2}` ?
src/java.net.http/share/classes/module-info.java line 106:
> 104: * <li><p><b>{@systemProperty jdk.httpclient.keepalive.timeout.h2}</b> (default: see below)<br>
> 105: * The number of seconds to keep idle HTTP/2 connections alive. If not set, then the
> 106: * jdk.httpclient.keepalive.timeout setting is used.
`{@code jdk.httpclient.keepalive.timeout}` ?
-------------
PR: https://git.openjdk.org/jdk/pull/11241
More information about the net-dev
mailing list