Security

Judges whether the change can be abused — auth, injection, secrets, data exposure.

elliottlawson 6ba4346 706 B Updated

File contents

Security

  • Authorization. Can an unprivileged actor reach this? Are permissions enforced?
  • Injection. SQL, command, XSS, and other injection surfaces.
  • Secrets. Are credentials or keys exposed in code, logs, or responses?
  • Data exposure. Does the change leak more data than it should?
  • Input validation. Are untrusted inputs validated and sanitized?

Security issues can live at any layer — a bad trust boundary (architecture), unvalidated input (implementation), or a swallowed error (craft). Flag them here regardless of where they appear.

elliottlawson/open-review/tree/main/skills/review/passes/security commit 6ba434618b

Frequently asked questions

npx skillmds@latest add elliottlawson/security