-
Notifications
You must be signed in to change notification settings - Fork 113
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 or insecure "Content-Security-Policy" header (API) #5757
Conversation
✔️ Deploy Preview for chef-automate ready! 🔨 Explore the source changes: 83f5676 🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-automate/deploys/618c118ac81a7d0007a5d11f 😎 Browse the preview: https://deploy-preview-5757--chef-automate.netlify.app |
Kudos, SonarCloud Quality Gate passed! |
8a95162
to
789f293
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me
I added |
Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>
Signed-off-by: Venkatesh-rengasamy <vrengasa@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
789f293
to
83f5676
Compare
Kudos, SonarCloud Quality Gate passed! |
Signed-off-by: Venkatesh-rengasamy vrengasa@progress.com
🔩 Description: What code changed, and why?
Add Content-Security-Policy header to resolve blank screen after SAML signin.
⛓️ Related Resources
#4856
👍 Definition of Done
👟 How to Build and Test the Change
Login into SAML account and see you are getting home page insteadof blank screen.
✅ Checklist
All PRs must tick these:
With occasional exceptions, all PRs from Progress employees must tick these:
make spell
in any component directory)All PRs from Progress employees should tick these if appropriate:
Please add a note next to any checkbox above if you are NOT ticking it.
📷 Screenshots, if applicable