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

incumbent settings object? #74

Open
ak-14 opened this issue Sep 4, 2017 · 5 comments
Open

incumbent settings object? #74

ak-14 opened this issue Sep 4, 2017 · 5 comments

Comments

@ak-14
Copy link
Contributor

ak-14 commented Sep 4, 2017

What is the purpose of the incumbent settings object (step 4 in algorithm 4.9)?

4.9. Process response to request as COWL

@deian
Copy link
Member

deian commented Nov 7, 2017

The note right after should clarify this:

Note, the client associated with the request is null when navigating, so we use the incumbent settings object to get or set the COWL state of the context that initiated the request.

or am I missing something?

@deian
Copy link
Member

deian commented Nov 10, 2017

@ak-14 ^

@ak-14
Copy link
Contributor Author

ak-14 commented Nov 11, 2017

@deian if I remember correctly the thing I was confused about is when navigating, my understanding is that a new context is created and it isn't clear to me why we care about the cowl state from the context that initiated the request. Shouldn't we just create a new cowl state for the new context?

@deian
Copy link
Member

deian commented Nov 11, 2017

Ah, because we need to make sure that even making the request is okay. Suppose I am tainted as a.com, I should not be able to navigate to b.com since doing so would (potentially) leak a.com data to b.com

@ak-14
Copy link
Contributor Author

ak-14 commented Nov 14, 2017

Doesn't navigation always correspond to a fetch request? (Which is handled by 4.8)

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

2 participants