Catalog
addyosmani/security-and-hardening

addyosmani

security-and-hardening

Hardens code against vulnerabilities. Use when auditing an input handler for vulnerabilities, when handling user input, authentication, data storage, or external integrations, or when checking a login flow is safe against the OWASP Top Ten. Use when building any feature that accepts untrusted data, manages user sessions, or interacts with third-party services. Use when auditing dependencies for known vulnerabilities, triaging package-manager audit findings, or assessing supply-chain risk in a new package. Use when personal data or privacy compliance (GDPR, CCPA) is involved.

NewUpdated Oct 29, 2026

Security and Hardening

Overview

Security-first development practices for web applications. Treat every external input as hostile, every secret as sacred, and every authorization check as mandatory. Security isn't a phase — it's a constraint on every line of code that touches user data, authentication, or external systems.

When to Use

  • Building anything that accepts user input
  • Implementing authentication or authorization
  • Storing or transmitting sensitive data
  • Integrating with external APIs or services
  • Adding file uploads, webhooks, or callbacks
  • Handling payment or PII data

Process: Threat Model First

Controls bolted on without a threat model are guesses. Before hardening, spend five minutes thinking like an attacker:

  1. Map the trust boundaries. Where does untrusted data cross into your system? HTTP requests, form fields, file uploads, webhooks, third-party APIs, message queues, and LLM output — plus the local values that look internal because the OS handed them to you: another process's command line or environment, filenames on a shared volume, a path in a job payload. Trust follows who wrote a value, not which channel delivered it. Every boundary is attack surface.
  2. Name the assets. What's worth stealing or breaking? Credentials, PII, payment data, admin actions, money movement.
  3. Run STRIDE over each boundary — a quick lens, not a ceremony:
Threat Ask Typical mitigation
Spoofing Can someone impersonate a user/service? Authentication, signature verification
Tampering Can data be altered in transit or at rest? Integrity checks, parameterized queries, HTTPS
Repudiation Can an action be denied later? Audit logging of security events
Information disclosure Can data leak? Encryption, field allowlists, generic errors
Denial of service Can it be overwhelmed? Rate limiting, input size caps, timeouts
Elevation of privilege Can a user gain rights they shouldn't? Authorization checks, least privilege
  1. Write abuse cases next to use cases. For each feature, ask "how would I misuse this?" — then make that your first test.

If you can't name the trust boundaries for a feature, you're not ready to secure it. This is OWASP A04: Insecure Design — most breaches begin in design, not code.

The Three-Tier Boundary System

Always Do (No Exceptions)

  • Validate all external input at the system boundary (API routes, form handlers)
  • Parameterize all database queries — never concatenate user input into SQL
  • Encode output to prevent XSS (use framework auto-escaping, don't bypass it)
  • Use HTTPS for all external communication
  • Hash passwords with bcrypt/scrypt/argon2 (never store plaintext)
  • Set security headers (CSP, HSTS, X-Frame-Options, X-Content-Type-Options)
  • Use httpOnly, secure, sameSite cookies for sessions
  • Run the detected package manager's native audit against the committed lockfile before every release

Ask First (Requires Human Approval)

  • Adding new authentication flows or changing auth logic
  • Storing new categories of sensitive data (PII, payment info)
  • Adding new external service integrations
  • Changing CORS configuration
  • Adding file upload handlers
  • Modifying rate limiting or throttling
  • Granting elevated permissions or roles

Never Do

  • Never commit secrets to version control (API keys, passwords, tokens)
  • Never log sensitive data (passwords, tokens, full credit card numbers)
  • Never trust client-side validation as a security boundary
  • Never disable security headers for convenience
  • Never use eval() or innerHTML with user-provided data
  • Never store sessions in client-accessible storage (localStorage for auth tokens)
  • Never expose stack traces or internal error details to users

Hardening Controls

The rules below are the workflow; a concrete implementation of each lives in references/hardening-patterns.md. Open the section you need when you reach that code, not before.

Injection, XSS, and access control

  • Parameterize every query. Never build SQL, NoSQL, or shell commands from input strings.
  • Encode output through the framework's auto-escaping. If raw HTML is unavoidable, sanitize with an allowlist sanitizer first.
  • Check authorization on every request, not just authentication: the authenticated user must own, or be permitted on, the specific resource (A01, IDOR).

Patterns: Injection, XSS, Access control.

Authentication and sessions

  • Hash passwords with bcrypt (≥12 rounds), scrypt, or argon2. The session secret comes from the environment, never from code.
  • Session cookies are httpOnly, secure, and sameSite: 'lax' or 'strict' (the CSRF defense; 'none' sends the cookie on cross-site requests), with a bounded maxAge.

Pattern: Authentication.

Headers, CORS, and responses

  • Security headers on every response (helmet or the framework equivalent); CSP starts from default-src 'self' and is tightened, not loosened.
  • CORS restricted to an explicit origin list from configuration. Never * with credentials.
  • Strip sensitive fields (passwordHash, reset tokens) before any response. Error bodies are generic; internals go to server logs only.

Patterns: Misconfiguration, Sensitive data exposure.

Input validation and uploads

  • Validate at the boundary with a schema: allowlisted shape, lengths, enums, formats. Reject with 422 and structured details; downstream code uses only the parsed, typed value.
  • Uploads: allowlist MIME types, cap size, verify content (magic bytes) when it matters. The extension proves nothing.

Patterns: Schema validation, File upload.

Server-side fetches (SSRF)

Any URL the user influences — webhooks, import-from-URL, image proxies, link previews — can be aimed at internal services. Allowlist scheme and host, resolve all DNS records and reject any private or reserved address (loopback, link-local 169.254.169.254, private, unique-local, for IPv4 and IPv6), and forbid redirects. That check still has a DNS-rebinding TOCTOU gap: for high-risk surfaces, pin the resolved IP or put a filtering agent in front.

Pattern: SSRF.

Destructive operations on derived paths

A delete, move, or overwrite is only as safe as the value naming its target, and trust follows who wrote that value, not which channel delivered it: another process's command line is as attacker-controlled as a form field. A shape check proves well-formedness, not authorization. Before the call, require all three: the resolved target (symlinks resolved) sits under an allowlisted root; it is at least one level below that root; and it carries ownership evidence read before the operation. On refusal, log the rejected target and stop; never fall back to a broader default path.

Why the check is weaker than it reads (marker self-attestation, check/use races): Destructive paths. Worked code: ../../references/security-checklist.md.

Rate limiting

Limit the API generally and auth endpoints strictly (about 10 attempts per 15 minutes). Once more than one process serves traffic, in-memory counters silently become max × instances, or never fire on serverless: back the limiter with a shared store.

Pattern: Rate limiting.

Secrets

Secrets come from the environment. .env.example is committed with placeholders; real .env* files and key material are gitignored; grep the staged diff before committing. A secret that reaches a remote is compromised the moment it lands: rotate it first, then purge history.

Pattern: Secrets management.

Dependencies and supply chain

  1. Find the installation boundary and manager. Use the workspace root that owns the lockfile, or an independent nested project only when it is outside that workspace. Corroborate packageManager (when present), the lockfile, and CI; stop on disagreement or competing lockfiles. Pin the manager version.
  2. Block dependency scripts before first execution. Bootstrap with scripts disabled or a documented fail-closed policy, inspect the pending script source, approve only the minimum, commit the policy, then verify with a clean frozen/immutable install. Never blanket-approve.
  3. Run the native audit against the committed lockfile before every release. Triage critical/high by reachability (runtime, build, test, deploy paths) and fix availability. Never apply forced remediation (npm audit fix --force or equivalent) automatically, since forced fixes may cross declared dependency ranges; preview, read changelogs, test each upgrade. Document every deferral with a reason and a review date.
  4. Audits only match known advisories. They do not catch a newly malicious or typosquatted package (cross-env vs crossenv). Review new dependencies, lockfile diffs, and script-policy changes together: ownership, maintenance, release age, provenance, transitive graph. Verify registry signatures where supported (npm audit signatures, pnpm audit signatures) and treat their absence as a signal to investigate, not automatic proof of compromise (A06, LLM03).

Triage decision tree: Dependency audit triage. Manager matrix and install-script gate: ../../references/security-checklist.md.

Personal data and privacy

Hardening asks "can an attacker read it?" Privacy asks "should we hold it at all, and for how long?" The cheapest data to protect, breach, and comply over is the data you never collected; treat personal data as a liability to minimize.

  • Classify fields as you add them (non-personal, PII, sensitive) and handle each class accordingly. You cannot protect, or honor a deletion request for, data you cannot find.
  • Collect only against a stated purpose. "Might be useful later" is latent breach scope, not a purpose. Keep PII out of telemetry (the observability-and-instrumentation skill makes the same point from the ops side).
  • Set retention up front, then actually delete. Every personal-data store needs a TTL and a working deletion path, including backups, caches, search indexes, and analytics copies.
  • Support the data-subject rights your jurisdiction requires (GDPR, CCPA, and kin): export, correct, delete. Design the schema so a user's data is findable and erasable, not smeared irreversibly across systems.
  • Consent gates collection and third-party sharing, and is auditable. Sending PII to an analytics, ad, or LLM vendor is sharing; the vendor needs a data-processing agreement. Make region a configurable policy, not a hardcoded assumption.

Classification table: Data classification. A privacy incident starts the breach-notification clock; run the postmortem with the debugging-and-error-recovery skill.

AI / LLM features

Calling an LLM — chatbots, summarizers, agents, RAG — adds a new attack surface; map it to the OWASP Top 10 for LLM Applications (2025):

  • Model output is untrusted input (LLM05). Never into eval, SQL, a shell, innerHTML, or a file path; parse defensively, validate against a schema, then encode.
  • Prompts can be hijacked (LLM01). Untrusted text in the context — a user message, a fetched page, a PDF — can carry instructions. The system prompt is not a security boundary; enforce permissions in code.
  • Keep secrets, other tenants' data, and the full system prompt out of the context window (LLM02, LLM07); scope tool permissions, validate every tool argument, and confirm destructive actions (LLM06); cap tokens, request rate, and recursion depth (LLM10); partition RAG embeddings per tenant and validate documents before indexing (LLM08).

Pattern: LLM output handling.

Review Checklist

Before sign-off, walk ../../references/security-checklist.md: it covers authentication, authorization, input, data protection and privacy, headers and CORS, dependencies and supply chain, AI/LLM, and error handling, plus the OWASP quick-reference tables.

Common Rationalizations

Rationalization Reality
"This is an internal tool, security doesn't matter" Internal tools get compromised. Attackers target the weakest link.
"We'll add security later" Security retrofitting is 10x harder than building it in. Add it now.
"No one would try to exploit this" Automated scanners will find it. Security by obscurity is not security.
"The framework handles security" Frameworks provide tools, not guarantees. You still need to use them correctly.
"It's just a prototype" Prototypes become production. Security habits from day one.
"Threat modeling is overkill here" Five minutes of "how would I attack this?" prevents the design flaws no control can patch later.
"It's just LLM output, it's only text" That "text" can be a SQL statement, a script tag, or a shell command. Treat it like any untrusted input.
"The audit passed, so the dependency is safe" Audits match known advisories. They do not detect a newly malicious package or make unreviewed install scripts safe to execute.
"Collect it now, we might need it later" Data you don't hold can't be breached, subpoenaed, or mis-deleted. "Might need it" is breach scope, not a purpose.
"We'll handle deletion requests manually" Manual erasure misses backups, caches, and analytics copies. If the schema can't find a user's data, you can't honor the request — design for it.
"Compliance is legal's problem, not ours" Export, deletion, retention, and consent are schema and code. Legal can't bolt them on after you've smeared PII across ten systems.

Red Flags

  • User input passed directly to database queries, shell commands, or HTML rendering
  • A delete, move, or overwrite whose target comes from a payload, a config value, or another process's command line, guarded only by a shape check on the path
  • Secrets in source code or commit history
  • API endpoints without authentication or authorization checks
  • Missing CORS configuration or wildcard (*) origins
  • No rate limiting on authentication endpoints, or an in-memory limiter in front of more than one instance
  • Stack traces or internal errors exposed to users
  • Dependencies with known critical vulnerabilities, competing lockfiles at one installation boundary, non-reproducible installs, or blanket-approved scripts
  • Server fetches user-supplied URLs without an allowlist (SSRF)
  • LLM/model output passed into a query, the DOM, a shell, or eval
  • Secrets, PII, or the full system prompt placed inside an LLM context window
  • Personal data collected with no stated purpose, retention limit, or deletion path
  • PII sent to analytics/ad/LLM vendors with no consent or data-processing agreement
  • "Delete my account" that only flips a flag while the personal data lingers in stores and backups

Verification

After implementing security-relevant code:

  • The native audit has no unmitigated reachable critical/high findings; CI preserves the authoritative lockfile and blocks unreviewed dependency scripts
  • No secrets in source code or git history
  • All user input validated at system boundaries
  • Destructive filesystem operations resolve symlinks, then verify allowlisted root, minimum depth, and ownership before running
  • Authentication and authorization checked on every protected endpoint
  • Security headers present in response (check with browser DevTools)
  • Error responses don't expose internal details
  • Rate limiting active on auth endpoints, backed by a shared store when more than one instance serves traffic
  • Server-side URL fetches validated against an allowlist (no SSRF)
  • LLM/model output validated and encoded before use (if AI features present)
  • Personal data is classified, minimized to a stated purpose, and has a retention limit
  • Deletion and export requests work end-to-end (including backups, caches, and analytics copies)
Files2
2 files · 13.9 KB

Select a file to preview

Overall Score

88/100

Grade

A

Excellent

Grades are signals, not a certification. Always review a skill yourself before use.

Safety

87

Quality

92

Clarity

85

Completeness

86

Summary

Security and Hardening is a comprehensive skill that teaches threat modeling, OWASP Top 10 defense patterns, and secure development practices for web applications. It guides agents through building security-first code by establishing threat models first, then applying concrete hardening controls across authentication, input validation, data protection, dependency management, and AI/LLM safety. The skill references a supporting patterns document with TypeScript code examples for each control.

Static Analysis Findings

2 findings

Patterns detected by deterministic static analysis before AI scoring. Hover over any finding code for detailed information and remediation guidance.

Credential Exposure
SEC-020Direct .env File Access12x in 2 files

Direct .env file access

SKILL.md.env2x
references/hardening-patterns.md.env10x
Command Injection
SEC-011Dynamic Shell Eval2x in 1 file

Shell eval/exec of dynamic content

SKILL.mdeval`2x

Detected Capabilities

file readreference documentationthreat modeling guidancecode pattern examplessecurity checklist templatesdependency audit decision makingdata classificationLLM output validation

Trigger Keywords

Phrases that agents use to match this skill to user intent.

audit for vulnerabilitiessecure user inputowasp complianceimplement authenticationprotect sensitive datareview access controlharden api endpointsaudit dependenciesprivacy compliancesecure llm integration

Risk Signals

INFO

SEC-011: Shell eval/exec of dynamic content

SKILL.md | Lines: "Never use `eval()` or `innerHTML`" (listed in Never Do section)
INFO

SEC-011: Shell eval/exec of dynamic content

references/hardening-patterns.md | XSS section example shows BAD usage of eval pattern for reference (as anti-pattern)
INFO

SEC-020: Direct .env file access

SKILL.md | Secrets section: ".env.example is committed with placeholders; real `.env*` files and key material are gitignored"
INFO

SEC-020: Direct .env file access

references/hardening-patterns.md | Secrets Management section: documents .env, .env.local gitignore patterns and `git diff --cached` pre-commit checks

Referenced Domains

External domains referenced in skill content, detected by static analysis.

genai.owasp.orglocalhost

Use Cases

  • Audit user input handlers and form validation for injection vulnerabilities
  • Implement secure authentication and session management with password hashing and secure cookies
  • Design access control checks to prevent privilege escalation (IDOR, broken authz)
  • Harden API endpoints with security headers, CORS policy, and rate limiting
  • Validate file uploads by type, size, and content before storage
  • Protect against SSRF by validating server-side URL fetches against allowlists
  • Audit dependencies for known vulnerabilities and manage supply-chain risk
  • Implement data classification and deletion workflows for GDPR/CCPA compliance
  • Secure LLM integrations by validating and encoding model output before use
  • Review destructive filesystem operations (delete, move, overwrite) for path traversal risks

Quality Notes

  • Excellent threat modeling framework (STRIDE) that connects design decisions to implementation
  • Clear tier system (Always Do, Ask First, Never Do) with actionable boundaries
  • Comprehensive OWASP Top 10 coverage with prevention patterns, not just warnings
  • Strong emphasis on context over rules — e.g., explaining why eval is dangerous rather than just forbidding it
  • References documentation (hardening-patterns.md) are well-structured with concrete TypeScript examples
  • Extensive coverage of often-neglected areas: destructive path operations, SSRF validation, supply-chain risk, privacy/GDPR
  • LLM security section ties OWASP Top 10 for LLM Applications to practical defense patterns
  • Red Flags section distills common mistakes into scannable checklist
  • Verification checklist at end provides sign-off criteria
  • Common Rationalizations table addresses real objections developers raise
  • Secrets management guidance correctly emphasizes rotation over just history rewriting
  • TOCTOU gap in SSRF validation is explicitly documented, showing realistic threat awareness
Model: claude-haiku-4-5-20251001Analyzed: Oct 29, 2026

Reviews

Add this skill to your library to leave a review.

No reviews yet

Be the first to share your experience.

Version History

  1. v2.4

    Content updated

    ✦ AIHardening patterns reference updated; SKILL.md body unchanged.

    2026-10-29

    LATEST
  2. v2.3

    Content updated

    ✦ AIReference documentation updated in hardening-patterns.md.

    2026-10-11

    View This Version
  3. v2.2

    Content updated

    ✦ AIClarifies session cookie `sameSite` values, adds rationale for avoided forced dependency fixes, refines privacy-deletion and consent-audit language, and expands LLM prompt-injection mitigations with…

    2026-09-23

    View This Version
  4. v2.1

    Content updated

    ✦ AISKILL.md substantially reorganized; references/hardening-patterns.md added with detailed patterns reference.

    2026-09-21

    View This Version
  5. v2.0

    Contract changed: description

    ✦ AIExpands activation scope to include audit of input handlers and login flows against OWASP Top Ten; adds detailed guidance on destructive path operations, dependency audit triage, and supply-chain…

    triggering2026-09-09

    View This Version
  6. v1.1

    Content updated

    ✦ AIAdds threat-modeling process, SSRF patterns, and supply-chain guidance; one SEC-011 finding.

    2026-06-28

    View This Version
  7. v1.0

    2026-05-02

    View This VersionInitial version

Use addyosmani/security-and-hardening in your dev environment

Command Palette

Search for a command to run...