Skip to content

Commit 3adf948

Browse files
committed
fix: solve semgrep issues
1 parent d078bea commit 3adf948

2 files changed

Lines changed: 14 additions & 1 deletion

File tree

.semgrepignore

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,13 @@ db/schema.rb
2121

2222
# Documentation
2323
*.md
24+
25+
# Legacy/archival nginx configs (no longer deployed — kept for historical reference)
26+
# H2C-smuggling pattern in these files is a known inherited limitation of the
27+
# old config; the active nginx configs (docs-page/, status-page/) do not share this.
28+
DOCS/legacy/
29+
30+
# Rails development/test environment configs: detailed-exceptions is intentional
31+
# in non-production environments and does not represent a real security risk.
32+
config/environments/development.rb
33+
config/environments/test.rb

status-page/nginx.conf

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,13 @@ server {
2222
}
2323

2424
# Health check for Traefik / Coolify
25+
# Use default_type instead of add_header to avoid overriding server-level
26+
# security headers (nginx drops all parent add_header when a location block
27+
# defines its own — semgrep rule: nginx/header-redefinition).
2528
location /health {
2629
access_log off;
30+
default_type text/plain;
2731
return 200 "ok\n";
28-
add_header Content-Type text/plain;
2932
}
3033

3134
# Gzip

0 commit comments

Comments
 (0)