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

Update integration with HTML #56

Closed
fred-wang opened this issue May 14, 2021 · 4 comments
Closed

Update integration with HTML #56

fred-wang opened this issue May 14, 2021 · 4 comments

Comments

@fred-wang
Copy link
Contributor

From https://wicg.github.io/private-network-access/#integration-html:

The initialize the Document object algorithm is amended with an extra step in between the existing steps 10 and 11, before the new Document's CSP list is initialized:
12 Set document’s IP address space to response’s IP address space.

new Document's CSP list is step 12 of https://html.spec.whatwg.org/multipage/browsing-the-web.html#initialise-the-document-object ; so I think this should be " existing steps 11 and 12".

The run a worker algorithm is amended with two extra steps in between the existing steps 14.4 and 14.5, after the new WorkerGlobalScope's referrer policy is set:

In current version of https://html.spec.whatwg.org/multipage/workers.html#run-a-worker, this happens in 13.6, so I guess it's "between 13.6 and 13.7"?

@letitz
Copy link
Collaborator

letitz commented May 17, 2021

Thanks for the corrections!

@annevk
Copy link

annevk commented May 17, 2021

I suspect this should be integrated with policy container?

@letitz
Copy link
Collaborator

letitz commented May 21, 2021

Yes! The address space property of documents is implemented using PolicyContainer in Chrome, and that should be reflected in the spec.

@letitz
Copy link
Collaborator

letitz commented May 25, 2021

Wow, am I glad that the policy container exists. That was so easy! Please feel free to re-open if you notice other errors.

This issue was closed.
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

No branches or pull requests

3 participants