Skip to content
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

query stripping #1457

Closed
ghost opened this issue May 29, 2022 · 15 comments
Closed

query stripping #1457

ghost opened this issue May 29, 2022 · 15 comments

Comments

@ghost
Copy link

ghost commented May 29, 2022

From #1206 (comment):

ignore

pref("privacy.query_stripping.enabled", false);
pref("privacy.query_stripping.strip_list", "");

Are there any thoughts on using these settings as Librewolf does? Or is it considered redundant with corresponding lists in uBlock?

@rusty-snake
Copy link
Contributor

rusty-snake commented May 29, 2022

  1. uB can this (as you said).
  2. uB list are easier to maintain than a string.
  3. uB has an advanced syntax (e.g. per domain/url/..., regex support, match by param value, ...)
  4. Will be great when development has hit the point when it's used by ETPstrict.

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented May 29, 2022

38c403d

do not ask about ETP custom (I get that this is not in ETP quite yet)

to answer your questions: redundant with uBO. Nice to have for other FF users down the track in ETP in case they don't READ THE WIKI, but it's rather limited. Also AF does not want to add, enable, move, change active/inactive status and piss around with prefs. When it lands it lands, not need to add churn

@Thorin-Oakenpants Thorin-Oakenpants changed the title Thoughts on using built-in query stripping? Thoughts on using built-in query stripping? [leave it alone: it will be part of ETP Strict] May 29, 2022
@Thorin-Oakenpants
Copy link
Contributor

just FYI

MOAR FYI
intent-to-prototype.

  • Tim did say same-site navigation wouldn't be affected

PANTS is already onto it you guys, relax, I got this

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Jun 3, 2022

The pref is pretty lightweight ATM, we could add it juiced up as fallback - @fxbrit whadda think - what was yours? I think it was everything bar mkt_tok, right? But I'm also happy to not do anything with it - one less pref. But then having it documented is also kinda OK.

But it also means we need to make sure we keep it updated with anything new, and I could do without that

@fxbrit
Copy link
Collaborator

fxbrit commented Jun 3, 2022

what was yours?

I've used Brave's list for a while now and I haven't seen any kind of breakage. I must say that when I followed the example URLs provided in the issues Legitimate URL Shortener was doing the job just fine, so there's surely redundancy if you use that list.

@Thorin-Oakenpants
Copy link
Contributor

I know it's redundant for those who READ THE WIKI (and add those lists)

What I'm asking is should we add it, beefed up as a default fallback for non-READERS-of-the-almighty-WIKI, and it also means it's documented, and users can even add their own shit to it. It won't take affect until FF102 (because I'm not going to touch 7016)

So yay or nay? It would simply go under 2700s (ETP), not that it's restricted to ETP

@rusty-snake
Copy link
Contributor

I used the global rules from AdGuard URL Tracking filter: https://codeberg.org/rusty-snake/firefox-config/src/commit/d6e4e47f09335df2f1b6212a3a7c2cdbae156371/assets/user-overrides.js#L131

Note that there are a few sites witch break even with parameters like utm_source removed.

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Jun 3, 2022

does it actually requireETP?

  • e.g. in 102 fuck ... nightly 103 the default is true and stays that way even if you switch to ETP standard

@fxbrit
Copy link
Collaborator

fxbrit commented Jun 3, 2022

I vote yes, it's a good fallback and we can eventually re-allign with firefox in the near future as they add more stuff.

a note for the future in case some users have breakage: query stripping is logged in the console so it's not too hard to debug.

in 102 the default is true and stays that way even if you switch to ETP standard

I think that's just a Nightly thing, it probably won't be that way in stable: https://searchfox.org/mozilla-central/source/browser/app/profile/firefox.js#1832

@Thorin-Oakenpants
Copy link
Contributor

I think that's just a Nightly thing

yeah, you're right. In Beta 102 it changes with standard strict ... and if for example you are in standard, and flip it on, you are changed to custom

@Thorin-Oakenpants
Copy link
Contributor

I used the global rules from AdGuard URL Tracking filter

yup. not going to add that, but that's why adding the list pref is handy, users can tweak it

@Thorin-Oakenpants Thorin-Oakenpants changed the title Thoughts on using built-in query stripping? [leave it alone: it will be part of ETP Strict] Thoughts on using built-in query stripping? Jun 3, 2022
@Thorin-Oakenpants Thorin-Oakenpants changed the title Thoughts on using built-in query stripping? Thoughts on ~~using~~ adding query stripping? Jun 3, 2022
@Thorin-Oakenpants Thorin-Oakenpants changed the title Thoughts on ~~using~~ adding query stripping? query stripping Jun 3, 2022
@Thorin-Oakenpants
Copy link
Contributor

OK, so this pref is dead: https://bugzilla.mozilla.org/show_bug.cgi?id=1764291 - @fxbrit, you can remove it from LW if you want parity with the ALMIGHTY AF

@fxbrit
Copy link
Collaborator

fxbrit commented Jun 20, 2022

thanks pants :-)

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Jun 20, 2022

heh https://bugzilla.mozilla.org/show_bug.cgi?id=1775069 ,, just as well we use uBO, and also do not start in PB Mode

@Thorin-Oakenpants
Copy link
Contributor

https://bugzilla.mozilla.org/show_bug.cgi?id=1697982#c11 ... so apparently you can "add" to the list, but IDFC. IDK if "add" is a typo, IDK if this applies in PB mode - not adding it. Hopefully they just up the default to match Brave

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants