libredirect / browser_extension

A browser extension that redirects popular sites to alternative privacy friendly frontends
https://libredirect.github.io
GNU General Public License v3.0
3.28k stars 121 forks source link

Redirects stopped working after update #293

Closed wrhou closed 2 years ago

wrhou commented 2 years ago

Went to Youtube just now and it didn't redirect me to any of the Piped instances I specified. Same with twitter. Checked Libredirect settings just to see if they were reset, they're still the same as before the update as far as I can tell.

ManeraKai commented 2 years ago

Reset your settings. I'll keep this issue opened for other people too.

Sargastic commented 2 years ago

Same problem here (guess a lot of other people will have it). Resetting settings did the trick.

IkelAtomig commented 2 years ago

Resetting settings did the trick.

It is because v1.7 & 2.0 are incompatible. So, It wasn't working. After you reset, It's just v2.0 settings. So, It's working as expected.

sxe commented 2 years ago

Can we please get a notification after an update if this is ever happening again? It's not the first time either and mildly frustrating to not know what's going on. Besides that, setting up everything again, isn't the most fun either.

ManeraKai commented 2 years ago

Can we please get a notification after an update if this is ever happening again?

We're planning to add a popup warning of an automatic settings reset after every Major release.

Besides that, setting up everything again, isn't the most fun either.

Removing the front ends settings from the settings page reduces the amount of settings till the point of just redirection settings and this is maintainable.

sxe commented 2 years ago

Removing the front ends settings from the settings page reduces the amount of settings till the point of just redirection settings and this is maintainable.

Not sure what you are telling me here to be honest. I noticed there are no more instance settings at all. Is that supposed to happen? Is there no way anymore to, for example, set a specific setting for piped that is automatically applied to all instances?

ManeraKai commented 2 years ago

No it's still there but got better: https://libredirect.github.io/faq.html#what_is_unify_settings

sxe commented 2 years ago

Ah cool, is that literally setting a cookie for each instance? I am asking cause my browser deletes cookies after shut down and that would delete those settings as well. So I would have to exclude each instances separately from deleting the cookies. In 1.7 it was super comfortable to not have to worry about cookies at all.

ManeraKai commented 2 years ago

There's a mechanism for that. It doesn't just copy the settings, it also stores them in LibRedirect's settings. So even if you cleared the cookies and relaunched your browser, they will be rewritten again. We also did this mechanism for Tor and incognito mode but at the moment they're having some problems. though for regular browsers, it will work correctly.

All those changes and this one specifically made it incompatible. I also rewrote it almost completely and cleaned the code, you can check the change log here: https://github.com/libredirect/libredirect/releases/tag/v2.0.0

You really can't make it backwards compatible after all those changes.

sxe commented 2 years ago

Hey yeah thx all good. I read the changelog already but it wasn't really clear to me what happened form reading it.

Maybe a short section like What changed would be good when big changes like this happen in the future. To explain a bit more straight forward the implications for the normal user.

Anyways, thx for all the work you are doing. Love the addon.

ManeraKai commented 2 years ago

Maybe a short section like What changed would be good when big changes like this happen in the future. To explain a bit more straight forward the implications for the normal user.

@EdwardLangdon and me will start writing more informative change logs with links to every issue. It will turn out good.

Anyways, thx for all the work you are doing. Love the addon.

❤️

PTsuarez commented 2 years ago

I've noticed another issue with the most recent update.

I only want to have youtube redirect for embedded videos only which is the option that I enabled and worked perfectly before but now it doesn't it always redirects completely instead of just for embedded videos. Have I missed something or is it just not working?

IkelAtomig commented 2 years ago

@owintwist I understand your anger and concern. We will try to do better next time. This is the first time we are working on a project like this. We aren't a Company that can take care of all things, Just people trying to help other people in our free time and out of passion.

And that operation is basically to wipe all settings out (!).

I think you could exported and then reset and tried resetting.

and a kept-open ticket on github

We opened the ticket to keep a log of that issue as soon as many people asked about the new changes.

that's not the first time you do so

I am not sure about this.

ManeraKai commented 2 years ago

@PTsuarez fixed your problem 👍️

mjholub commented 2 years ago

