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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: F-23 Paginated PROPFIND/SEARCH crashes when cached pages were evicted

1 participant