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

Permission policy needs to be checked when owner set changes #243

Open
kenchris opened this issue Nov 6, 2023 · 0 comments
Open

Permission policy needs to be checked when owner set changes #243

kenchris opened this issue Nov 6, 2023 · 0 comments
Labels
bug Something is wrong or under specified

Comments

@kenchris
Copy link
Contributor

kenchris commented Nov 6, 2023

I will assume that it is quite common that a site using shared workers, will create the worker, then turn on compute pressure in that worker, and then when additional same origin sites are loaded, they will connect to the shared worker.

As currently spec'ed, even if the second site doesn't have appropriate permission policy set, it will be able to receive telemetry data as we only check the permission policy when we start observing.

There doesn't look to be any nice hooks to hook into when owner set changes so we might need to monkey patch

@kenchris kenchris added the bug Something is wrong or under specified label Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong or under specified
Projects
None yet
Development

No branches or pull requests

1 participant