Other parameters to add to the list of blocked by CleanLinks out of the box
- Dominant language
- JavaScript
- Stars
- 85
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
Having failed to find a separate issue on the proposals for parameters that are not currently removed from URLs by the CleanLinks with default settings, and having no opportunity to create a topic in the disabled `Discussions`, I propose here to supplement the list of sanitization rules with the following parameters, which I did not find mentioned in the extension description [on addons.mozilla.org](https://addons.mozilla.org/en-GB/firefox/addon/clean-links-webext/), or in `README.md`, or [in wiki](https://github.com/Cimbali/CleanLinks/wiki/):
1. `alternateChannel`
2. `at`
3. `do-waremd5`
4. `eBP`
5. `epic_affiliate`
6. `hid`
7. `otpToken` (LinkedIn)
8. `partner`
9. `ref`
10. `refId`
11. `shopId`
12. `slocation`
13. `src`
14. `trackingId`
15. `yclid` (yandex)
16. `ysclid` (they are not removed on [the test page](https://cimbali.github.io/CleanLinks/))
I know nothing about regular expressions, which to me are only slightly more understandable than cuneiform or runic script. But if the extension removed URL suffixes starting with `?at=` (and others from the list above), I'd be happy.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with README.md and the linked test page to confirm which listed parameters currently remain, then locate the extension’s default sanitization rules. The work is done when the proposed parameters are removed with default settings and the test page confirms the expected cleaned URLs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- security
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100