Skip to content

feat: Handle missing Prelude response type#17

Merged
86667 merged 2 commits intomasterfrom
handle_unknown_prelude_blocked_reasons
Feb 2, 2026
Merged

feat: Handle missing Prelude response type#17
86667 merged 2 commits intomasterfrom
handle_unknown_prelude_blocked_reasons

Conversation

@86667
Copy link
Contributor

@86667 86667 commented Jan 27, 2026

  • Handle Prelude's Blocked: inavlid_phone_number variant
  • Also default when blocked reason is unknown

Before:

ERROR Failed to communicate with SMS provider error=error decoding response body
→ HTTP 500 Internal Server Error

After:

INFO Phone number blocked for reason: InvalidPhoneNumber. prelude id: ...
→ HTTP 403 Forbidden — "Phone number blocked for verification"

@86667 86667 changed the title feat: Handle missing Peelude response type feat: Handle missing Prelude response type Jan 27, 2026
@86667 86667 merged commit 51a34d3 into master Feb 2, 2026
2 checks passed
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.

2 participants