Skip to content
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

Add CSP require-trusted-types-for header #180

Open
2 tasks
acrosman opened this issue Nov 20, 2022 · 0 comments
Open
2 tasks

Add CSP require-trusted-types-for header #180

acrosman opened this issue Nov 20, 2022 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@acrosman
Copy link
Owner

Summary
The two HTML files lack a require-trusted-types-for CSP directive. The overall application would be more secure with it.

Please describe the problem you are trying to solve.
ElectronNegativity uses Google CSP Evaluator which is currently flagging the lack of a
require-trusted-types-for CSP directive.

Proposed Solution

  • Add require-trusted-types-for 'script' to the CSP headers of both files.
  • Update the render scripts to avoid directly setting innerHTML and other things that violate the header.
@acrosman acrosman added enhancement New feature or request help wanted Extra attention is needed labels Nov 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant