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

Spec: Effects of k-anonymity not described well #1324

Open
alexmturner opened this issue Nov 4, 2024 · 3 comments
Open

Spec: Effects of k-anonymity not described well #1324

alexmturner opened this issue Nov 4, 2024 · 3 comments
Assignees
Labels
spec Relates to the spec

Comments

@alexmturner
Copy link
Contributor

Moving @brusshamilton's original issue from the Private Aggregation repo here now that the monkey patches have been migrated.

From the spec, it sounds like all Private Aggregation contributions from a non-k-anonymous bid in a Protected Audience auction will processed. According to the Protected Audience explainer (and the Chrome implementation), the only contributions from non-k-anonymous bids that are counted are those that use the "reject-reason" signal.

@morlovich
Copy link
Collaborator

So I implemented most of the latter for the on-event stuff; with TODOs to do it for regular histograms (which probably involves monkey-patching PAgg :( ) and also for the magic reject-reason thing.

@morlovich morlovich self-assigned this Nov 4, 2024
@alexmturner
Copy link
Contributor Author

Ah great -- I'm hoping we won't need any monkey patches, but very happy to export some new algorithms in the PAA spec if needed

@morlovich
Copy link
Collaborator

Well, basically the main difference is that https://patcg-individual-drafts.github.io/private-aggregation-api/#dom-privateaggregation-contributetohistogram at the last step would be appending stuff to a global's contribution list, not a global one. (And we would only use it if it's k-anon, and also if the auction has a winner, only if the winner is shown, but that's a me problem).

Also the way the permissions policy check works needs to be different --- compare https://wicg.github.io/turtledove/#:~:text=If%20this%27s%20allowed%20to%20use%20is%20false%2C%20throw%20a%20TypeError. (partly because, uhm, we can't throw DOM exceptions, and because of the privateAggregation-not-available-at-top-level-run thing).

@JensenPaul JensenPaul added the spec Relates to the spec label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Relates to the spec
Projects
None yet
Development

No branches or pull requests

3 participants