Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

feat: add missing csp #770

Merged
merged 1 commit into from
Sep 10, 2024
Merged

feat: add missing csp #770

merged 1 commit into from
Sep 10, 2024

Conversation

ilayda-cp
Copy link
Contributor

@ilayda-cp ilayda-cp commented Sep 10, 2024 •
edited
Loading

Done

  • Added security headers

How to QA

Testing

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

Issue / Card

Fixes #14417

@webteam-app
Copy link
Collaborator

Demo

Jenkins

demos.haus

@ilayda-cp ilayda-cp self-assigned this Sep 10, 2024
chillkang
chillkang approved these changes Sep 10, 2024
View reviewed changes
Copy link
Contributor

@chillkang chillkang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM👍

@ilayda-cp ilayda-cp merged commit afd9cca into main Sep 10, 2024
10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@chillkang chillkang chillkang approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ilayda-cp ilayda-cp

Labels

Review: Code +1 Review: QA +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ilayda-cp @webteam-app @chillkang