Skip to content

docs: fix broken links in server READMEs and RabbitMQ guides - #4965

Open
sarankumar1325 wants to merge 1 commit into
awslabs:mainfrom
sarankumar1325:docs/fix-broken-readme-links
Open

sarankumar1325 wants to merge 1 commit into
awslabs:mainfrom
sarankumar1325:docs/fix-broken-readme-links

Conversation

@sarankumar1325

@sarankumar1325 sarankumar1325 commented Oct 30, 2026 •

Copy link
Copy Markdown

Fixes

Summary

A link check over the repo's Markdown found several links that return 404. This PR fixes the ones where the correct target is clear.

Changes

  • Amazon MQ RabbitMQ guides (rabbitmq_performance_optimization_best_practice.md, rabbitmq_setup_best_practice.md): three links had the domain doubled (https://docs.aws.amazon.com/https://docs.aws.amazon.com/..., https://aws.amazon.com/https://aws.amazon.com/...). Removed the duplicate prefix.
  • Docs-site links with a trailing slash: https://awslabs.github.io/mcp/servers/<name>/ returns 404, but the same URL without the slash works (it matches the site's sitemap.xml). Fixed in the aws-serverless, aws-bedrock-custom-model-import, aws-dataprocessing, eks and dynamodb (AGENTS.md) docs.
  • aws-iot-sitewise-mcp-server README: the LICENSE link pointed to src/aws-iot-sitewise/LICENSE. Changed it to src/aws-iot-sitewise-mcp-server/LICENSE.
  • aws-healthomics-mcp-server README: github.com/awslabs/mcp-proxy-for-aws is now github.com/aws/mcp-proxy-for-aws.
  • aws-serverless-mcp-server README: removed the Changelog link. src/aws-serverless-mcp-server/CHANGELOG.md doesn't exist and isn't in the repo history.
  • Nova Canvas sample README: src/nova-canvas-mcp-server was removed in chore: remove 12 deprecated server directories #3165. The link now points to that folder at the commit just before the removal, so the sample still links to the code it describes.
  • Made the same URL changes in the matching docusaurus/i18n/ja copies. No translated text was changed.

User experience

Readers who click these links now reach the intended pages instead of 404s. Code and behaviour are unchanged.

Checklist

  • I have reviewed the contributing guidelines
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Testing: I checked every new or changed URL with curl -L, and all return HTTP 200.

Note: I used an AI coding assistant to find the broken links and prepare the edits. I reviewed every change and verified every link myself.

Is this a breaking change? N

RFC issue number: N/A

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Copilot AI balanced review requested due to automatic review settings October 30, 2026 03:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

This branch has not been deployed

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

Labels

None yet

Projects

Status: To triage

Development

Successfully merging this pull request may close these issues.

2 participants