P2: express.json() middleware runs on all requests including GET #14

New Issue

Closed

opened 2026-05-13 20:40:04 -05:00 by null · 0 comments

null commented

2026-05-13 20:40:04 -05:00

Owner

Severity: P2 — Medium

File: server/index.js ~line 256

Problem: express.json() middleware runs for all requests including GET /api/health. GET requests with body may cause unexpected behavior.

Impact: Minor — violates REST conventions, wastes minimal resources.

Fix: Move express.json() to only apply to POST routes, or use express.json() conditionally.

## Severity: P2 — Medium **File:** server/index.js ~line 256 **Problem:** express.json() middleware runs for all requests including GET /api/health. GET requests with body may cause unexpected behavior. **Impact:** Minor — violates REST conventions, wastes minimal resources. **Fix:** Move express.json() to only apply to POST routes, or use express.json() conditionally.