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
Added support for camerCase csp in app-config #24723
Conversation
Changed Packages
|
ce0c4af
to
659b23a
Compare
659b23a
to
b5ea989
Compare
Signed-off-by: Vincenzo Scamporlino <vincenzos@spotify.com>
b5ea989
to
2a61a25
Compare
Signed-off-by: Vincenzo Scamporlino <vincenzos@spotify.com>
2a61a25
to
a1dc547
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.
🎉 👍
@@ -47,7 +47,7 @@ describe('readHelmetOptions', () => { | |||
csp: { | |||
key: ['value'], | |||
'img-src': false, | |||
'script-src-attr': ['custom'], | |||
scriptSrcAttr: ['custom'], |
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.
Could we keep a test that makes sure we still support the old format too? Maybe with a deprecation warning too?
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.
ops good catch
Signed-off-by: Vincenzo Scamporlino <vincenzos@spotify.com>
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.
Nice! 👍
Let's 🎉
Thank you for contributing to Backstage! The changes in this pull request will be part of the |
Uffizzi Cluster |
Hey, I just made a Pull Request!
This PR adds support for camelCase CSP directives in app-config.
Fixes #24716
✔️ Checklist
Signed-off-by
line in the message. (more info)