Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing security headers #616

Closed
cedneve opened this issue Feb 5, 2023 · 3 comments
Closed

Missing security headers #616

cedneve opened this issue Feb 5, 2023 · 3 comments
Labels
Under review Issues currently being reviewed

Comments

@cedneve
Copy link

cedneve commented Feb 5, 2023

As part of an external pentest, the following recommendation was formulated for Oxalis:

Configure the following HTTP headers:
• X-Content-Type-Options
• Referrer-Policy
• Permissions-Policy
• Content-Security-Policy
• X-Frame-Options
• Strict-Transport-Security (for HTTPS only)

It seems those security headers are missing in the HTTP responses leading to a medium security issue.

Could you add those or do you wish that we propose a fix to be merged into Oxalis to fix this ?

@dladlk
Copy link

dladlk commented Feb 6, 2023

image

@runekock
Copy link

runekock commented May 4, 2023

@cedneve
Those headers are for browsers. I don't think they accomplish anything in this context. If you disagree, please explain for each header why it is a good idea.

@aaron-kumar aaron-kumar added the Under review Issues currently being reviewed label Dec 9, 2023
@aaron-kumar
Copy link
Member

Things like "Strict-Transport-Security (for HTTPS only)" can be set it through Servlet container like Tomcat and e.g. through Cloudfront. Outside the scope of Oxalis.
Converting it to discussion, just in case you want to continue discussion...

@OxalisCommunity OxalisCommunity locked and limited conversation to collaborators Dec 9, 2023
@aaron-kumar aaron-kumar converted this issue into discussion #650 Dec 9, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
Under review Issues currently being reviewed
Projects
None yet
Development

No branches or pull requests

4 participants