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

"first party context" section concerns #206

Closed
pes10k opened this issue Mar 28, 2023 · 3 comments
Closed

"first party context" section concerns #206

pes10k opened this issue Mar 28, 2023 · 3 comments
Labels
privacy-needs-resolution Issue the Privacy Group has raised and looks for a response on.

Comments

@pes10k
Copy link

pes10k commented Mar 28, 2023

This issue is being filed as part of the requested PING review and was broken off from this previous issue

The sub-section on "first party contexts" is doubly confusing.

  • do the authors mean same domain or same site (i.e., same eTLD+1)? The text uses these terms interchangeably, but they mean different things.
  • Is the site restricted to only first-party contexts, or available to first-parties by default, but able to be extended / delegated to third-parties? The first paragraph implies the former, and the second paragraph suggests the latter
@kenchris
Copy link
Contributor

kenchris commented Mar 29, 2023

Thanks for pointing it out. The section is from the time when Google worked on this spec and we missed updating this part. All the algorithms use the same-origin-domain check so we do same domain and not eTLD+1.

I updated the spec text (see associated PR 207), but I have to go closely through the spec algorithms to make sure this is correctly spec'ed as I noticed that a check is missing for workers.

@anssiko
Copy link
Member

anssiko commented Mar 30, 2023

(This issue once addressed should also close #179.)

@anssiko
Copy link
Member

anssiko commented Jun 2, 2023

With #207 all occurrences of the undefined "first-party context" concept are now replaced with and reconstructed using the following normatively defined concepts:

I believe the concern raised in this issue has been addressed, thus I'm closing this issue. Thanks @pes10k for flagging this one and helping remove this undefined concept from the spec that was an unfortunate leftover from the earlier version.

(I've opened #215 to track the same-origin check for workers separately.)

@anssiko anssiko closed this as completed Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
privacy-needs-resolution Issue the Privacy Group has raised and looks for a response on.
Projects
None yet
Development

No branches or pull requests

3 participants