-
Notifications
You must be signed in to change notification settings - Fork 26
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
Google Ads does not respect Critical-CH restart #150
Comments
This sounds like a Chromium issue, rather than a spec issue. |
I'm writing a WPT for this case to verify |
Haven't gotten to a working reproduction but will try again tomorrow here: https://chromium-review.googlesource.com/c/chromium/src/+/4377410 |
I was able to reproduce spec-respecting behavior (critical-ch being respected at the end (and/or in the middle) of a redirect chain as long as the chain is cross-origin) in https://chromium-review.googlesource.com/c/chromium/src/+/4377410 I cannot reproduce the issue highlighted in this task and when I try to load the site above it now works. Not sure if something changed on the backend or there was something else going on. |
Hey @arichiv thanks for looking into this. This was a critical issue for us so we shipped a temporary workaround to our backend in order to preserve user experience. The workaround is to return a 307 redirect to the same link URL along with the Its worth mentioning that we have not seen this issue for other cross-origin redirect chains, only for Google Ads (i.e. example1->example2.com works fine but google.com->example2.com does not). If we need to repro this live, I think we can put a link to a test server in any google ad and click it. Let me know if I can help here. Is is possible to approach this from an Ads-specific perspective? Is it possible to raise this issue to the Google Ads team? Is it possible there is specific handling for google ads clicks in the CH code, or specific handling of CH on Google Ads clicks? |
@zhengweiwithoutthei could you take a look? It might be some sort of CORS issue. |
It looks like this issue has been abandoned, can we pick this up again? The issue still exists |
Hello, I work for an ad tech company and we have found some edge cases where the critical-ch restart does not play nicely with redirects between domains, or "wrapped domains". Unfortunately this is a very important and common use case in the ads world, including Google Ads.
Example Google Ads link: https://www.google.com/aclk?sa=L&ai=DChcSEwi-l7XdtPL9AhUR7u0KHQrmDQkYABAAGgJkZw&sig=AOD64_0hvmE49iyDD2sX3y9g3VMWQztshg&q&adurl&ved=2ahUKEwim3a_dtPL9AhXlRkEAHcuQDKgQ0Qx6BAgHEAE
Steps to repro
critical-ch
header requesting two critical client hints: platform-version and model.Problem
Expected Result:
Actual Result:
More details
I have not been able to reproduce this with other domains that 302 redirect to each other, it seems specific to Google Ads right now. I can also confirm that only one origin (our server) is requesting a critical hint retry, not multiple.
Please note that since this is a critical issue for us we are urgently working to mitigate this on our end, so this may not be reproducible in the wild after a few days. If this becomes an issue let me know and I can set up a test domain where we can reproduce this issue live.
I have also attached a bunch of screenshots from when I reproduced this issue. It is happening consistently (100% of the time) if the browser doesnt already have the critical hints cached (i.e. new incognito window or clear cache).
Thanks!
The text was updated successfully, but these errors were encountered: