Skip to content

fix: keep pagination query defaults outside Annotated metadata - #732

Merged
nuclearcat merged 1 commit into
kernelci:mainfrom
nuclearcat:fix/pagination-query-defaults
Nov 3, 2026
Merged

nuclearcat merged 1 commit into
kernelci:mainfrom
nuclearcat:fix/pagination-query-defaults

Conversation

@nuclearcat

Copy link
Copy Markdown
Member

FastAPI 0.162 rejects defaults embedded in Query metadata when fastapi-pagination registers its parameter dependency, preventing the API from importing at add_pagination(app).

Define limit and offset with Annotated Query constraints and separate defaults. Override the inherited offset as well, preserving the default page size, offset, and unlimited maximum page size.

Add regression coverage for pagination registration, defaults, offsets, large pages, invalid query parameters, and OpenAPI generation.

FastAPI 0.162 rejects defaults embedded in Query metadata when fastapi-pagination registers its parameter dependency, preventing the API from importing at add_pagination(app).

Define limit and offset with Annotated Query constraints and separate defaults. Override the inherited offset as well, preserving the default page size, offset, and unlimited maximum page size.

Add regression coverage for pagination registration, defaults, offsets, large pages, invalid query parameters, and OpenAPI generation.

Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
@nuclearcat
nuclearcat added this pull request to the merge queue Nov 3, 2026
Merged via the queue into kernelci:main with commit 0a68082 Nov 3, 2026
3 checks passed
@nuclearcat
nuclearcat deleted the fix/pagination-query-defaults branch November 3, 2026 16:50
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.

1 participant