RFR: 8328914: Document the java.security.debug property in javadoc [v3]
Koushik Muthukrishnan Thirupattur
duke at openjdk.org
Fri Feb 21 00:17:19 UTC 2025
On Thu, 20 Feb 2025 21:49:49 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> Koushik Muthukrishnan Thirupattur has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - 8328914: Document the java.security.debug property in javadoc
>> - 8328914: Document the java.security.debug property in javadoc
>
> src/java.base/share/classes/java/security/doc-files/security-related-system-properties.html line 66:
>
>> 64: which determines what trace messages are printed during execution.
>> 65: To view security properties, security providers, and TLS-related settings,
>> 66: specify the -XshowSettings:security option in the java command.
>
> This sentence seems out-of-place here. While I think it is useful to mention this, it has nothing to do with the `java.security.debug` property, so it seems a little confusing, and I suggest we not include it right now. Perhaps we can add another section on security related VM options later.
Removed.
> src/java.base/share/classes/java/security/doc-files/security-related-system-properties.html line 67:
>
>> 65: To view security properties, security providers, and TLS-related settings,
>> 66: specify the -XshowSettings:security option in the java command.
>> 67: You can enable debugging in JGSS and other related technologies with various system properties or environment variables.
>
> This is referring to other system properties like `sun.security.jgss.debug`. Since we are not documenting those other properties here (yet), we should remove this sentence.
Removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23569#discussion_r1964539681
PR Review Comment: https://git.openjdk.org/jdk/pull/23569#discussion_r1964539551
More information about the security-dev
mailing list