-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Dependency 'uri-template-lite' does not pass strict Content Security Policy #1007
Comments
Oh.. A lot of issues emerged after merging PR with
Awesome! 👍 How are you going to fix it? |
I swap the |
The only thing that I am concerned about is that |
Use a different url template dependency that does not have the XSS vulnerability as stated in Redocly#1007. Also corrected a test that was not conform the spec.
I would love to see this in a new version! We are currently blocked by this pretty heavily. Any idea when a new release candidate can be released? |
Going to release a new version on weekends |
Perfect, thanks! |
@RomanHotsiy Any update on releasing the fix? Unfortunately, we are still blocked by this issue. |
@RomanHotsiy Sorry to bother you again, Roman, but can I help somehow to streamline this process of the release? I don't want to be a nuisance, but this PR improves the security a lot of the project :) |
Sorry for super long delay. I was traveling. I am about to release a new version. |
Our application uses a strict Content Security Policy (script-src 'self'). This policy does not allow any runtime code generation. However, Redoc is not able to render anymore as the dependency, uri-template-lite, defines functions by concatenating string. This practice is vulnerable for XSS attacks as we can hardly distinguish XSS attacks from normal behaviour.
The error you get when the CSP policy is strict:
Currently, I'm preparing a PR to fix this issue.
The text was updated successfully, but these errors were encountered: