Skip to content

fix(api): preserve webhook validation status codes - #3832

Open
r266-tech wants to merge 1 commit into
vectorize-io:mainfrom
r266-tech:fix/webhook-validation-status
Open

fix(api): preserve webhook validation status codes#3832
r266-tech wants to merge 1 commit into
vectorize-io:mainfrom
r266-tech:fix/webhook-validation-status

Conversation

@r266-tech

Copy link
Copy Markdown
Contributor

Fixes #3830.

Webhook routes now preserve OperationValidationError.status_code and reason, matching the existing bank route behavior instead of returning a generic 500. Malformed webhook UUID path parameters now return 422 before reaching the engine.

The patch stays local to the five webhook routes and does not change engine validation or persistence behavior.

Tests cover all five validation-error paths plus malformed IDs for delete, update, and delivery listing.

Verification:

  • 8 focused HTTP regressions passed
  • Ruff check and format passed for both touched files
  • git diff --check passed

@strix-security

strix-security Bot commented Aug 27, 2026

Copy link
Copy Markdown

Strix Security Review

No security issues found.

Updated for ca7fc19.


Reviewed by Strix
Re-run review · Configure security review settings

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.

Webhook routes return 500 instead of 403 when the operation validator rejects

1 participant