Skip to content

Sync from upstream (15 commits, 6 files changed)#363

Merged
holta merged 15 commits intoiiab:masterfrom
janeczku:master
Feb 25, 2026
Merged

Sync from upstream (15 commits, 6 files changed)#363
holta merged 15 commits intoiiab:masterfrom
janeczku:master

Conversation

@holta
Copy link
Copy Markdown
Member

@holta holta commented Feb 25, 2026

Thanks @deldesir and @chapmanjacobd if you can take a quick look reviewing!

webysther and others added 15 commits January 26, 2026 19:36
  - 【style】 constrain add-to-shelves dropdown height and enable scroll
  - 【style】 prevent overly tall menus from overflowing the page
  - 【style】 constrain add-to-shelves dropdown height and enable scroll
  - 【style】 prevent overly tall menus from overflowing the page
regex should not impose a space after "l'" otherwise detection for "L'arbre" fails.
…in init response

The HandleInitRequest() function rewrites image URLs to point to the local
Calibre-Web instance, but library_sync was left pointing to storeapi.kobo.com.
This caused Kobo e-readers to call Kobo's servers for sync instead of the local
server, resulting in no books being synced when running behind a reverse proxy.
request_username() is used as flask-limiter's key_func for the OPDS
blueprint. The limiter evaluates key_func in a before_request handler,
before the route's auth decorator runs. When no Authorization header is
present, request.authorization is None, causing an AttributeError and
a 500 response instead of the expected 401.

Guard against None so unauthenticated requests fall back to an empty
string key, allowing the auth decorator to handle the 401 correctly.

Fixes #3592

Disclaimer: AI assisted—humans supervised.
@holta holta merged commit a6459c1 into iiab:master Feb 25, 2026
2 of 3 checks passed
@holta
Copy link
Copy Markdown
Member Author

holta commented Feb 25, 2026

I've gambled again, merging experimentally. As the "test install" CI is not indicative when the PR is coming from upstream.

(Hopefully in future we can refine that, so that the CI does a provisional merge of the upstream branch into IIAB Calibre-Web's latest master/main branch, before running test install!?)

cc: @Akatama

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants