-
-
Notifications
You must be signed in to change notification settings - Fork 511
Fix dead link checker: correct www.github.com URLs and exclude internal routes #2339
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: commjoen <[email protected]>
Co-authored-by: commjoen <[email protected]>
Co-authored-by: commjoen <[email protected]>
Co-authored-by: commjoen <[email protected]>
|
🌐 GitHub Pages Preview Ready! Your static preview is now available at: 📄 What's included:
For full functionality testing: Use the Docker preview instead. 🔄 Auto-updates: This preview will be updated automatically when you push new commits to this PR. Static preview with Thymeleaf generation by GitHub Actions |
|
@copilot we stil get Summary
|
Co-authored-by: commjoen <[email protected]>
Fixed in commit 5cc4677. Added regex pattern |
|
📸 Visual Diff Available! Screenshots have been generated comparing your changes with the main branch. Download Visual Diff Artifacts The artifacts contain:
Compare these images to see the visual impact of your changes! Visual diff generated by GitHub Actions |
|
🔨 Preview Build Complete! Your changes have been built and pushed to GitHub Container Registry. 🐳 Docker Image: 📦 Download & Test Locally:
# Download the artifact, extract it, then:
docker load < wrongsecrets-preview.tar
docker run -p 8080:8080 wrongsecrets-preview🚀 Alternative - Pull from Registry: docker pull ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2339-df45fe5
docker run -p 8080:8080 ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2339-df45fe5Then visit: http://localhost:8080 📝 Changes in this PR: Visual diff screenshots will be available shortly... Preview built by GitHub Actions |
|
📸 Visual Diff Ready! Screenshots comparing your changes with the main branch are available: 📁 Download Visual Diff Artifacts 🖼️ Included screenshots:
🔍 How to review:
💡 Tip: Use an image comparison tool or open both images in separate browser tabs to spot differences easily. Visual diff generated by GitHub Actions • PR #2339 |
|
🌐 GitHub Pages Preview Ready! Your static preview is now available at: 📄 What's included:
For full functionality testing: Use the Docker preview instead. 🔄 Auto-updates: This preview will be updated automatically when you push new commits to this PR. Static preview with Thymeleaf generation by GitHub Actions |
|
📸 Visual Diff Available! Screenshots have been generated comparing your changes with the main branch. Download Visual Diff Artifacts The artifacts contain:
Compare these images to see the visual impact of your changes! Visual diff generated by GitHub Actions |
|
📸 Visual Diff Available! Screenshots have been generated comparing your changes with the main branch. Download Visual Diff Artifacts The artifacts contain:
Compare these images to see the visual impact of your changes! Visual diff generated by GitHub Actions |
|
🔨 Preview Build Complete! Your changes have been built and pushed to GitHub Container Registry. 🐳 Docker Image: 📦 Download & Test Locally:
# Download the artifact, extract it, then:
docker load < wrongsecrets-preview.tar
docker run -p 8080:8080 wrongsecrets-preview🚀 Alternative - Pull from Registry: docker pull ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2339-79c03c0
docker run -p 8080:8080 ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2339-79c03c0Then visit: http://localhost:8080 📝 Changes in this PR: Visual diff screenshots will be available shortly... Preview built by GitHub Actions |
|
🔨 Preview Build Complete! Your changes have been built and pushed to GitHub Container Registry. 🐳 Docker Image: 📦 Download & Test Locally:
# Download the artifact, extract it, then:
docker load < wrongsecrets-preview.tar
docker run -p 8080:8080 wrongsecrets-preview🚀 Alternative - Pull from Registry: docker pull ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2339-79c03c0
docker run -p 8080:8080 ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2339-79c03c0Then visit: http://localhost:8080 📝 Changes in this PR: Visual diff screenshots will be available shortly... Preview built by GitHub Actions |
|
📸 Visual Diff Ready! Screenshots comparing your changes with the main branch are available: 📁 Download Visual Diff Artifacts 🖼️ Included screenshots:
🔍 How to review:
💡 Tip: Use an image comparison tool or open both images in separate browser tabs to spot differences easily. Visual diff generated by GitHub Actions • PR #2339 |
|
📸 Visual Diff Ready! Screenshots comparing your changes with the main branch are available: 📁 Download Visual Diff Artifacts 🖼️ Included screenshots:
🔍 How to review:
💡 Tip: Use an image comparison tool or open both images in separate browser tabs to spot differences easily. Visual diff generated by GitHub Actions • PR #2339 |
|
🧹 Preview Cleanup Complete The static preview for this PR has been removed from GitHub Pages. Thanks for contributing to WrongSecrets! 🎉 Cleanup completed by GitHub Actions |
What kind of changes does this PR include?
Description
The dead link checker was failing due to 141+ invalid GitHub URLs using the
wwwsubdomain, which doesn't exist for github.com, and lychee attempting to validate internal application routes.Changes:
https://www.github.com/→https://github.com/across:src/main/resources/templates/welcome.html(48 URLs)static-site/pr-2125/pages/welcome.html(48 URLs)README.md(45+ URLs in contributors + LinkedIn share link).lycheeignore(returns 403 Forbidden)^/[^/].*to.lycheeignoreto exclude internal application routes (e.g.,/challenge/challenge-1,/images/screenshot.png) from link validationImpact:
Link checker errors reduced from 248 to 112. The InvalidPathToUri warnings for internal routes are now eliminated by excluding them from validation.
Relations
Closes #2338
References
N/A
Checklist:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.