GET /icons/<domain>/icon.png panics for www.zborovna.sk #221

Closed
opened 2025-11-07 06:28:57 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @mprasil on GitHub (Feb 3, 2019).

I'm not sure what is causing it yet, this is what I see in the logs:

[2019-02-03 21:24:30][reqwest::async_impl::client][DEBUG] redirecting to GET 'https://www.zborovna.sk/novinky/index.php'
[2019-02-03 21:24:30][reqwest::async_impl::response][DEBUG] Response: '200 OK' for https://www.zborovna.sk/novinky/index.php
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: ParseIntError { kind: InvalidDigit }', src/libcore/result.rs:997:5
Originally created by @mprasil on GitHub (Feb 3, 2019). I'm not sure what is causing it yet, this is what I see in the logs: ``` [2019-02-03 21:24:30][reqwest::async_impl::client][DEBUG] redirecting to GET 'https://www.zborovna.sk/novinky/index.php' [2019-02-03 21:24:30][reqwest::async_impl::response][DEBUG] Response: '200 OK' for https://www.zborovna.sk/novinky/index.php thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: ParseIntError { kind: InvalidDigit }', src/libcore/result.rs:997:5 ```
GiteaMirror added the bug label 2025-11-07 06:28:57 -06:00
Author
Owner

@mprasil commented on GitHub (Feb 3, 2019):

As per discussion with @dani-garcia on our chat, this is most likely due to incorrect character in sizes attribute of <link rel="apple-touch-icon"> - the site is using "×" (multiplication character) instead of "x" to separate the numbers:

<link rel="apple-touch-icon" sizes="76×76" href="/images/touch-icon-ipad.png">
@mprasil commented on GitHub (Feb 3, 2019): As per discussion with @dani-garcia on our chat, this is most likely due to incorrect character in `sizes` attribute of `<link rel="apple-touch-icon">` - the site is using "×" (multiplication character) instead of "x" to separate the numbers: ``` <link rel="apple-touch-icon" sizes="76×76" href="/images/touch-icon-ipad.png"> ```
Author
Owner

@BlackDex commented on GitHub (Feb 4, 2019):

Is fixed in: #381

@BlackDex commented on GitHub (Feb 4, 2019): Is fixed in: #381
Author
Owner

@dani-garcia commented on GitHub (Feb 4, 2019):

Fixed in #381

@dani-garcia commented on GitHub (Feb 4, 2019): Fixed in #381
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#221