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

CSP:EE does not support Trusted Types CSP directives #628

Open
tosmolka opened this issue Dec 5, 2023 · 5 comments · May be fixed by w3c/webappsec-cspee#29
Open

CSP:EE does not support Trusted Types CSP directives #628

tosmolka opened this issue Dec 5, 2023 · 5 comments · May be fixed by w3c/webappsec-cspee#29

Comments

@tosmolka
Copy link

tosmolka commented Dec 5, 2023

CSP:EE spec defines Effective Directive Value as a static list of supported CSP directives. CSP:EE was written before Trusted Types and thus the list does not include CSP directives trusted-types and require-trusted-types-for defined in Trusted Types spec.

This means CSP:EE can't be used for validating Trusted Types enforcement within embedded content. We probably need a change in the specification and then follow-up changes in the implementation (ref chromium).

We have a scenario where we would like to use CSP:EE to permit embedding of web app via iframe if and only if it enforces Trusted Types. This is intended as a defense in depth mechanism for first party web apps that are already supposed to enforce Trusted Types. It would be good if CSP:EE could actually validate this assumption.

This was also discussed in Chromium bug 1446253

@shhnjk
Copy link
Member

shhnjk commented Dec 5, 2023

Hi @tosmolka,

I'm happy to update the spec, but I'm not sure when we can get to the implementation (due to other backlogs).
Do you know if anyone in Edge is willing to help implement this change?

@tosmolka
Copy link
Author

@shhnjk , we are in discussions with Edge team about the implementation, not yet sure about the timelines. I'll reach out once I know more. Do you guys have any updates about the spec? Thanks.

@shhnjk
Copy link
Member

shhnjk commented Mar 15, 2024

The spec change is up at w3c/webappsec-cspee#29 now :) Let's see how it goes.

@tosmolka
Copy link
Author

@shhnjk , any update regarding the change in spec? Edge team is still reviewing the changes needed in the implementation, not yet sure about the timelines on our side. Thanks.

@tosmolka
Copy link
Author

Hello @shhnjk , resurrecting this older discussion, I also commented in the webappsec-cspee PR. Any updates about this issue? Thanks.

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 a pull request may close this issue.

2 participants