You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
discovery(nginx): security-headers (CSP, X-Frame, etc.) duplicated at 3+ locations in autobot-slm.conf.j2 — DRY via include snippet #6842
Multiple location blocks in autobot-slm.conf.j2 repeat the same set of add_header lines:
Line 44 (server-level default)
Line 269 (Grafana fallback)
Line 288 (Prometheus fallback)
Each block has CSP + X-Frame-Options + X-Content-Type-Options + X-XSS-Protection + Referrer-Policy. PR #6751 had to update CSP at all three sites; the next change will too.
Commit:f853a15 — fix(nginx): DRY security headers via nginx snippets (#6842)
Acceptance Criteria Met:
✅ Two snippet files deployed via Ansible template tasks — autobot-security-headers-strict.conf.j2 + autobot-security-headers-iframe-host.conf.j2; Ansible ensures /etc/nginx/snippets/ directory exists
✅ All duplicate locations replaced with include directive — server level (strict), /grafana/api/live/ (iframe-host), /grafana/ (iframe-host), and colocated / location (strict)
✅ CSP observable on /, /login, /api/* via server-level strict snippet; /grafana/ via iframe-host snippet; /prometheus/ inherits server-level (no add_header override)
Discovery filed:#7737 — /slm/index.html sub-location still drops security headers (pre-existing gap; requires this issue's snippets as prerequisite for fix)
Activity
fix(nginx): DRY security headers via nginx snippets (#6842) (#7736)
mrveiss commentedon May 16, 2026
✅ Closed — fix merged via PR #7736
Commit: f853a15 — fix(nginx): DRY security headers via nginx snippets (#6842)
Acceptance Criteria Met:
Discovery filed: #7737 — /slm/index.html sub-location still drops security headers (pre-existing gap; requires this issue's snippets as prerequisite for fix)