diff --git a/README.md b/README.md index aed72f0..a86ccaa 100644 --- a/README.md +++ b/README.md @@ -1640,6 +1640,10 @@ authenticity. termination. - **CORS**: The server allows all origins by default (`Access-Control-Allow-Origin: *`). Restrict this in production via reverse proxy configuration if needed. +- **Content-Security-Policy**: The server sets a strict CSP header on all + responses, restricting resource loading to same-origin and disabling + dangerous features (object embeds, framing, base tag injection). The + embedded SPA works without `'unsafe-inline'` for scripts or styles. --- diff --git a/internal/middleware/middleware.go b/internal/middleware/middleware.go index e8260ca..e2af1d2 100644 --- a/internal/middleware/middleware.go +++ b/internal/middleware/middleware.go @@ -180,3 +180,36 @@ func (mware *Middleware) MetricsAuth() func(http.Handler) http.Handler { }, ) } + +// cspPolicy is the Content-Security-Policy header value applied to all +// responses. The embedded SPA loads scripts and styles from same-origin +// files only (no inline scripts or inline style attributes), so a strict +// policy works without 'unsafe-inline'. +const cspPolicy = "default-src 'self'; " + + "script-src 'self'; " + + "style-src 'self'; " + + "connect-src 'self'; " + + "img-src 'self'; " + + "font-src 'self'; " + + "object-src 'none'; " + + "frame-ancestors 'none'; " + + "base-uri 'self'; " + + "form-action 'self'" + +// CSP returns middleware that sets the Content-Security-Policy header on +// every response for defense-in-depth against XSS. +func (mware *Middleware) CSP() func(http.Handler) http.Handler { + return func(next http.Handler) http.Handler { + return http.HandlerFunc( + func( + writer http.ResponseWriter, + request *http.Request, + ) { + writer.Header().Set( + "Content-Security-Policy", + cspPolicy, + ) + next.ServeHTTP(writer, request) + }) + } +} diff --git a/internal/server/routes.go b/internal/server/routes.go index 9cc0103..ced61e4 100644 --- a/internal/server/routes.go +++ b/internal/server/routes.go @@ -29,6 +29,7 @@ func (srv *Server) SetupRoutes() { } srv.router.Use(srv.mw.CORS()) + srv.router.Use(srv.mw.CSP()) srv.router.Use(middleware.Timeout(routeTimeout)) if srv.sentryEnabled {