Skip to content

feat(lab5): ZAP baseline + auth + Semgrep + correlation#1267

Open
Philip-78 wants to merge 2 commits into
inno-devops-labs:mainfrom
Philip-78:feature/lab5
Open

feat(lab5): ZAP baseline + auth + Semgrep + correlation#1267
Philip-78 wants to merge 2 commits into
inno-devops-labs:mainfrom
Philip-78:feature/lab5

Conversation

@Philip-78

Copy link
Copy Markdown

Goal

Run DAST (ZAP unauthenticated + authenticated) and SAST (Semgrep) against Juice Shop, then correlate findings across both tools.

Changes

  • submissions/lab5.md β€” ZAP baseline + auth scan tables, Semgrep top-10 rules, SAST/DAST correlation

Testing

  • ZAP baseline: 10 unique alert types (no SQL Injection found)
  • ZAP authenticated: 12 unique alert types including SQL Injection (High)
  • Semgrep: 22 findings (12 ERROR, 10 WARNING) across 1000 files with 533 rules
  • Correlated finding: SQL Injection confirmed by both express-sequelize-injection (Semgrep) and ZAP active scan on /rest/products/search

Artifacts & Screenshots

  • submissions/lab5.md β€” full analysis

  • Task 1 β€” ZAP baseline + auth + 10-20Γ— ratio analysis
  • Task 2 β€” Semgrep top-10 + triage shortcut + false positive sample
  • Bonus β€” Correlation table with confirmed cross-tool SQL Injection finding

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