-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Remove query trackers from Hive email newsletters #30731
Comments
Address debounce in brave/brave-browser#30731
I couldn't find any documentation about what the parameters represent in their documentation. However, it seems likely that The
and so it contains a unique user ID that we should remove. |
Unfortunately, we can't strip them out of all URLs because it breaks these two types of pages:
Therefore we'll have to exempt the |
https://app.hive.co/email/28077/view/public?h_sid=1271475706-5aba9932b99a399aeb76f234&hash=a5a07febf520004 https://app.hive.co/email//elt/?h_sid=48e61db721-599f6907b822446c56ff58e4&hash=7c241dd75c08b18&next=https%3A%2F%2Fwww.complex.com%2Fpop-culture%2F2018%2F06%2Fjohnny-knoxville-gets-smoked-by-spicy-wings-hot-ones See brave/brave-browser#30731.
Verified with
Verified test plan from brave/brave-core#18723 (comment).
|
Verified on
STEPS: ACTUAL RESULTS:
|
https://www.hive.co/ is an email marketing platform that appears to insert unique
h_sid
andh_slt
query parameters.Sample unsubscribe link: https://app.hive.co/email/elt/?h_sid=3d...22d0&hash=20...2&next=https%3A%2F%2Fapp.hive.co%2Femail%2Funsubscribe%2Ftour%2F%3Ftour_id%3D130179%26user_id%3D33...9%26hash%3Db72...d6%26utm_source%3Dhive%26utm_medium%3Demail%26utm_campaign%3Dhive_email_id_196293_pizza-city-fest
I confirmed that neither
h_sid
norh_slt
are necessary for unsubscribing from a list.Also this contains a sample link that can be debounced: https://app.hive.co/email/elt/?h_sid=3d...22d0&hash=20..2&next=https%3A//www.hive.co/%3Ftid%3Dfteremail%26utm_source%3Dhive%26utm_medium%3Demail%26utm_campaign%3Dhive_email_id_196293_pizza-city-fest
The text was updated successfully, but these errors were encountered: