Error loading from the disk cache pages obtained by blocking remote fonts.

[SCmkQrK1NhwcWoPJF3FImaqHnG2PqPLWBt2GtSU]

Prerequisites

• [ 🗸 ] I verified that this is not a filter issue
  • Filter issues MUST be reported at filter issue tracker
• [ 🗸 ] This is not a support issue or a question
  • Support issues and questions are handled at /r/uBlockOrigin
• [ 🗸 ] I performed a cursory search of the issue tracker to avoid opening a duplicate issue
  • Your issue may already be reported.
• I tried to reproduce the issue when...
  • [ 🗸 ] uBlock Origin is the only extension
  • [ 🗸 ] uBlock Origin with default lists/settings
  • [ 🗸 ] using a new, unmodified browser profile
• [ 🗸 ] I am running the latest version of uBlock Origin
• [ 🗸 ] I checked the documentation to understand that the issue I report is not a normal behavior

Description

When uBlock blocks remote fonts, the downloaded pages (except images) in the disk cache are not stored correctly: they are not then loaded offline, but should be loaded.

A specific URL where the issue occurs

https://example.com/

Steps to Reproduce

1. Create a new clean Firefox profile.
2. Install uBlock Origin (https://addons.mozilla.org/firefox/addon/ublock-origin/).
3. In uBlock Origin, enable "no remote fonts".
4. Go online mode and download a page (not an image), for example, https://example.com/ (checked). (I'm using localhost.)
5. Go offline mode and hold the browser in it.
6. The option "no remote fonts" can be turned off or not to change – it does not matter.
7. Reload the page opened on step 4 from the disk cache. 2 options: a and b: 7.a. Re-open the tab with this page. 7.b.1. "Reload the page bypassing the cache" (Ctrl+F5). (The result is not important: in fact, in offline mode, this will reload the page from the disk cache without checking it.) 7.b.2. "Reload this Page with cache" (F5). The page will not load from the cache, but will report a "browser is offline mode" error. But the page should load. Disregarding the "no remote fonts" is changing the case, but I think that the blocking is needed. Cache is also needed.

Expected behavior:

The page will load from the cache.

Actual behavior:

The page will not load from the cache.

Your environment

• uBlock Origin version: 1.16.20.
• Browser Name and version: Firefox 56.
• Operating System and version: Linux Ubuntu 18.

[gorhill]

This is unfortunately a side effect of the fix to https://github.com/uBlockOrigin/uBlock-issues/issues/42 -- in which the workaround suggested by Firefox devs in https://bugzilla.mozilla.org/show_bug.cgi?id=1376932 was used.

[gorhill]

I am experimenting and it seems if I just use no-cache instead of no-cache, no-store, must-revalidate, I still cannot reproduce #42, and that also seems to fix the offline issue reported here.

[gorhill]

I am going to revert the fix here -- the side effects are undesirable for whoever does not care to work in offline mode.

Since either ways cause side effects which may not be acceptable to group of users, I will add an advanced setting to let users configure how uBO should affect caching for the purpose of dealing with https://bugzilla.mozilla.org/show_bug.cgi?id=1376932.

The default value will be the one used before, i.e. the one causing the issue here.

New advanced setting: cacheControlForFirefox1376932.

Default value: no-cache, no-store, must-revalidate.

If using no-cache, no-store, must-revalidate:

• Undesirable side effect: documents themselves for which uBO has to inject CSP directives as a result of filters/ruleset won't be available offline.

If using no-cache:

• Undesirable side effect: One will need to explicitly cache-bypass reload a page each time uBO has to inject CSP directives as a result of filters/ruleset. Note that such cache-bypass reload does not affect only the document itself, but also all secondary resources inside that document.

This new advanced setting will be removed once https://bugzilla.mozilla.org/show_bug.cgi?id=1376932 is fixed.

[gwarser]

Firefox bug has been closed. I will like to test, but don't see a way to configure cacheControlForFirefox1376932 to not be applied.

[uBlock-user]

@gwarser Were you able to test afterall ?

[gwarser]

Testing all the time, and I think it works.

example.com, Reddit, GitHub with no-remote-fonts or example.com##^a/github.com##^img/example.com##^a loads fine from cache in offline mode with filters applied. I want to check https://github.com/uBlockOrigin/uBlock-issues/issues/42 again.

~I think this can be permanently disabled for Firefox 70.~

[gwarser]

Hmmm. Something is racing. When testing on https://old.reddit.com/ with reddit.com##^img thumbnails placeholders are sometimes visible after refresh (<img> is present).

[gwarser]

HTML filtering is not applied on session restore: https://www.amazon.com/ with amazon.com##^img, but this may be different issue.

[gwarser]

No glitches when using CSP

||amazon.com^$csp=img-src 'none'
||reddit.com^$csp=img-src 'none'

[ghost]

Is this fixed or just closed? I see still in advanced settings that it is not empty, but as it appears at the beginning of this issue. This file can't be loaded from cache, and I have to wait about 1 minute just for it to load when editing data on IMDb pages like this one: https://contribute.imdb.com/updates?edit=tt2543472/keywords&ref_=ttkw_kw_edt Direct url of that resource is: https://m.media-amazon.com/images/I/A1m0xZZQVsL.js So if I wanna increase cache efficiency aka reduce network usage and waiting, I just use no-cache? If I don't care about being tracked a little or get few ads a little? What is difference between no-cache and nothing there? I care about speed and reduced data usage, and I already disabled cache partitioning and rcwn, and my net speed is 4 kB/s. Yup, very slow, kind of ultra slow 2G in village, so I feel the differences more than anybody. OK, I just tried no-cache header only, but still it reloads on each page load. Even after restarting Firefox. What am I supposed to do? I would care less if I had faster Internet. I wanna use my cache more. I tried using Redirector extension to copy that file to my local server and load from there, but it said it could not because of csp. So sure we are talking same issue here.

[uBlock-user]

@GoblinLegislator https://github.com/uBlockOrigin/uBlock-issues/issues/229#issuecomment-423525943

[ghost]

Still don't understand. Why can't I use cached js file? Why does it have to reload from network always? Tried both advanced options, result is same. Should I try to somehow exclude whole file or site from uBlock? Is there option in uBlock to disable html filtering or whatever is causing this? What are consequences? This is caused by HTML filtering (injection)?

OK, I tested with all addons disabled (safe mode). And still this js file does not want to be cached. Even if I try loading it by pressing Enter key on url, or by pressing refresh key. However when I open file url directly, it is cached. Just not on contribute.imdb.com. So not uBlock issue. But can't understand why this is happening. Maybe best to block it and hope page will work without issues. (Screenshot is with addons still enabled, but I also tested with all of them disabled.)

[mpeter50]

The referenced Bugzilla issue (https://bugzilla.mozilla.org/show_bug.cgi?id=1376932) was closed with status RESOLVED FIXED, 5 years ago. As I see, the cache control workaround setting still uses no-cache, no-store, must-revalidate. Is that really still needed? Was the Bugzilla issue closed by mistake?

As I understand, websites not being available offline is not the only downside of this setting, but also more network usage which is quite bad for those on a mobile data connection, and besides that battery drain when the user is on a slow connection, limited by the network provider.

[stephenhawk8054]

@gorhill Sorry, is the one in this reddit thread related to this issue? OP also used no-remote-fonts: * true

https://old.reddit.com/r/uBlockOrigin/comments/1dz6du7/i_managed_to_break_firefoxs_asset_caching/

[gorhill]

Not directly related, the issue is injecting a CSP header in non-document resources, this also happen in Chromium as can be seen in logger when forcing a reload with cache bypass. Chromium is seemingly unaffected because it does not have that special code path taken in Firefox which also inject a cache invalidation header.

I am not sure how far in the past is the regression, I believe at some point a test for document resource might have been removed, leading to the current issue of injecting CSP header in non-document resource. I will work on a fix.

---

This is the regression, a test for document resource was removed, leading to the reported issue: Implement network filter option replace= (Nov 2023)