RFR: 8296804: Document HttpClient configuration properties in java.net.http module-info [v7]
Michael McMahon
michaelm at openjdk.org
Fri Nov 25 11:45:03 UTC 2022
On Fri, 25 Nov 2022 07:20:26 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Michael McMahon has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update to be compatible with JDK-8297030
>
> src/java.net.http/share/classes/module-info.java line 59:
>
>> 57: * <li><p><b>{@systemProperty jdk.httpclient.connectionWindowSize}</b> (default: 2^26)<br>
>> 58: * The HTTP/2 client connection window size in bytes. The maximum size is 2^31-1. This value cannot be
>> 59: * smaller than the stream window size.
>
> Should this additionally point to the `jdk.httpclient.windowsize` system property? Something like `This value cannot be smaller than the stream window size, which can be configured through {@code jdk.httpclient.windowsize} system property`?
Good idea!
> src/java.net.http/share/classes/module-info.java line 84:
>
>> 82: * <li>ssl</li>
>> 83: * <li>trace</li>
>> 84: * <li>channel</li>
>
> Hello Michael, is it intentional that we haven't documented `content` in this list?
Thanks for noticing that omission Jai!
> src/java.net.http/share/classes/module-info.java line 136:
>
>> 134: * <li><p><b>{@systemProperty jdk.internal.httpclient.disableHostnameVerification}</b> (default: false)<br>
>> 135: * If true, hostname verification in SSL certificates is disabled. This is a system property only and not
>> 136: * available in {@code conf/net.properties}. It is provided for testing purposes only.
>
> From what I can see in the code `jdk.internal.net.http.common.Utils#hostnameVerificationDisabledValue()`, we consider an empty value for this property to imply `true`. Perhaps we should note that here?
Yes, agreed!
-------------
PR: https://git.openjdk.org/jdk/pull/11241
More information about the net-dev
mailing list