Closed Bug 1672664 Opened 4 years ago Closed 4 years ago

Favicon missing for some bookmarks

Categories

(Firefox :: Bookmarks & History, defect)

Firefox 82
defect

Tracking

()

RESOLVED DUPLICATE of bug 1662693
Tracking Status
firefox82 --- affected
firefox83 --- affected
firefox84 --- unaffected

People

(Reporter: tim, Unassigned)

Details

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:82.0) Gecko/20100101 Firefox/82.0

Steps to reproduce:

Bookmarked the following pages:

https://app.clearscore.com/login
https://www.goldenpages.ie/
https://new.three.co.uk/account/login#/

Actual results:

The favicon for these pages is missing from the bookmarks.

Expected results:

The favicon for these pages should be displayed in the bookmarks. This works in Google Chrome and Microsoft Edge.

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:82.0) Gecko/20100101 Firefox/82.0

Hi,

I have managed to reproduce it on Release version 82 and Beta 83.0b4. The first 2 links from the description works accordingly in latest Nightly 84.0a1 (2020-10-26), the favicon is displayed for those but not for the last one: https://new.three.co.uk/account/login#/.
Further, I will move this over to a component so developers can take a look over it. If this is not the correct component please feel free to change it to an appropriate one.
The favicon is displayed correctly for all 3 websites in Chrome.
Maybe this one fixed the issue for the first 2 websites in latest Nightly 84:
https://bugzilla.mozilla.org/show_bug.cgi?id=1662693

Thanks for your input.

Status: UNCONFIRMED → NEW
Component: Untriaged → Bookmarks & History
Ever confirmed: true

The first 2 were likely fixed by bug 1662693
The third one is served with Cache-Control: private, no-cache, no-store, must-revalidate, and it's a page icon, we consider this a server side misconfiguration mistake, and differently from bug 1662693, where we introduced a workaround for root icons, we don't plan to support this case atm.
Doing the opposite would break other sites that send no-store on purpose because effectively the icon should not be stored.

So the third case is a wontfix (website's bug). For the rest I'll just dupe the bug.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.