Cimbali / Cimbali/CleanLinks

Other parameters to add to the list of blocked by CleanLinks out of the box

Open
#162 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.