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

Consider omitting Attribution Reporting request headers when there is none attribution support #1368

Open
linnan-github opened this issue Jul 10, 2024 · 4 comments · May be fixed by #1380
Open
Labels
app-to-web compat issue that may affect web compatibility enhancement New feature or request

Comments

@linnan-github
Copy link
Collaborator

Currently when there is none attribution support (neither web nor os), the Attribution-Reporting-Eligible header may still be sent and Attribution-Reporting-Support header is also sent as empty. We may consider omitting both to reduce bandwidth and a better indicator that the request is not eligible for Attribution Reporting.

@apasel422
Copy link
Collaborator

I'm not sure that omitting the Attribution-Reporting-Eligible header is the right option, as opposed to including it but setting it to the empty string, as the absence of that header in some contexts still corresponds to the request being trigger-eligible. So it might be beneficial to tell the reporting origin that it is eligible for nothing, as opposed to letting it assume it can still register triggers.

But it depends on whether we consider the presence/absence/value of these headers to be a fingerprinting surface.

@apasel422 apasel422 added enhancement New feature or request app-to-web compat issue that may affect web compatibility labels Jul 10, 2024
@dmdabbs
Copy link
Contributor

dmdabbs commented Jul 10, 2024

Sending all GREASED values would signal "nothing" without sending nothing, though wouldn't address fingerprinting concern.

@apasel422
Copy link
Collaborator

Sending all GREASED values would signal "nothing" without sending nothing, though wouldn't address fingerprinting concern.

Yes, the header would still be subject to greasing.

@linnan-github
Copy link
Collaborator Author

So it might be beneficial to tell the reporting origin that it is eligible for nothing, as opposed to letting it assume it can still register triggers.

But it depends on whether we consider the presence/absence/value of these headers to be a fingerprinting surface.

Agreed that it may be beneficial in this aspect to distinguish these two cases. As for the fingerprinting, I think it's already true for requests with Attribution-Reporting-Support header, so should be fine, at least not regression.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app-to-web compat issue that may affect web compatibility enhancement New feature or request
Projects
None yet
3 participants