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

Interaction with SOP #211

Closed
martinthomson opened this issue Oct 17, 2016 · 2 comments
Closed

Interaction with SOP #211

martinthomson opened this issue Oct 17, 2016 · 2 comments
Assignees
Labels

Comments

@martinthomson
Copy link
Member

The data that the browser provides an application comes from a service. We sort of assume that the browser is at some point going to start enforcing origin isolation, but never really point out the place where that happens.

From Stephen Farrell's review of the protocol parts.

@martinthomson
Copy link
Member Author

Critically, we might explain why the data we receive from a push service can be safely treated as same-origin and therefore provided to the origin that the message was sent to.

@beverloo
Copy link
Member

We have the following text in the specification:

A push subscription is a message delivery context established between the user agent and the push service on behalf of a webapp. Each push subscription is associated with a service worker registration and a service worker registration has at most one push subscription.

It defines the association between a push subscription and a service worker registration. The registration has a scope url that is an absolute URL that includes the origin. The availability of this data on the client-side is therefore restricted to that origin.

Whether the data send over the push subscription strictly comes from an application server for that origin is an unknown. If foo.com shares their private key with bar.com so that the latter can send messages on their behalf, there's nothing we can do about it. However, we can reasonably assume that this happened at the discretion of foo.com.

Does this cover what you'd like to see explained in the spec? I'll propose something if so.

@beverloo beverloo self-assigned this Nov 3, 2016
beverloo added a commit to beverloo/push-api that referenced this issue Feb 17, 2017
beverloo added a commit to beverloo/push-api that referenced this issue Feb 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants