Skip to content

docs: fix redirecting urls in markdown files#352

Open
sakethpanuganti wants to merge 3 commits into
FasterXML:3.xfrom
sakethpanuganti:fix/markdown-urls
Open

docs: fix redirecting urls in markdown files#352
sakethpanuganti wants to merge 3 commits into
FasterXML:3.xfrom
sakethpanuganti:fix/markdown-urls

Conversation

@sakethpanuganti
Copy link
Copy Markdown

fix: normalize and repair broken markdown URLs

  • Upgrade HTTP links to HTTPS across all docs
  • Replace redirecting badge and guidance URLs with canonical targets
  • Migrate GitHub links from master to main branch
  • Update external service URLs (Travis CI, Maven Central, ASM, CERT)

Comment thread afterburner/README.md Outdated
Comment thread SECURITY.md

The recommended mechanism for reporting possible security vulnerabilities follows
so-called "Coordinated Disclosure Plan" (see [definition of DCP](https://vuls.cert.org/confluence/display/Wiki/Coordinated+Vulnerability+Disclosure+Guidance)
so-called "Coordinated Disclosure Plan" (see [definition of DCP](https://certcc.github.io/confluence/display/Wiki/Coordinated+Vulnerability+Disclosure+Guidance/)
Copy link
Copy Markdown
Member

@cowtowncoder cowtowncoder May 25, 2026

Choose a reason for hiding this comment

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

As per Claude review:


SECURITY.md CERT link — trailing slash and shaky destination. The new URL is
https://certcc.github.io/confluence/display/Wiki/Coordinated+Vulnerability+Disclosure+Guidance/ — a Confluence-style path served from GitHub Pages, ending in /. It currently returns 200 because it's the redirect target of the old vuls.cert.org URL, but this looks like a mirror that may not be stable long-term.

The canonical CERT/CC guidance now lives at https://www.cert.org/vulnerability-analysis/vul-disclosure.cfm or under CMU SEI.

Worth double-checking with maintainers whether they'd prefer the SEI page; at minimum, the trailing slash on a Confluence-style URL is suspicious.

...

Consider replacing with the SEI CERT/CC page (e.g. https://www.cert.org/vulnerability-analysis/vul-disclosure.cfm or
https://insights.sei.cmu.edu/.../coordinated-vulnerability-disclosure-process/) instead — those are more likely to outlive a GitHub Pages mirror.

Copy link
Copy Markdown
Member

@cowtowncoder cowtowncoder left a comment

Choose a reason for hiding this comment

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

Looks good overall, just one concern wrt SECURITY.md link; when that is resolved, happy to merge!

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