-
Notifications
You must be signed in to change notification settings - Fork 142
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
Fix CSP so that the policy can be enforced and not just report only #346
Comments
dalelane
changed the title
Fix CSP policy so that the policy can be enforced and not just report only
Fix CSP so that the policy can be enforced and not just report only
Nov 5, 2020
dalelane
added a commit
that referenced
this issue
Nov 25, 2020
Contributes to: #346 Signed-off-by: Dale Lane <[email protected]>
taxinomitis/src/lib/restapi/config.ts Lines 52 to 54 in e665e30
|
taxinomitis/src/lib/restapi/config.ts Lines 59 to 62 in e665e30
|
Some of the errors that these are hiding are from angular - see https://docs.angularjs.org/api/ng/directive/ngCsp for details |
dalelane
added a commit
that referenced
this issue
Dec 4, 2020
Contributes to: #346 Signed-off-by: Dale Lane <[email protected]>
dalelane
added a commit
that referenced
this issue
Dec 11, 2020
Contributes to: #346 Signed-off-by: Dale Lane <[email protected]>
dalelane
added a commit
that referenced
this issue
Dec 12, 2020
Incremental improvements towards being able to enforce CSP properly. Contributes to: #346 Signed-off-by: Dale Lane <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The CSP policy currently used is not correct.
taxinomitis/src/lib/restapi/config.ts
Lines 43 to 90 in 3d32953
This was brought to light after a recent version update of the
helmet
module. To avoid breakages, the CSP was switched to report-only as a temporary workaround.taxinomitis/src/lib/restapi/index.ts
Lines 42 to 44 in e665e30
The CSP needs to be fixed so that the enforcement can be re-enabled.
The text was updated successfully, but these errors were encountered: