Skip to content

adding remote secret proxy pattern - #2771

Merged
commjoen merged 1 commit into
masterfrom
hotfix-ai-patterns
Oct 26, 2026
Merged

commjoen merged 1 commit into
masterfrom
hotfix-ai-patterns

Conversation

@commjoen

@commjoen commjoen commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

What kind of changes does this PR include?

  • Fixes or refactors
  • A new challenge
  • Additional documentation: adding secret injection proxy pattern to reason fixes.
  • Something else

Description

Relations

References

Checklist:

  • All the contributions made are solely the work of me and my co-authors
  • I used AI to generate parts of the content.
  • I tested the changes in this PR (if applicable)
  • I added unit tests to ensure my change works (when change in Java or on front-end code)
  • I added UI tests to ensure my UI changes work (when change in the overall UI, not needed if just adding a challenge)
  • The PR passes pre-commit hooks and automated tests

@github-actions

Copy link
Copy Markdown

🌐 GitHub Pages Preview Ready!

Your static preview is now available at:
🔗 Preview PR #2771

📄 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

@github-actions

Copy link
Copy Markdown

🔨 Preview Build Complete!

Your changes have been built and pushed to GitHub Container Registry.

🐳 Docker Image: ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2771-70c6ea0

📦 Download & Test Locally:

  1. 📁 Download Docker Image Artifact (look for wrongsecrets-preview-pr-2771)
  2. Load and run the image:
# Download the artifact, extract it, then:
docker load < wrongsecrets-preview.tar
docker run -p 8080:8080 -p 8090:8090 wrongsecrets-preview

🚀 Alternative - Pull from Registry:

docker pull ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2771-70c6ea0
docker run -p 8080:8080 -p 8090:8090 ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2771-70c6ea0

Then visit: http://localhost:8080

📝 Changes in this PR:
- src/main/resources/explanations/challenge69_reason.adoc
- src/main/resources/explanations/challenge72_reason.adoc
- src/main/resources/explanations/challenge73_reason.adoc

Visual diff screenshots will be available shortly...


Preview built by GitHub Actions

@github-actions

Copy link
Copy Markdown

📸 Visual Diff Ready!

Screenshots comparing your changes with the main branch are available:

📁 Download Visual Diff Artifacts

🖼️ Included screenshots:

  • pr-home.png vs main-home.png - Welcome page comparison
  • pr-about.png vs main-about.png - About page comparison
  • pr-challenge.png vs main-challenge.png - Challenge page comparison

🔍 How to review:

  1. Download the artifact zip file
  2. Extract and compare the pr-* and main-* images side by side
  3. Look for visual differences in layout, styling, and content

💡 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 #2771

@commjoen
commjoen merged commit 682d8b9 into master Oct 26, 2026
21 checks passed
@commjoen
commjoen deleted the hotfix-ai-patterns branch October 26, 2026 08:27

This branch was successfully deployed

1 active deployment
github-pages — fdebb067 Deployed Oct 26, 2026 by commjoen via generate-static-preview #447
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.

1 participant