Skip to content

fix: eight query providers fabricated a success for unimplemented actions #145

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
skyoo2003 merged 2 commits into from
Sep 5, 2026
Merged

fix: eight query providers fabricated a success for unimplemented actions #145

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ca465be
test: add reproducer for fabricated successes on unserved operations
skyoo2003 Sep 5, 2026
2e1a69a
fix: eight query providers fabricated a success for unimplemented act…
skyoo2003 Sep 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions changes/unreleased/Fixed-20260906-140000.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
kind: Fixed
body: 'Eight query-protocol providers no longer fabricate a success. `rds`, `neptune`, `docdb`, `redshift`, `elasticache`, `autoscaling`, `cloudformation` and `elasticloadbalancingv2` answered any operation they do not implement with HTTP 200 and an empty `<ActionResponse/>`, so a caller was told an operation succeeded when nothing ran. The body also had no `<ActionResult>` wrapper, so botocore raised `KeyError` rather than returning a value — `neptune.describe_db_cluster_endpoints()` crashed the SDK. They now hand the operation to the generic CRUD engine, which serves the 223 CRUD-shaped ones from the real store and refuses the rest with a clean `InvalidAction`'
time: 2026-09-06T14:00:00.000000+09:00
custom:
Issue: "145"
4 changes: 2 additions & 2 deletions docs/coverage.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ Per operation, from the [fidelity manifest](fidelity-manifest.md):
| Tier | Operations |
|---|---|
| `hand-verified` | 4,496 |
| `auto-crud` | 4,970 |
| `unimplemented` | 2,941 |
| `auto-crud` | 5,193 |
| `unimplemented` | 2,718 |
| **total known** | **12,407** |

## Why a registered service can serve nothing
Expand Down
Loading