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 FLEDGE.md #703

Merged
merged 1 commit into from
Jul 31, 2023
Merged

Update FLEDGE.md #703

merged 1 commit into from
Jul 31, 2023

Conversation

thegreatfatzby
Copy link
Contributor

K-anon description update to reflect changes in application.

PR Note - I believe this is correct and equivalent to the update I proposed in 677 for another doc, but I do want to sneak a question in; is it correct that the Interest Group can bid immediately upon being declared in the browser without the name or other elements of the IG passing a k-anon test? We were discussing this internally and I re-read the main doc, and I believe that may have been a requirement previously but don't see it now...but it's a pretty important detail so wanted to ask, and can update docs to clarify.

K-anon description update to reflect changes in application.
@JensenPaul
Copy link
Collaborator

is it correct that the Interest Group can bid immediately upon being declared in the browser without the name or other elements of the IG passing a k-anon test?

Correct, there is no k-anon requirement to submit a bid. To win the auction the bid's ad URL must pass a k-anon test. To access the interest group's name in reportWin() it must also pass another k-anon test. Note that k-anon is not yet enforced today.

@JensenPaul JensenPaul merged commit 37e2e62 into WICG:main Jul 31, 2023
1 check passed
github-actions bot added a commit that referenced this pull request Jul 31, 2023
SHA: 37e2e62
Reason: push, by JensenPaul

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants