Skip to content

fix(dav): ignore paginated pages with evicted cache entries - #66113

Open
lazerg wants to merge 1 commit into
nextcloud:masterfrom
lazerg:fix/issue-65080-paginate-evicted-entries
Open

lazerg wants to merge 1 commit into
nextcloud:masterfrom
lazerg:fix/issue-65080-paginate-evicted-entries

Conversation

@lazerg

@lazerg lazerg commented Oct 31, 2026

Copy link
Copy Markdown

When the pagination cache loses individual entries but keeps the count key, PaginatePlugin::onMethod() passed null to XMLWriter::writeRaw() and the request failed with a 500. The page is now read in full first, and if an entry is missing the request is handled like one without a pagination token.

AI disclosure: this change was prepared with Claude (claude-opus-5-5) and reviewed by me.

Fixes #65080

Assisted-by: Claude:claude-opus-5-5
Signed-off-by: lazerg <lazerg2@gmail.com>
@lazerg
lazerg requested a review from a team as a code owner October 31, 2026 08:08
@lazerg
lazerg requested review from CarlSchwan, come-nc, provokateurin and sorbaugh and removed request for a team October 31, 2026 08:08

This branch has not been deployed

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

Labels

None yet

1 participant