Skip to content

Conversation

@Tsopic
Copy link
Member

@Tsopic Tsopic commented Jan 2, 2026

Summary

  • Add security headers middleware (X-Frame-Options, X-Content-Type-Options, X-XSS-Protection, Referrer-Policy, Permissions-Policy)
  • Increase bcrypt cost factor from 10 to 12 for stronger password hashing
  • Replace vulnerable xlsx package with exceljs in frontend
  • Add error message sanitization to prevent internal details leakage in API responses
  • Create .env.example template documenting all environment variables

Test Plan

  • All Go tests pass (19 packages)
  • Frontend builds successfully
  • npm audit shows 0 vulnerabilities
  • cookie package override verified (1.1.1)

🤖 Generated with Claude Code

@codecov
Copy link

codecov bot commented Jan 2, 2026

Codecov Report

❌ Patch coverage is 85.00000% with 3 lines in your changes missing coverage. Please review.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   68.80%   68.83%   +0.02%     
==========================================
  Files          70       72       +2     
  Lines       10480    10499      +19     
==========================================
+ Hits         7211     7227      +16     
- Misses       2976     2978       +2     
- Partials      293      294       +1     
Flag Coverage Δ
integration 68.83% <85.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c20ea33...4a04f86. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Tsopic and others added 5 commits January 5, 2026 17:19
Adds X-Frame-Options, X-Content-Type-Options, X-XSS-Protection,
Referrer-Policy, and Permissions-Policy headers to all API responses.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Improves password hashing strength against brute-force attacks.
New cost factor is 4x slower to compute (2^12 vs 2^10 iterations).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fixes CVE prototype pollution and ReDoS vulnerabilities in xlsx package.
exceljs provides equivalent functionality with active maintenance.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Prevents leaking internal details (SQL errors, file paths, IP addresses)
in 5xx error responses. Validation errors remain user-friendly.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Documents all environment variables with descriptions and defaults.
Helps new developers understand required vs optional configuration.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Tsopic Tsopic force-pushed the security/fixes-2026-01-02 branch from e62f663 to 4a04f86 Compare January 5, 2026 15:20
@Tsopic Tsopic merged commit e4e7073 into main Jan 5, 2026
12 checks passed
@Tsopic Tsopic deleted the security/fixes-2026-01-02 branch January 5, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants