Skip to content

8367897: Document the javax.net.debug property in javadoc#30097

Open
0zzy0sbourne wants to merge 1 commit intoopenjdk:masterfrom
0zzy0sbourne:JDK-8367897-javax-net-debug-doc
Open

8367897: Document the javax.net.debug property in javadoc#30097
0zzy0sbourne wants to merge 1 commit intoopenjdk:masterfrom
0zzy0sbourne:JDK-8367897-javax-net-debug-doc

Conversation

@0zzy0sbourne
Copy link
Copy Markdown

@0zzy0sbourne 0zzy0sbourne commented Mar 5, 2026

Adds javadoc documentation for the javax.net.debug system property to the javax.net.ssl
package, following the same pattern used for java.security.debug in the java.security
package. A new doc-files/debug-system-property.html page is created documenting all
supported options, and a link to it is added to the package-info.java.

make docs-javadoc

Results:
Finished building target 'docs-javadoc' in configuration 'linux-x86_64-server-release'


Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue
  • Change requires a CSR request matching fixVersion 27 to be approved (needs to be created)

Issue

  • JDK-8367897: Document the javax.net.debug property in javadoc (Enhancement - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/30097/head:pull/30097
$ git checkout pull/30097

Update a local copy of the PR:
$ git checkout pull/30097
$ git pull https://git.openjdk.org/jdk.git pull/30097/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 30097

View PR using the GUI difftool:
$ git pr show -t 30097

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/30097.diff

Using Webrev

Link to Webrev Comment

@bridgekeeper
Copy link
Copy Markdown

bridgekeeper bot commented Mar 5, 2026

👋 Welcome back 0zzy0sbourne! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk
Copy link
Copy Markdown

openjdk bot commented Mar 5, 2026

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@openjdk openjdk bot added security security-dev@openjdk.org net net-dev@openjdk.org labels Mar 5, 2026
@openjdk
Copy link
Copy Markdown

openjdk bot commented Mar 5, 2026

@0zzy0sbourne The following labels will be automatically applied to this pull request:

  • net
  • security

When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing lists. If you would like to change these labels, use the /label pull request command.

@0zzy0sbourne 0zzy0sbourne marked this pull request as ready for review March 6, 2026 10:14
@openjdk openjdk bot added the rfr Pull request is ready for review label Mar 6, 2026
@mlbridge
Copy link
Copy Markdown

mlbridge bot commented Mar 6, 2026

Webrevs

@bradfordwetmore
Copy link
Copy Markdown
Contributor

bradfordwetmore commented Mar 26, 2026

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

@openjdk openjdk bot added the csr Pull request needs approved CSR before integration label Mar 26, 2026
@openjdk
Copy link
Copy Markdown

openjdk bot commented Mar 26, 2026

@bradfordwetmore has indicated that a compatibility and specification (CSR) request is needed for this pull request.

@0zzy0sbourne please create a CSR request for issue JDK-8367897 with the correct fix version. This pull request cannot be integrated until the CSR request is approved.

Copy link
Copy Markdown
Contributor

@bradfordwetmore bradfordwetmore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

@@ -0,0 +1,132 @@
<!doctype html>
<!--
Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
Copy link
Copy Markdown
Contributor

@bradfordwetmore bradfordwetmore Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

<body LANG="en-US" DIR="LTR">
<h1><B>{@systemProperty javax.net.debug}</B></h1>
<p> To monitor SSL/TLS activity, you can set the <code>javax.net.debug</code>
system property, which determines what trace messages are printed during
Copy link
Copy Markdown
Contributor

@bradfordwetmore bradfordwetmore Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

</tr>
<tr>
<th scope="row"><code>ssl</code></th>
<td>Turn on SSL debugging. If specified by itself, all non-widening
Copy link
Copy Markdown
Contributor

@bradfordwetmore bradfordwetmore Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

</thead>
<tbody>
<tr>
<th scope="row"><code>all</code></th>
Copy link
Copy Markdown
Contributor

@bradfordwetmore bradfordwetmore Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

<code>handshake</code>:
<ul>
<li><code><b>verbose</b></code>: Verbose handshake
message printing</li>
Copy link
Copy Markdown
Contributor

@bradfordwetmore bradfordwetmore Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bradfordwetmore, thanks for making a comment in an OpenJDK project!

All comments and discussions in the OpenJDK Community must be made available under the OpenJDK Terms of Use. If you already are an OpenJDK Author, Committer or Reviewer, please click here to open a new issue so that we can record that fact. Please Use "Add GitHub user bradfordwetmore" for the summary.

If you are not an OpenJDK Author, Committer or Reviewer, simply check the box below to accept the OpenJDK Terms of Use for your comments.

Your comment will be automatically restored once you have accepted the OpenJDK Terms of Use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

csr Pull request needs approved CSR before integration net net-dev@openjdk.org rfr Pull request is ready for review security security-dev@openjdk.org

Development

Successfully merging this pull request may close these issues.

2 participants