Embedded videos are defaulting to YT (some content is loaded from Invidious, but generally I need to allow YT's garbage through uMatrix). I'd like to upload screenshots, but GH throws some unknown error atm. Can be checked with uMatrix.

ManeraKai commented 2 years ago

@154pinkchairs I fixed this bug.

mpeter50 commented 2 years ago

I've read that you want to just reset the settings of users without question starting from the next version, if they are coming from 1.x.x. I wouldn't recommend doing that, because the user will lose what instances they have enabled/disabled. Personally I have put some time into selecting which ones I want to use and don't want to repeat it if not necessary.

Instead, there could be 2 buttons on the upcoming warning page:

Also, I would recommend releasing the version with the warning page as soon as possible, because most people won't come to github, but will just remove the extension because it's broken.

Overall the extension looked pretty good so I switched from Privacy Redirect, but changes like these make me reconsider that decision. PR just worked, and this one throws my preferences in the trash bin.

IkelAtomig commented 2 years ago

Well, We are busy. @mpeter50 , We will try to do so within this month.

mpeter50 commented 2 years ago

By the way, if someone hasn't yet deleted the settings, and they are on version 2.0.0, it's still possible to recover your enabled instances, and take a look at your old settings, by following these instructions:

  1. Type about:debugging into the URL bar
  2. Click on "This Firefox" on the left side image
  3. Scroll down until you find the card for LibRedirect, and click "Inspect" image

After that, the old settings are visible in the table on the right. Column width can be adjusted by dragging a column separator.

  1. In the new browser tab, select the "Storage" tab
  2. On the left side open the dropdown for "Extension Storage" and click on the item that is revealed. image
mpeter50 commented 2 years ago

Well, We are busy. @mpeter50 , We will try to do so within this month.

I understand that, it happens with everyone, but please be more careful. Unless you have to hurry with an update, you can test it and polish it until needed.

Also, sorry if I seemed to be harsh, I didn't intend so.

ManeraKai commented 2 years ago

@mpeter50 There's also a problem with updating the instances list. Like imagine if your selected instance got removed from the official list, it will just disappear from your selection. Same when adding more instances:

which philosophy? They both are correct statements.

Till now if you press update instances, your selection will be reset due to those factors. Idk how to implement it in a more practical way.

IkelAtomig commented 2 years ago

@owintwist May I know why was the ping for and what do you like to tell here by the facepalm emoji ?

I understand that, it happens with everyone, but please be more careful. Unless you have to hurry with an update, you can test it and polish it until needed.

Well, We tested the addon a lot, You can even visit the Libredirect Matrix room. You can see my matrix ID and Manerakai's discussing and testing the Newer one.

The One wrong thing is, We didn't check for Compatibility with older version as it was new thing, I didn't check actually.

Also, sorry if I seemed to be harsh, I didn't intend so.

It's okay, Don't be.

mpeter50 commented 2 years ago

@ManeraKai

Till now if you press update instances, your selection will be reset due to those factors. Idk how to implement it in a more practical way.

Hmm, I didn't know this. I think it would be useful to have a warning box near the instance selectors that selection is reset on updating the instance list, while it works that way.

On how to handle this better I have a few ideas, but I don't know how much work would it be to make it work this way.

For whether to automatically allow newly added instances: I think there could be a toggle button for this, one for every service type (so separately for youtube, for reddit, for medium, etc). I see that you store the list of enabled instances under a separate key (e.g. invidiousNormalRedirectsChecks). When updating the instance list, the contents of this key would be kept, and new instances would be added to it if the toggle is set to add new instances automatically. The addon would know if an instance is new if it is included in the newly downloaded list but isn't in the old list (so this would probably need to be decided in the update procedure, before overwriting the instance list with the new one).

For the situation when instances are removed: The content of the enabled instance list is still kept. If a removed instance was enabled, it is not removed from the enabled instance list, but automatic redirection would only happen to instances which are

On the long term this could lead to very long enabled instance lists because of the enabled-but-now-unknown instances, but the UI could show them dimmed out in the main instance list (like this), with 2 active buttons, so it is manageable: delete, move to custom instance list. This way the user could see if one of their enabled instances was removed from your list, and decide whether they still want to use it or not.

What do you think?

Dyras commented 2 years ago

I think mpeter50's suggestion to have a toggle next to every instance type would be a good idea. It caught me a bit off guard that the instances aren't updated automagically, ever.

I think automagically updating the instance list every x day, with an opt-out feature for every instance type would be the best solution. I think there's a minority of people that opt-out of certain instances, so for the majority of people automatic updates would be a good thing, IMO. For the power users, there's a way to opt-out if they want their own list for whatever reason.

ManeraKai commented 2 years ago

I think I'll start implementing this suggestion in 2.2, I'll release 2.1 after some days.

wadbr commented 2 years ago

I am having this issue with twitter only, been happening for 2 updates I think (I reset my settings both times). Tried selecting a single instance, using tor, selecting all instances etc but nothing is working so far.

Opening twitter.com redirects fine, anything else (a profile, a tweet link etc) brings up normal twitter

Dyras commented 2 years ago

@wadbr Did you try resetting the settings? Edit: Clearly I'm visually impaired, missing that you've done it twice already :s

ManeraKai commented 2 years ago

@wadbr I'll see whats wrong.

@Dyras he already did two times.

ManeraKai commented 2 years ago

@mpeter50 Can you open a separate issue and we'll complete from there? I'll close this one.

mpeter50 commented 2 years ago

Sure! I've made the above new issue