Skip to content

feat: add helmet security headers via separate config module - Fixes …#205

Merged
imanchalsingh merged 1 commit into
imanchalsingh:mainfrom
jikrana1:feat/add-helmet-config-module-198
Jul 8, 2026
Merged

feat: add helmet security headers via separate config module - Fixes …#205
imanchalsingh merged 1 commit into
imanchalsingh:mainfrom
jikrana1:feat/add-helmet-config-module-198

Conversation

@jikrana1

Copy link
Copy Markdown
Contributor

🛡️ Issue: #198 - Add Helmet Middleware for HTTP Security Headers

📝 Summary

This PR adds the helmet middleware to improve security by setting secure HTTP headers. The configuration is isolated in a dedicated module (src/config/helmet.js).

✨ Changes Made:

  • Created: src/config/helmet.js – dedicated module for Helmet configuration.
  • Refactored: app.js now imports and uses helmetMiddleware.
  • Verified: CORS, Socket.IO, and all existing endpoints work without any issues.

🔒 Security Headers Added:

  • X-Content-Type-Options, X-Frame-Options, X-XSS-Protection, Strict-Transport-Security, Referrer-Policy.

✅ Benefits:

  • Follows security best practices.
  • Configuration is centralized and easy to customize later.

🔗 Related Issue:

Closes #198

@vercel

vercel Bot commented Jun 30, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the imanchalsingh Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented Jun 30, 2026

Copy link
Copy Markdown

👷 Deploy request for file-sharingsystem pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit f54387e

@github-actions github-actions Bot added SSoC26 Official label of org backend and removed SSoC26 Official label of org labels Jun 30, 2026
@imanchalsingh imanchalsingh added SSoC26 Official label of org Easy Point - 20 Medium Point - 30 and removed Easy Point - 20 labels Jul 8, 2026
@imanchalsingh imanchalsingh merged commit e9ca047 into imanchalsingh:main Jul 8, 2026
4 of 5 checks passed
@jikrana1 jikrana1 deleted the feat/add-helmet-config-module-198 branch July 8, 2026 05:30
@jikrana1

jikrana1 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

Hi @imanchalsingh ,
I contributed to this project and PR was merged, but I don't see my name in the contributors list.
Could you please add it when you get a chance?
Thanks!

@imanchalsingh

Copy link
Copy Markdown
Owner

Hi @imanchalsingh , I contributed to this project and PR was merged, but I don't see my name in the contributors list. Could you please add it when you get a chance? Thanks!

Wait for leaderboard updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Medium Point - 30 SSoC26 Official label of org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Helmet Middleware to Improve HTTP Security Headers

2 participants