Skip to content

docs: fix broken Node.js, NGINX, and systemd links#2378

Closed
Njoxpy wants to merge 1 commit into
expressjs:mainfrom
Njoxpy:link-improve
Closed

docs: fix broken Node.js, NGINX, and systemd links#2378
Njoxpy wants to merge 1 commit into
expressjs:mainfrom
Njoxpy:link-improve

Conversation

@Njoxpy

@Njoxpy Njoxpy commented Jun 12, 2026

Copy link
Copy Markdown

Comprehensive Documentation Audit & Broken Link Fixes Overview

  • This update addresses multiple broken, outdated, and non-functional external documentation links within the production best practices guide (best-practice-performance.md). Over time, changes to the routing structures of official platforms like Node.js, NGINX, and freedesktop.org caused several high-traffic reference links to return 404 Not Found or Directory Not Found errors. This pull request resolves those navigation dead-ends to ensure developers have immediate, reliable access to the correct external resources.

Detailed Technical Breakdown

  • NGINX Reference Corrections
  • Issue: The links referencing ngx_http_gzip_module and load_balancing were missing explicit file extensions, causing the remote NGINX server to throw 404 Not Found errors instead of resolving the paths.
    • Resolution: Modified the target endpoints to include the required .html extensions (e.g., changing from /load_balancing to /load_balancing.html), successfully restoring access to the official web server documentation.
  • Node.js API Route Updates
  • Issue: Links pointing to core Node.js modules—specifically cli, console, process, domain, and cluster—relanded users on generic server errors reading Directory not found due to legacy trailing slashes or missing file components.
    • Resolution: Standardized the URL formats to point directly to the specific asset structures utilized by the modern Node.js documentation server (e.g., swapping out /api/cluster for the absolute web path /api/cluster.html).
  • Systemd Reference Modernization
  • Issue: The reference hyperlink tracking systemd.unit pointed to an outdated, insecure HTTP URL that failed to route users to the actual unit file definitions.
    • Resolution: Upgraded the URI scheme to modern HTTPS standards and remapped the target path to point to the active, live-updating repository path /man/latest/index.html on the freedesktop.org domain.
  • Multi-Language Localization Sync
  • Scope of Impact: Rather than fixing these references solely within the English documentation framework, the identical URL corrections were audited, updated, and verified across all ten localized translation files in the repository workspace.
    • Affected Locales: English (en), German (de), Spanish (es), French (fr), Italian (it), Japanese (ja), Korean (ko), Brazilian Portuguese (pt-br), Simplified Chinese (zh-cn), and Traditional Chinese (zh-tw).

- Update NGINX gzip and load balancing links to include missing `.html` extensions to resolve 404 errors.
- Correct Node.js API documentation links (cli, console, process, domain, cluster) to fix "Directory not found" errors.
- Update outdated systemd reference link to point to the latest active documentation over HTTPS.
- Apply these URL corrections across all localized translation files.
@Njoxpy Njoxpy requested a review from a team as a code owner June 12, 2026 17:00
@netlify

netlify Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit d7ff540
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6a2c3b2d81eea70008ce45c2
😎 Deploy Preview https://deploy-preview-2378--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 99 (🟢 up 2 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@krzysdz

krzysdz commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Translations (including links in translated docs) are managed through Crowdin.

Duplicate of #2324

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants