Fix cookie set without its protective attributes in Express

A cookie is written without `httpOnly`, `secure`, or `sameSite`. Missing `httpOnly` turns any cross-site scripting bug into session theft; missing `secure` sends the cookie over plain HTTP; missing `sameSite` attaches it to cross-site requests. A cookie holding no sensitive value may not need all three, which is why the finding names the ones it did not find rather than assuming the worst.

medium likely Express CWE-614 / OWASP A05:2021

The vulnerable pattern in Express

MEDIUM possible Cookie set without its protective attributes A05:2021 src/app.ts:28:3 26 │ 27 │ // insecure-cookie: no httpOnly, no secure, no sameSite. 28 │ res.cookie('session', rows.rows[0].id) │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ cookie written without httpOnly/secure/sameSite 29 │ 30 │ res.json({ ok: true })

This finding comes from the Express fixture in the owlwarden test suite. This cookie is missing protections: httpOnly keeps the cookie out of reach of JavaScript, so a cross-site scripting bug cannot read the session; secure stops the cookie being sent over plain HTTP; sameSite stops the browser attaching the cookie to cross-site requests.

The corrected handler

Pass the attributes to res.cookie.

res.cookie('session', token, {
  httpOnly: true,
  secure: process.env.NODE_ENV === 'production',
  sameSite: 'lax',
})

If you are not using Express

Set httpOnly, secure, and sameSite when writing a cookie that carries anything the user would not want read or replayed.

Check your own repository

npx owlwarden scan
npx owlwarden explain insecure-cookie

Runs on your machine. No account, no telemetry, no network unless you ask. In CI, SARIF uploads to code scanning and the exit code is the gate.

Other Express checks

Rules with a tested Express example.

insecure-cookie for every framework / All rules / owlwarden