email_header_injection

Strip CRLF from email header values before sending trivial effort

// WRONG
mailer.sendMail({ to: req.query.to, subject: req.query.subject });

// RIGHT — strip \r\n or use a framework that does it for you
const safe = (v) => String(v).replace(/[\r\n]/g, "");
mailer.sendMail({ to: safe(req.query.to), subject: safe(req.query.subject) });

Suppress a confirmed non-issue with a committed .pullguardignore entry (pullguard ignore locally, or comment /pullguard ignore <fingerprint> <reason> on the PR — the fingerprint is printed in the PR comment’s Triage section). Entries support expiresAt for time-boxed snoozes.

Security findings at major or critical severity — and any critical finding — always surface: .pullguardignore cannot hide them. The reviewed paths that keep them visible are acknowledged (reviewed, stays in reports) and, for a confirmed false positive, a reasoned false_positive entry — visible and audited, excluded only from the merge block.