-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
FileSummary (filelist) uses JS Eval #11030
Comments
@nextcloud/javascript seems that the simplest thing here is just moving this to plain javascript. Converting to vue will happen when the whole fiellist is moved to vue. For now just moving it to plain js is probably easiest and quickest. |
This comment has been minimized.
This comment has been minimized.
thus, handlebars tempalte compilation in general is evil? |
@blizzz not by definition. Unsafe eval is evil 😉 . You can also compile your templates and upload the compiled js and use the templates then. |
we didn't do ship precompiled templates because $forgotten_issues |
Fixes #11030 For https://github.com/orgs/nextcloud/projects/18 This template is now compiled so this no longer has to happen in the browser. Another step towards a stricter CSP. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Fixes #11030 For https://github.com/orgs/nextcloud/projects/18 This template is now compiled so this no longer has to happen in the browser. Another step towards a stricter CSP. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
For https://github.com/orgs/nextcloud/projects/18
The FileSummary uses handlebars and compiles the templates on the fly. This prevents a stricter CSP.
The text was updated successfully, but these errors were encountered: