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

Downloading images stored in cache not working in Firefox 123 #91

Closed
scotty49 opened this issue Feb 22, 2024 · 4 comments
Closed

Downloading images stored in cache not working in Firefox 123 #91

scotty49 opened this issue Feb 22, 2024 · 4 comments

Comments

@scotty49
Copy link

scotty49 commented Feb 22, 2024

I have been using this addon for several months without any issues, however the recent upgrade of Firefox to version 123.0 seems to prevent Tab Image Saver from reading images from the cache, even with privacy.partition.network_state set to false. This means every image has to be downloaded twice - once to view in the browser and once to save with Tab Image Saver. Could this new issue possibly be related to this security fix listed on Firefox 123.0, See All New Features, Updates and Fixes?

CVE-2024-1554: fetch could be used to effect cache poisoning

Reporter
scarlet
Impact
moderate

Description

The fetch() API and navigation incorrectly shared the same cache, as the cache key did not include the optional headers fetch() may contain. Under the correct circumstances, an attacker may have been able to poison the local browser cache by priming it with a fetch() response controlled by the additional headers. Upon navigation to the same URL, the user would see the cached response instead of the expected response.
References

[Bug 1816390](https://bugzilla.mozilla.org/show_bug.cgi?id=1816390)
@scotty49 scotty49 changed the title privacy.partition.network_state set to false not working in Firefox 123 Downloading images stored in cache not working in Firefox 123 Feb 22, 2024
@mcdamo
Copy link
Owner

mcdamo commented Feb 26, 2024

Thankyou for the info @scotty49.

The privacy.partition.network_state workaround appears to no longer have any effect for addons since Firefox 123.0. This was useful since Firefox added cache partitioning - see #71

This will have a negative impact on all addons that download images.

@scotty49
Copy link
Author

Thanks for the reply. Hopefully Mozilla will eventually implement a workaround for addons (although I won't be holding my breath waiting for it).

@mcdamo
Copy link
Owner

mcdamo commented Mar 15, 2024

Found a method to download from cache, released in addon v4.4.0. Still requires the cache workaround.

@scotty49
Copy link
Author

The cache fix works like a dream. Thank-you so much for following up on this. Where I live (Australia) a lot of city homes have their broadband Internet delivered via 50+ year old copper wire, so not having to download large images from galleries twice can make a very big difference. With my Firefox cache capacity size increased to 2GB from the default of 256MB, Tab Image Saver enables me to easily save 10-20 image tabs in one go without re-downloading, even if the pictures are very big (over 25MB).

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

2 participants