Skip to content

fix: align update operation delete api#2464

Open
immanuwell wants to merge 1 commit into
quay:mainfrom
immanuwell:httptransport/fix-update-operation-delete
Open

fix: align update operation delete api#2464
immanuwell wants to merge 1 commit into
quay:mainfrom
immanuwell:httptransport/fix-update-operation-delete

Conversation

@immanuwell
Copy link
Copy Markdown
Contributor

this fixes a small matcher internal API mismatch.

docs/spec said DELETE /matcher/api/v1/internal/update_operation/{digest} and showed 200, but the handler actually does uuid.Parse on that path segment. so a client following the docs gets 400, which is kinda rough.

repro:

  1. call DELETE /matcher/api/v1/internal/update_operation/sha256:0000000000000000000000000000000000000000000000000000000000000000
  2. before this patch it returns 400 because the handler expects a UUID update operation id, not a digest
  3. after this patch the contract is {id} and successful deletes return 204

tests: go test ./...

Signed-off-by: immanuwell <pchpr.00@list.ru>
@immanuwell immanuwell requested a review from a team as a code owner May 31, 2026 15:37
@immanuwell immanuwell requested review from crozzy and removed request for a team May 31, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant