-
Notifications
You must be signed in to change notification settings - Fork 159
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
Naming alignment with PCM #173
Comments
Looking at the issue you linked it was intentional.
and
@csharrison did not mention the reason but I guess that if it was a common attribute developers would have to align with the stricter (PCM) version and use a low-entropy id even on Chrome? |
Yes this was intentional, because it avoids the need to do user agent sniffing to support both API versions. It may be possible to be clever and use one ID for both systems as long as the ID is properly truncated in PCM, but the current spec for PCM says that inserting a too-big value will cause the API to fail: Closing this out but Maud feel free to re-open if you have any other suggestions. |
Great, thanks! |
Follow-up from PCM/#30: one attribute name isn't consistent across the Attribution Reporting spec and the PCM spec.
attributionsourceid
attributionsourceeventid
Has this one fallen through the cracks or is there a reason why they're different? Do they have to be named differently due to the difference in data limitations?
The text was updated successfully, but these errors were encountered: