Skip to content

Conversation

@MikeMcC399
Copy link
Contributor

Description

In documents:

update all link usage from http to https protocol.

Motivation and Context

As mentioned in PR #2297, some old links are still using the http protocol instead of https.

These links were put there 5 to 10 years ago, and in the meantime each of the sites is accessible through the https protocol.

Even for read-only website access, https is now considered the default protocol choice. Google Chrome, for instance, will automatically upgrade all http navigation to https. Documentation in this repo should map to this default by ensuring all links use https.

Testing Details

On Ubuntu 24.04.3 LTS, with Node.js 22.20.0 LTS

npm install [email protected] -g
find . -name "*.md"  ! -path "./.git/*" | xargs -n 1 markdown-link-check -c markdown_link_check_config.json

No dead links should be reported.

Example Output(if appropriate)

Types of changes

  • Documentation
  • Version change (Update, remove or add more Node.js versions)
  • Variant change (Update, remove or add more variants, or versions of variants)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (none of the above)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • All new and existing tests passed.

@MikeMcC399 MikeMcC399 marked this pull request as ready for review October 18, 2025 13:20
@nschonni nschonni merged commit 329f0f1 into nodejs:main Oct 18, 2025
3 checks passed
@MikeMcC399 MikeMcC399 deleted the docs/upgrade-http-links-to-https branch October 19, 2025 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants