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

[WIP] Enable get() in fenced frames with network access revoked. #220

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

VergeA
Copy link
Contributor

@VergeA VergeA commented Jan 21, 2025

After script running in a fenced frame successfully resolves a call to window.fence.disableUntrustedNetwork(), the fenced frame gains access to Shared Storage via get().

This patch refactors the get() algorithm to be accessible from Window and SharedStorageWorklet scopes, but the Window branch will fail outside of a fenced frame tree with network disabled.

This patch also specifies a new Permissions Policy, fenced-unpartitioned-storage-read, which can be used to disable access to get() in fenced frames. Its default allowlist is *.

After script running in a fenced frame successfully resolves a call to `window.fence.disableUntrustedNetwork()`, the fenced frame gains access to Shared Storage via `get()`.

This patch refactors the `get()` algorithm to be accessible from `Window` and `SharedStorageWorklet` scopes, but the `Window` branch will fail outside of a fenced frame tree with network disabled.
Merge main into read2
@VergeA VergeA marked this pull request as ready for review January 21, 2025 20:08
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 this pull request may close these issues.

1 participant