Skip to content

Fix for context & logging for challenge 74 - #2694

Merged
commjoen merged 1 commit into
masterfrom
fix-logging-and-ctx
Sep 26, 2026
Merged

commjoen merged 1 commit into
masterfrom
fix-logging-and-ctx

Conversation

@commjoen

@commjoen commjoen commented Sep 26, 2026 •

Copy link
Copy Markdown
Collaborator

What kind of changes does this PR include?

  • Fixes or refactors: update for challenge74: now with bigger ctx on the model & far less agressive logging
  • A new challenge
  • Additional documentation
  • 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

🦙 Llama Container Preview Ready!

You can test this container preview locally using the following commands:

# 1. Log in to GitHub Container Registry (if not already authenticated)
echo "YOUR_GITHUB_TOKEN" | docker login ghcr.io -u \commjoen --password-stdin

# 2. Run the preview container
docker run --rm \
  --name wrongsecrets-llama \
  --memory=2g \
  -p 1234:1234 \
  ghcr.io/github.repository/wrongsecrets-llamaserver-pr:pr-{{ github.event.pull_request.number }}

Note: The specific tag for this specific commit is also available using the commit-SHA suffix if needed.

@github-actions

Copy link
Copy Markdown

🌐 GitHub Pages Preview Ready!

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

📄 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-2694-6460469

📦 Download & Test Locally:

  1. 📁 Download Docker Image Artifact (look for wrongsecrets-preview-pr-2694)
  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-2694-6460469
docker run -p 8080:8080 -p 8090:8090 ghcr.io/owasp/wrongsecrets/wrongsecrets-pr:pr-2694-6460469

Then visit: http://localhost:8080

📝 Changes in this PR:
- src/main/java/org/owasp/wrongsecrets/challenges/kubernetes/llama/LlamaService.java

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 #2694

@commjoen
commjoen merged commit 0beb10e into master Sep 26, 2026
21 checks passed
@commjoen
commjoen deleted the fix-logging-and-ctx branch September 26, 2026 22:10

This branch was successfully deployed

1 active deployment
github-pages — c9c7f029 Deployed Sep 26, 2026 by commjoen via generate-static-preview #376
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