-
-
Notifications
You must be signed in to change notification settings - Fork 60
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 favicon for book is not working for Apple (iOS/macOS) #1180
Comments
@BPerlakiH is this related to that old ticket? kiwix/kiwix-apple#789 (comment) |
@rgaudin Yes it seems so, it is the very same function that we are referring to. |
@veloman-yunkan Is this function working at all? tested properly? |
@kelson42 I don't think that it is tested properly. I see a potential issue with it when the illustration's URL points to a redirected address (I don't consider the obvious silly case when the favicon's URL is plain wrong).
Does this debugging statement work? Does it print the problematic URL? Is the resource at that URL accessible? What is the HTTP response to a GET request to that URL? |
@BPerlakiH Please share the detail for a failing example so we get a chance to fix the bug. This is a priority. |
This is the message we get back on the console:
and the list goes on.. If I parse out the urls:
each of those can be opened in browser (or otherwise), and the HTTP response is |
The problem was that our build of |
On Kiwix Apple, we are using the libkiwix function to get the image data for the books, like so:
https://github.com/kiwix/kiwix-apple/blob/820869b2d9ddc40f88b6ed1f9f4a0bd3f0de128a/Model/Entities/ZimFileMetaData/ZimFileMetaData.mm#L159-L170
Unfortunately for all books, this function comes back with an error, and no data is returned.
We fall into the catch block here:
libkiwix/src/book.cpp
Line 244 in 610b8cb
I am not sure if it's Apple only specific, but it's consistently failing for every book, that we just fetched from the catalog.
The text was updated successfully, but these errors were encountered: