Skip to content

Fix broken CVE links in 1.4.1 downloads page#4435

Merged
jbonofre merged 2 commits into
apache:mainfrom
hemasrishalini:fix-credential-rotation-check
May 14, 2026
Merged

Fix broken CVE links in 1.4.1 downloads page#4435
jbonofre merged 2 commits into
apache:mainfrom
hemasrishalini:fix-credential-rotation-check

Conversation

@hemasrishalini

@hemasrishalini hemasrishalini commented May 14, 2026

Copy link
Copy Markdown
Contributor

Currently the CVE links on the 1.4.1 downloads page point to incorrect security advisory paths due to uppercase route values.

This change updates the advisory URLs to lowercase paths so the CVE links resolve correctly.

Fixes #4429

Checklist

  • 🛡️ Don't disclose security issues! (contact security@apache.org)
  • 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic
  • 🧾 Updated CHANGELOG.md (if needed)
  • 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

@dimas-b dimas-b 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.

Thanks for the fix, @hemasrishalini !

@github-project-automation github-project-automation Bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board May 14, 2026
@jbonofre jbonofre enabled auto-merge (squash) May 14, 2026 15:22
@jbonofre

Copy link
Copy Markdown
Member

I pushed an empty commit here to trigger CI 😄

@jbonofre jbonofre merged commit f23fb45 into apache:main May 14, 2026
24 checks passed
@github-project-automation github-project-automation Bot moved this from Ready to merge to Done in Basic Kanban Board May 14, 2026
@hemasrishalini

Copy link
Copy Markdown
Contributor Author

Thanks @dimas-b and @jbonofre for the approvals and merge! Really happy to get my first PR merged 😄

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.

The CVE links are broken

3 participants