Skip to content

L-1: Missing Secure Flag on Session Cookie

Low
nick-pape published GHSA-5j35-xr4g-vwf4 Mar 22, 2026

Package

npm @grackle-ai/server (npm)

Affected versions

<=0.70.4

Patched versions

>=0.70.5

Description

Impact

The session cookie is set with HttpOnly; SameSite=Lax; Path=/ but does not include the Secure flag. This means the cookie will be sent over plain HTTP connections.

Since the server binds to 127.0.0.1 by default and uses HTTP (not HTTPS), this is acceptable for localhost use. However, when --allow-network is used to bind to 0.0.0.0, cookies could be transmitted over insecure network connections and intercepted by an attacker.

Affected code:

  • packages/server/src/session.ts:76 — cookie string lacks ; Secure attribute

Patches

Not yet patched.

Fix: Conditionally add ; Secure when served over HTTPS or when --allow-network is enabled:

const securePart = isHttps ? "; Secure" : "";
return `${SESSION_COOKIE_NAME}=${cookieValue}; HttpOnly; SameSite=Lax; Path=/${securePart}; Max-Age=${maxAge}`;

Workarounds

Do not use --allow-network over untrusted networks without a TLS-terminating reverse proxy.

References

  • OWASP: Secure Cookie Attribute
  • File: packages/server/src/session.ts

Severity

Low

CVE ID

No known CVE

Weaknesses

No CWEs