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

Don't Redirect "View on ..." links #30

Closed
matteqa opened this issue Mar 29, 2020 · 30 comments
Closed

Don't Redirect "View on ..." links #30

matteqa opened this issue Mar 29, 2020 · 30 comments
Labels
enhancement 💪 New feature or request

Comments

@matteqa
Copy link

matteqa commented Mar 29, 2020

When you click a View on YouTube link in Invidious it redirects back to invidious. It would be more useful if this actually went to YouTube as there is sometimes issues with some videos.

@SimonBrazell
Copy link
Owner

Hi @matteqa, yes I agree, this should work, I'll include this in the next build.

To fix this I can compare the initiator URL to the list of instances on the invidious repo and not redirect from these. But that means it won't work for any private or unlisted instances but I'm not sure how many of those there are anyway.

@SimonBrazell SimonBrazell added the bug 🐞 Something isn't working label Mar 29, 2020
@Jpxe
Copy link
Contributor

Jpxe commented Mar 30, 2020

Wouldn't it be possible to compare the initiator URL to the instance variable set by the user to get private instances working?

@SimonBrazell
Copy link
Owner

@ToughGuyKunio yes that's a much better idea, I'll do that instead.

Thanks for the coffees btw mate, very much appreciated.

@SimonBrazell
Copy link
Owner

Please let me know if there are any problems with redirects in the latest version, this applies to Invidious, Nitter, and Bibliogram (OSM doesn't seem to have a similar feature).

Not released on Chrome yet as the previous release still hasn't been approved...

@Jpxe
Copy link
Contributor

Jpxe commented Mar 31, 2020

No view on youtube/twitter/instagram links work for me. Tested on both custom and standard instances.

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

Same issue here, tried on Firefox 74 with v1.1.18 but didn't work for Invidious or Bibliogram (not sure what links to test on Nitter). Even with the custom variables set in the settings for the servers.

Servers tested:

invidio.us
invidious.snopyta.org
bibliogram.pussthecat.org

@SimonBrazell
Copy link
Owner

Hmm seems the behaviour is different on each browser... sorry I'll look into it.

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

Also tried installing the ZIP in Brave but that gave an error (I assume because it's the Firefox build)

@SimonBrazell
Copy link
Owner

No problem I think I've found the issue, seems Firefox doesn't have the initiator value, funny I thought it would be compatible with Chrome, I'll find the alternative and fix this up with a new build.

@matteqa I haven't tested the plugin on Brave, in theory it should work but will need a CRX file to load, you can unzip the FF archive and then Load unpacked when in Developer mode.

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

I unpacked the ZIP in the releases and added it in Developer mode but got this error:

Unrecognized manifest key 'browser_specific_settings'.

"browser_specific_settings": {
"gecko": {
"id": "{b7f9d2cd-d772-4302-8c3f-eb941af36f76}"
}
}

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

Removing that line from the manifest.json made it install but the View as buttons still redirect

@SimonBrazell
Copy link
Owner

@matteqa that should just be a warning, I get the same error on stock chrome but it still installs:

Screen Shot 2020-03-31 at 4 57 59 pm

See how it's a yellow warning symbol vs the red error. If that isn't the case then perhaps Brave handles extensions a little differently to Chromium.

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

For me it still was labelled as 1.1.16 in the extension until i removed the line. Odd that Brave handles it differently considering it's supposed to be standard Chromium underneath.

@Jpxe
Copy link
Contributor

Jpxe commented Mar 31, 2020

I've been using the build from Chrome Store in Brave for a good while and had no problems with it.

@SimonBrazell
Copy link
Owner

Must just be when trying to load unpacked extensions then, good to know.

I'm having issues with the Nitter 'View in Twitter' links, caused by another fix I added the other day that removes the Twitter service worker if present, it doesn't have access to the initiator like the background script does... Maybe I'll just push out the change for Invidious and Bibliogram for now?

@SimonBrazell SimonBrazell reopened this Mar 31, 2020
@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

The Chrome store version is working for me fine. Just had trouble installing this new version manually because Brave seems to handle warnings as errors.

Also it turns out the reason the View As links weren't working for me on Brave was because I had two versions of the extension installed. My fault.

@SimonBrazell
Copy link
Owner

Oh cool, yeah it was working on Chromium for me, and I'm getting it working on Firefox now, I'll push up a new build shortly.

Thanks for your help!

@SimonBrazell
Copy link
Owner

One other problem I've found with the way it's different on Firefox - if you click the watch on YouTube link, and then refresh that page on Chrome it redirects the page to Invidious like it should, but on Firefox it stays on Youtube... not sure if that will be a problem.

@SimonBrazell
Copy link
Owner

Seems to work as expected if I copy the url to another tab though so at least the normal use cases aren't broken, so I think we are good to go, minus the Nitter links which I still need to work out.

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

Either way could be seen as expected behavior so i wouldnt worry about it

@SimonBrazell
Copy link
Owner

Okay should be fixed now for Firefox in v1.1.19.

I've aborted the change for Nitter for now due to that other conflict I mentioned above.

@SimonBrazell
Copy link
Owner

Let me know how you go.

@matteqa another Aussie 🦘👋

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

👋

@matteqa
Copy link
Author

matteqa commented Mar 31, 2020

Looks all good on my end!

Thanks for developing this extension btw, it's really useful.

@SimonBrazell
Copy link
Owner

Awesome, I'll see if I can fix up the Nitter links in a future release!

No worries, it's a bit fun building and maintaining it, keeps me busy, especially lately when I'm stuck at home day and night!

@Jpxe
Copy link
Contributor

Jpxe commented Mar 31, 2020

Works for me too 👍

@SimonBrazell
Copy link
Owner

Thanks guys.

@DJCrashdummy
Copy link

DJCrashdummy commented May 12, 2020

the "open in twitter"-links on nitter are still redirected (at least in firefox)... is still being worked on this (then wouldn't it be better to reopen & perhaps rename this issue) or won't this be fixed?
or should a new issue be opened about this particular problem?

@SimonBrazell
Copy link
Owner

@DJCrashdummy yes you're right, this slipped my mind to be honest, I'll re-open this to help remind me in the future when I have more time to work on this project again.

@SimonBrazell SimonBrazell reopened this May 13, 2020
@SimonBrazell SimonBrazell added enhancement 💪 New feature or request and removed bug 🐞 Something isn't working labels May 13, 2020
@SimonBrazell SimonBrazell changed the title Don't Redirect "View on YouTube" links Don't Redirect "View on ..." links May 13, 2020
@SimonBrazell
Copy link
Owner

Should be completed now with 'open in twitter' now working as expected in v1.1.32, due to go live on AMO shortly.

mittermichal pushed a commit to mittermichal/privacy-redirect that referenced this issue Jul 21, 2022
mittermichal pushed a commit to mittermichal/privacy-redirect that referenced this issue Jul 21, 2022
mittermichal pushed a commit to mittermichal/privacy-redirect that referenced this issue Jul 21, 2022
mittermichal pushed a commit to mittermichal/privacy-redirect that referenced this issue Jul 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 💪 New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants