Check if final debounce URL's hostname has at least eTLD + 1 #23580
Labels
enhancement
OS/Android
Fixes related to Android browser functionality
OS/Desktop
privacy/debounce
URL debouncer
QA/No
release-notes/exclude
For AMP URLs (and there will probably be others), there are cases where the embedded target URL in the original URL that we want to debounce does not have a scheme, for e.g. https://theguadian.ampproject.org/c/s/www.newsarticleonguardian.com -- www.newsarticleonguardian.com is not a valid URL (and hence not parsed as one at least in Chromium and also by iOS it looks like) because it does not have a scheme. We create the right debounce URL by using a new property called
prepend_scheme : http | https
as described in #23121The trouble is that if the original URL looks something like https://theguadian.ampproject.org/c/s/foo, and
prepend_scheme
is https, then we debounce to https://foo/, which is not actually a valid website, even though https://foo/ is technically a valid URL.The proposed solution is (@fmarier):
We should do this for all debounce actions.
The text was updated successfully, but these errors were encountered: