Repository navigation
fix(security): dismiss 4 CWE-116 changelog script alerts as S3 - #846
Conversation
…135) All 4 js/incomplete-sanitization alerts flag pipe-only escaping in developer tooling scripts that write markdown documentation. Output goes to static .md files (docs/changelog/evidence/), not RegExp, shell exec, or SQL. Rated S3 per triage rubric (display-only output). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 8 minutes and 50 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
js/incomplete-sanitization(CWE-116) HIGH alerts in changelog analysis scriptsAlert Details
detect-abandoned-approaches.ts:192detect-abandoned-approaches.ts:193detect-multiple-attempts.ts:219track-file-evolution.ts:174Key distinction: Alert #28 (same CWE-116 rule) was S2/FIXED because its output fed into
new RegExp(). These 4 output exclusively to static.mddocumentation files.Changes
docs/security/CODEQL_ALERT_LOG.md— added Round 5 section with 4 dismissed alertstasks/security-triage-batch-7.md— per-alert triage analysisTest plan
🤖 Generated with Claude Code