Skip to content

feat(lab1): juice shop deploy + triage report + PR template + CI smoke test - #1

Merged
Wilikson173 merged 2 commits into
mainfrom
feature/lab1
Jun 12, 2026
Merged

feat(lab1): juice shop deploy + triage report + PR template + CI smoke test#1
Wilikson173 merged 2 commits into
mainfrom
feature/lab1

Conversation

@Wilikson173

Copy link
Copy Markdown
Owner

Goal

Deploy OWASP Juice Shop v20.0.0 locally, produce a triage report of the attack surface, bootstrap the PR template workflow, and add a CI smoke-test that verifies Juice Shop health on every PR.

Changes

  • submissions/lab1.md β€” full triage report with deployment details, security headers analysis, and top 3 risks mapped to OWASP Top 10:2025
  • .github/PULL_REQUEST_TEMPLATE.md β€” PR template with Goal / Changes / Testing / Artifacts sections and 3-item checklist
  • .github/workflows/lab1-smoke.yml β€” GitHub Actions workflow that pulls Juice Shop v20.0.0, waits up to 60s for readiness, and fails the build if homepage returns non-200

Testing

docker run -d --name juice-shop -p 127.0.0.1:3000:3000 bkimminich/juice-shop:v20.0.0

docker ps --filter name=juice-shop --format 'table {{.Names}}\t{{.Status}}\t{{.Ports}}'
juice-shop Up 9 minutes 127.0.0.1:3000->3000/tcp
curl -s -o /dev/null -w "HTTP %{http_code}\n" http://127.0.0.1:3000/
HTTP 200
curl -s http://127.0.0.1:3000/rest/admin/application-version
{"version":"20.0.0"}
curl -s http://127.0.0.1:3000/api/Products | jq '.data | length'
46

Artifacts & Screenshots

  • submissions/lab1.md β€” triage report
  • .github/PULL_REQUEST_TEMPLATE.md β€” PR template (auto-filled this description)
  • .github/workflows/lab1-smoke.yml β€” CI smoke test

Checklist

  • Title follows feat(labN): <topic> style
  • No secrets or large temp files committed
  • Submission file exists at submissions/lab1.md

@Wilikson173
Wilikson173 merged commit 2f82cb6 into main Jun 12, 2026
1 check passed
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