RFR: 8342807: Update links in java.base to use https:// [v3]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Jan 20 16:28:44 UTC 2025
On Fri, 17 Jan 2025 18:38:25 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:
>> Please review this cleanup PR which updates a total of 12 links to external documentation or references in `java.base` to use https instead of plain text http.
>>
>> Links in `java.security` and `share/data/tzdata` are excluded from this PR.
>>
>> This is a documentaton-only cleanup. No tests are added or updated. `noreg-cleanup` added in the JBS. All updated links have been verified to resolve and to show the expected content.
>>
>> There are two files here with non-Oracle copyright headers, not quite sure how to update those? (`java_md_aix.h` and `xss-common-qsort.h`).
>>
>> Changes beyond the obvious are as follows:
>>
>> `linux/native/libsimdsort/xss-common-qsort.h`:
>> The current link uses a non-default port number. Changed to using the default port number. Reported upstream as intel/x86-simd-sort#170
>>
>> `share/man/keytool.1`:
>> www.oracle.com redirects this from `/technetwork/java/javase/javasecarootcertsprogram-1876540.html` to `/java/technologies/javase/carootcertsprogram.html`. Using the new URL here. Should this be updated in Oracle-internal sources as well?
>>
>> `unix/classes/sun/net/PortConfig.java`
>> The current link no longer resolves. As a replacement, I suggest https://www.ibm.com/support/pages/node/886227 which answers a support question regarding ephemeral ports.
>>
>> `unix/native/libjava/*`,
>> Links redirect from "www.opengroup.org" to "pubs.opengroup.org". Using this new host in the URLs
>>
>> `unix/native/libjava/ProcessImpl_md.c`
>> A link to pasc.org now requires authentication. I changed this to be a Wayback link. Alternantives would be to delete this link, or to track down a revised version of the spec the linked Interpretation Request refers to.
>
> Eirik Bjørsnøs has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
>
> - Merge branch 'master' into links-use-https
> - Re-fix link to "Oracle Java Root Certificate program" in keytool.md
> - Merge branch 'master' into links-use-https
>
> # Conflicts:
> # src/java.base/share/man/keytool.1
> - Remove accidental "'"
> - Use Wayback link
> - Update copyright year
> - Update links in java.base to use https://
For the changes in `ProcessImpl_md.c`, I think we'll want to have the input of @RogerRiggs.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21633#issuecomment-2602836939
More information about the net-dev
mailing list