Skip to content

fix(sms): handle missing Twilio credentials#108

Open
rissrice2105-agent wants to merge 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/twilio-validator-null-input
Open

fix(sms): handle missing Twilio credentials#108
rissrice2105-agent wants to merge 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/twilio-validator-null-input

Conversation

@rissrice2105-agent

Copy link
Copy Markdown

Summary

  • makes validateTwilioCredentials() return a structured validation error for missing/non-object input instead of throwing
  • adds a regression test for null credential config
  • preserves existing validation behavior for valid Twilio credentials, invalid SID/token, phone format, and messaging service SID

Fixes #107.

Validation

  • node --test tests/twilio-validator.test.js - 16 passing
  • direct Node check for validateTwilioCredentials(null) returning { valid: false, errors: [...] }

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.

Twilio credential validator throws on null input

1 participant