-
Notifications
You must be signed in to change notification settings - Fork 260
Clarify ability to update interestGroup's userBiddingSignals through updateURL #760
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
Comments
As we discussed on Wednesday's WICG call, |
Our current design uses one interest group per advertiser domain. When a Chrome user visits the advertiser domain, they are added to a single IG, and We are looking to update Our understanding was that During Wednesday's WICG call, you suggested that we use the KV server to store and update user-level state instead. Am I remembering that correctly? I think we could technically do that, though it would mean dealing with a lot more keys in the KV server and looking up by |
I think we just need to break free of a little bit of history here :-). The decision to not update the userBiddingSignals via updateURL dates back to a time when updates still required k-anonymity of the Interest Group. In that setting, of course updating userBiddingSignals didn't make sense, because you didn't know who the user was at update time. Now that IG update does not have a k-anon requirement, it seems to me that allowing userBiddingSignals updates is entirely consistent with the rest of the design. |
So should @bmilekic et al open an InterestGroups issue requesting this amendment? |
@dmdabbs The work to implement this is tracked in https://crbug.com/1501828. |
Hello, we're trying to figure out the best place to passback user/device specific data known server side at join and update time that we can use at
generateBid()
time for decisionning. Looking at the spec/API and various discussions it seemed thatuserBiddingSignals
would be the best place however it's unclear if it's meant to be updated throughupdateURL
. Going throughtrustedBiddingSignals
could be an alternative but is less desirable than storing directly on device.After testing it's clear that it doesn't get updated. I can confirm by looking at chromium code that this field is not covered by the update.
https://source.chromium.org/chromium/chromium/src/+/main:content/browser/interest_group/interest_group_storage.cc;l=1507
https://source.chromium.org/chromium/chromium/src/+/main:content/browser/interest_group/interest_group_storage.cc;drc=a056443eb4e1a5922ce82b2d11a7b4e64e01f5a6;l=1560
If this is intentional, should we update https://github.com/WICG/turtledove/blob/main/FLEDGE.md to reflect that?
Currently the document says:
Thanks!
The text was updated successfully, but these errors were encountered: