Skip to content

pagination: Return error on missing resource key - #3975

Merged
mandre merged 2 commits into
gophercloud:mainfrom
shiftstack:keyed-pages-extra
Aug 20, 2026
Merged

mandre merged 2 commits into
gophercloud:mainfrom
shiftstack:keyed-pages-extra

Conversation

@stephenfin

Copy link
Copy Markdown
Contributor

A follow-up to #3947 that returns an error if the requested key was not found on a page, rather than silently ignoring the key.

This doesn't appear to have ever been used.

Signed-off-by: Stephen Finucane <[email protected]>
@github-actions github-actions Bot added the edit:gophercloud This PR updates common Gophercloud code label Aug 20, 2026
This is better than silently failing.

Signed-off-by: Stephen Finucane <[email protected]>
Assisted-By: Claude Opus 4.6 <[email protected]>
@github-actions github-actions Bot added semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Aug 20, 2026
@coveralls

coveralls commented Aug 20, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 63.585% (+0.004%) from 63.581% — shiftstack:keyed-pages-extra into gophercloud:main

@mandre mandre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for the quick fix.

@mandre
mandre added this pull request to the merge queue Aug 20, 2026
Merged via the queue into gophercloud:main with commit c846da8 Aug 20, 2026
70 checks passed
@mandre
mandre deleted the keyed-pages-extra branch August 20, 2026 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v2 This PR will be backported to v2 edit:gophercloud This PR updates common Gophercloud code semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants