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 headers to the karma run #1893

Merged
merged 11 commits into from
Apr 15, 2024
Merged

Add CSP headers to the karma run #1893

merged 11 commits into from
Apr 15, 2024

Conversation

rajsite
Copy link
Member

@rajsite rajsite commented Mar 3, 2024

Pull Request

Add docs for Content Security Policy configuration and run tests with a defined Content Security Policy.

🤨 Rationale

End goal is to be aware / prevent regressions to CSP safety (i.e. prevent adoption of a library that requires eval usage, etc).

👩‍💻 Implementation

Created security policies for the karma test runners aligned with OWASP Basic non-Strict CSP Policy with additions needed for nimble / specific test runners.

🧪 Testing

Rely on CI.

✅ Checklist

  • I have updated the project documentation to reflect my changes or determined no changes are needed.
    • README docs added to nimble components. Didn't really seem worth linking to it from all the wrappers so just captured it on components. Each of the frameworks has their own docs, i.e. Angular and Blazor and didn't seem like something we need to duplicate and merge in.

@rajsite rajsite marked this pull request as ready for review April 12, 2024 01:40
@rajsite rajsite enabled auto-merge (squash) April 15, 2024 21:31
@rajsite rajsite disabled auto-merge April 15, 2024 22:02
@rajsite rajsite merged commit ef6c82c into main Apr 15, 2024
11 checks passed
@rajsite rajsite deleted the enable-csp branch April 15, 2024 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants