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

Tracking Protection blocks downloads #48

Open
jfsen opened this issue Nov 15, 2019 · 9 comments
Open

Tracking Protection blocks downloads #48

jfsen opened this issue Nov 15, 2019 · 9 comments

Comments

@jfsen
Copy link

jfsen commented Nov 15, 2019

When pictures with the suffix :large are opened in separate tabs, the extension doesn't recognize them as pictures. This URL format is used for images uploaded to Twitter and shows them in their original dimensions. Removing the suffix throws out a compressed image with a max. size of 1200px

@mcdamo
Copy link
Owner

mcdamo commented Nov 24, 2019

I do not see any problem saving twitter images, whether :large, :orig or otherwise. Please give example links where this is failing for you.

@NNBR1
Copy link

NNBR1 commented Feb 9, 2020

Literally any picture link, with :large, or with :orig. I disabled all my addons, and the problem persists.

The URL structure is https://pbs.twimg.com/media/[filename].jpg:orig. Any URL using this will fail consistently. FF72.0.2 on Win10 x64.

@mcdamo
Copy link
Owner

mcdamo commented Feb 10, 2020

I cannot replicate this using current tab-image-saver v2.5.6 with default addon settings.

*.jpg:large or *.jpg:orig images are saved with *.jpg filenames.

@NNBR1
Copy link

NNBR1 commented Feb 11, 2020

I am using 2.5.6 with default settings (delete/reinstall addon). Immediately, I get a "Failed downloads: 1".

I was using this random picture as a test, refreshing all tabs inbetween attempts. https://pbs.twimg.com/media/EQdGtfmWsAM6KwW.jpg:orig
Any other non-twitter picture seems to work. Same behaviour in a clean profile.

Anotación 2020-02-11 030648
Anotación 2020-02-11 030632

What can I do to provide you a log or see any kind of error, or anything I could do to give some sort of report? I'm unable to find what can I do to give any more info on this error.

@mcdamo
Copy link
Owner

mcdamo commented Feb 11, 2020

@NNBR1 the debug console might have some useful information.

  1. Go to Firefox > Addons menu
  2. Next to Manage Your Extensions click the Cog button and select Debug addons
  3. Scroll down to Tab Image Saver and click Inspect
    This will open the debug console.

Now run the addon on your image and the output might be something like:

> GET https://pbs.twimg.com/media/EQdGtfmWsAM6KwW.jpg:orig   [HTTP/1.1 200 Connection established 6208ms]

imagesFailed > Map { "https://pbs.twimg.com/media/EQdGtfmWsAM6KwW.jpg:orig" → "EQdGtfmWsAM6KwW.jpg" }

If the GET did not succeed then there may be more information by expanding it down.

@eduard93
Copy link

eduard93 commented Feb 13, 2020

Got the same issue, turned out it's because of the tracking protection:
image

Disabled tracking protection:
image

And the extension now works.

Is it possible to catch that from extension? A more user-friendly error message (please disable tracking protection) could help.

@mcdamo
Copy link
Owner

mcdamo commented Feb 13, 2020

Yes, setting tracking protection to Strict I can get this error.

The error returned to the extension appears as general network failure so it can't detect when this issue is the cause of download failures.

I will add some info to the extension for users reduce or disable the level of tracking protection.

@mcdamo mcdamo changed the title Failed download for pictures ending with .jpg:large (Twitter) Tracking Protection blocks downloads Feb 14, 2020
@mcdamo mcdamo added the bug label Feb 14, 2020
@mcdamo
Copy link
Owner

mcdamo commented Feb 15, 2020

Unfortunately it appears Firefox is not planning to provide any solution for this, other extensions are having the same problem:

@NNBR1
Copy link

NNBR1 commented Feb 15, 2020

Weeeeell. At least the culprit has been found, and if it's up to Firefox to (not) solve, then I guess case closed. In case someone else finds this error, maybe a warning on the configuration page of the addon, like "try disabling tracking protection if some downloads, like Twitter, fail" would be fine, and that's about as much as can be done about it.

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

No branches or pull requests

4 participants