Amazon-cleaned URLs aren't actually cleaned of query params/strings
See original GitHub issueDescription
Amazon-cleaned URLs aren’t actually cleaned of query params/strings
Spun off from https://github.com/brave/brave-browser/issues/23315.
Steps to Reproduce
- install
1.45.87
- launch Brave
- load
https://www.amazon.com/Echo-Dot/dp/B07FZ8S74R/ref=sr_1_1?crid=1JM4ODN4MKLZO&keywords=alexa+echo&qid=1654644603&sprefix=alexa%2Caps%2C136&sr=8-1
- either context-click in the URL bar or click on the
Share this page
icon in the URL bar - choose
Copy clean link (only main URL)
- paste and look at the link
Actual result:
“Cleaned” URL still is: https://www.amazon.com/Echo-Dot/dp/B07FZ8S74R/ref=sr_1_1?crid=1JM4ODN4MKLZO&keywords=alexa+echo&qid=1654644603&sprefix=alexa%2Caps%2C136&sr=8-1
Expected result:
Cleaned URL should be: https://www.amazon.com/dp/B07FZ8S74R/
Reproduces how often:
100%
Brave version (brave://version info)
Brave | 1.45.87 Chromium: 106.0.5249.91 (Official Build) beta (x86_64) |
---|---|
Revision | fa96d5f07b1177d1bf5009f647a5b8c629762157-refs/branch-heads/5249@{#707} |
OS | macOS Version 11.7 (Build 20G817) |
Version/Channel Information:
- Can you reproduce this issue with the current release?
No
- Can you reproduce this issue with the beta channel?
Yes
- Can you reproduce this issue with the nightly channel?
Yes
Miscellaneous Information:
Issue Analytics
- State:
- Created a year ago
- Comments:5 (2 by maintainers)
Top Results From Across the Web
Caching content based on query string parameters
Describes how CloudFront forwards, caches, and logs query string parameters to your origin.
Read more >What's the best way to return a clean URL by removing ...
I'm assuming that any other query params should be left that aren't listed below since they're not defaults. searchCategoryId=all selectedFilter ...
Read more >Firefox 102: Query Parameter Stripping improves privacy
Some sites and services add tracking parameters to links to track users across sites. Facebook, for example, appends a unique fbclid string to ......
Read more >CleanURLs built into Brave shields - New Feature Feedback
ClearURLs also removes those pesky tracking parameters (utm_* for Google, fbclid for Facebook, Amazon, and so on).
Read more >What Are Query Strings & Parameters - How to Optimize
Yep, there may be query and path parameters within the URL. Each holds a different position and performs separate functions. Path Parameters. Positioned...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Apologies @stephendonner for not updating the original issue to make the current coverage clear. In the initial list, we added the Google
utm_*
parameters and then two Twitter-specific ones. Those should work fine. At the moment, there are no rules for cleaning Amazon URLs and so the expected “clean” URL is the original URL.We should add rules to clean the Amazon params that we can, but it’s not a bug in this feature, rather it’s an example of something that’s missing in our list.
We didn’t include Amazon in the first release of the list, but we can add it anytime. The list we use is in https://github.com/brave/adblock-lists/blob/master/brave-lists/clean-urls.json
@ryanbr Should we just copy the Adguard Amazon rules?