Icon extraction fails #4007

Closed
opened 2026-03-07 19:23:43 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @quthla on GitHub (Aug 10, 2019).

For saturn.de and bwin.com it will just create the miss file

Originally created by @quthla on GitHub (Aug 10, 2019). For saturn.de and bwin.com it will just create the miss file
GiteaMirror added the bug label 2026-03-07 19:23:43 -06:00
Author
Owner

@mprasil commented on GitHub (Aug 10, 2019):

What version are you using?

@mprasil commented on GitHub (Aug 10, 2019): What version are you using?
Author
Owner

@quthla commented on GitHub (Aug 10, 2019):

Latest docker image. Were you unable to reproduce the issue?

@quthla commented on GitHub (Aug 10, 2019): Latest docker image. Were you unable to reproduce the issue?
Author
Owner

@mprasil commented on GitHub (Aug 11, 2019):

I think I was able to reproduce that with saturn.de, but bwin.com works fine for me. I'm not sure why saturn.de fails though, I can see this in logs:

Download failed for https://www.saturn.de/favicon.ico

But when I try to poke the same URL with curl, it works fine.

@mprasil commented on GitHub (Aug 11, 2019): I think I was able to reproduce that with saturn.de, but bwin.com works fine for me. I'm not sure why saturn.de fails though, I can see this in logs: ``` Download failed for https://www.saturn.de/favicon.ico ``` But when I try to poke the same URL with curl, it works fine.
Author
Owner

@BlackDex commented on GitHub (Aug 13, 2019):

I have identified the bug this afternoon. It has to do with cookies, and rather the lack off but still sending the header. This breaks the saturn.de website. Trying to think of a good way to fix this.

@BlackDex commented on GitHub (Aug 13, 2019): I have identified the bug this afternoon. It has to do with cookies, and rather the lack off but still sending the header. This breaks the saturn.de website. Trying to think of a good way to fix this.
Author
Owner

@mprasil commented on GitHub (Oct 11, 2019):

This was fixed in PR #644

@mprasil commented on GitHub (Oct 11, 2019): This was fixed in PR #644
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#4007