Skip to content

security: add helmet middleware for X-Frame-Options / X-Content-Type-Options / etc. #128

@sacha-l

Description

@sacha-l

From the post-#121 security audit.

Severity: nit

File(s): server/server.js

Why: server doesn't set X-Frame-Options, X-Content-Type-Options, or CSP. Railway terminates TLS and may set some at the edge, but defense-in-depth at the app layer is cheap.

Suggestion: drop app.use(helmet()) after the CORS middleware. No CSP customisation needed since the server only serves JSON, not HTML.

🤖 Generated with Claude Code

Metadata

Metadata

Assignees

No one assigned

    Labels

    claude-suggestedIssue suggested by the Claude agent during a /ship-issue run

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions