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.

Sign up for GitHub

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

Jump to bottom

feat: Add security headers #768

Merged

ilayda-cp merged 2 commits into main from WD-14417-add-security-headers

Sep 10, 2024

Contributor

ilayda-cp commented Sep 9, 2024 •

edited

Loading

Done

Added security headers

How to QA

go to HTTP header analyzer
verify all the security headers exist.
go to https://jaas-ai-768.demos.haus/
Verify all the resources are shown correctly
Verify there is no behavior change (such as a link doesnt open)

Testing

This PR has tests
No testing required (explain why): added security headers

Issue / Card

Fixes #14417


          feat: Add security headers

f770ab5

ilayda-cp added Review: QA needed Review: Code needed labels

ilayda-cp self-assigned this

Collaborator

webteam-app commented Sep 9, 2024

Demo

Jenkins

demos.haus


          feat: Add security headers

8506b7f

steverydz approved these changes

View reviewed changes

Contributor

steverydz left a comment

LGTM 👍

steverydz added Review: Code +1 Review: QA +1 and removed Review: QA needed Review: Code needed labels

ilayda-cp merged commit 7505174 into main

10 checks passed

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

Labels

Review: Code +1 Review: QA +1