Skip to content
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

URGENT: Kiwix-desktop 2.1.0 can't download anything online (with latest kiwix-serve at library.kiwix.org) #684

Closed
kelson42 opened this issue Jan 21, 2022 · 3 comments · Fixed by kiwix/kiwix-desktop#769 or #686

Comments

@kelson42
Copy link
Collaborator

I have deployed kiwix-serve 3.1.2 on library.kiwix.org.

As a consequence kiwix-desktop can not list anything on the online library. It makes the OPDS requests, but there is always 0 books listed.

@kelson42
Copy link
Collaborator Author

I believe, I understand the problem. kiwix-desktop 2.0.5 and 2.1.0 make following OPDS requests:

http://library.kiwix.org:80/catalog/search?lang=eng&count=0

"oldest" kiwix-serve as interpreted count=0 as infinite and "newest" kiwix-serve interpretes it as zero book.

The strict interpretation seems strictly correct to me, but it breaks older kiwix-desktops and provides a bad user experience.

What seems less good, is that if count is undefined it returns 10 books max and if max= (so empty string) it returns indefinite.

I'm not 100% what is the smartest approach to fix the problem.

@juuz0
Copy link
Collaborator

juuz0 commented Jan 21, 2022

count=-1 gives me the results like previously. Not sure why but it just works
Should move this to kiwix-desktop I think

@kelson42
Copy link
Collaborator Author

kelson42 commented Jan 21, 2022

For the moment the best I can figure out is:

Fix/release kiwix-serve/libkiwix to:

  • Returns all books (indefinite numbers of book) if count=0, this would be a temporary patch/workaround to not break the forward compability of older kiwix-desktop with newer kiwix-serve. With the right explanative comment, this should be removed in 1 year or two to come to the proper behaviour (returning 0 books)
  • Clarify it this is normal that count being undefined does not behave like being empty string.

Fix/release kiwix-desktop to:

  • Use count=-1 in place of count=0 @juuz0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants