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.

Sign up for GitHub

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

Jump to bottom

[docsy] Work around inline-style CSP restriction #73

Closed
chalin opened this issue Sep 4, 2024 · 1 comment
Closed

[docsy] Work around inline-style CSP restriction #73

chalin opened this issue Sep 4, 2024 · 1 comment
Assignees
@chalin
Labels
docsy Docsy branch, and Docsy related items

Comments

@chalin
Copy link
Collaborator

chalin commented Sep 4, 2024

Originally posted by @chalin in #68 (comment)

Is there a reasonable workaround? Why do we need this? It's a bad look for a security project to disable security controls, even if they don't matter in this case.

Ok, I'll propose a workaround in a followup PR. Thanks for the feedback @JustinCappos.
@chalin chalin added the docsy Docsy branch, and Docsy related items label Sep 4, 2024
@chalin chalin mentioned this issue Sep 5, 2024
Merged
@chalin chalin self-assigned this Sep 5, 2024
@chalin chalin mentioned this issue Sep 5, 2024
Merged
@chalin chalin changed the title [docsy] Avoid inline styles since they violate the CSP [docsy] Work around inline-style CSP restriction Sep 5, 2024
@chalin
Copy link
Collaborator Author

chalin commented Sep 5, 2024

Fixed by #75

@chalin chalin closed this as completed Sep 5, 2024
@chalin chalin mentioned this issue Oct 3, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chalin chalin

Labels
docsy Docsy branch, and Docsy related items
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chalin