Skip to content

Voice menu: handle Telnyx call forwarding on the number - #82

Merged
ralyodio merged 1 commit into
masterfrom
fix/voice-call-forwarding
Oct 7, 2026
Merged

ralyodio merged 1 commit into
masterfrom
fix/voice-call-forwarding

Conversation

@ralyodio

@ralyodio ralyodio commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Replace failed on a number with Telnyx's own 'always forward' turned on: Telnyx refuses a Call Control connection while it is on. Status now says calls are forwarded (to which number); Replace switches forwarding off first and restores it if the connection change fails, so the number is never left answering nowhere. 2 new tests (237 total).

🤖 Generated with Claude Code

… on Replace, restore on failure

Telnyx refuses a Call Control connection while the number's own
'always forward' is on ("You cannot use automatic call forwarding with a
Call Control ... number"). Status now reports the forwarding as calls going
elsewhere; Replace switches it off first and puts it back if the connection
change fails.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

ThreatCrush Security Scan

10 finding(s)

HIGH/CRITICAL: 2 | MEDIUM: 2 | LOW: 6

Severity Rule Location
HIGH sh-unquoted-expansion-destructive public/install.sh:67
HIGH sh-remote-script-execution public/install.sh:89
MEDIUM sh-remote-script-execution public/install.sh:133
MEDIUM js-open-redirect src/app/numbers/page.tsx:152
LOW secret-generic-credential src/lib/providers/__tests__/twilio.test.ts:24
LOW secret-generic-credential src/lib/providers/__tests__/twilio.test.ts:46
LOW secret-generic-credential src/lib/providers/__tests__/twilio.test.ts:60
LOW secret-generic-credential src/lib/providers/__tests__/twilio.test.ts:78
LOW secret-generic-credential supabase/config.toml:287
LOW secret-generic-credential supabase/config.toml:319

Snippets are redacted; ThreatCrush never prints matched credential material.

@ralyodio
ralyodio merged commit ecf71fd into master Oct 7, 2026
12 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.

1 participant