wx44wx/pipecat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #2627 from jessieweiyi/main

Browse Source 
Support TwilioFrameSerializer region/edge settings. Close #2625.
This commit is contained in:
Mark Backman
2025-09-24 09:13:10 -04:00
committed by GitHub
parent 7f4d3a2f02 88614b312f
commit c4762c1a92
1 changed files with 31 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

47
src/pipecat/serializers/twilio.py
View File

@@ -61,6 +61,8 @@ class TwilioFrameSerializer(FrameSerializer):
call_sid: Optional[str] = None, call_sid: Optional[str] = None,
account_sid: Optional[str] = None, account_sid: Optional[str] = None,
auth_token: Optional[str] = None, auth_token: Optional[str] = None,
region: Optional[str] = None,
edge: Optional[str] = None,
params: Optional[InputParams] = None, params: Optional[InputParams] = None,
): ):
"""Initialize the TwilioFrameSerializer. """Initialize the TwilioFrameSerializer.
@@ -70,12 +72,36 @@ class TwilioFrameSerializer(FrameSerializer):
call_sid: The associated Twilio Call SID (optional, but required for auto hang-up). call_sid: The associated Twilio Call SID (optional, but required for auto hang-up).
account_sid: Twilio account SID (required for auto hang-up). account_sid: Twilio account SID (required for auto hang-up).
auth_token: Twilio auth token (required for auto hang-up). auth_token: Twilio auth token (required for auto hang-up).
region: Twilio region (e.g., "au1", "ie1"). Must be specified with edge.
edge: Twilio edge location (e.g., "sydney", "dublin"). Must be specified with region.
params: Configuration parameters. params: Configuration parameters.
""" """
if not call_sid or not account_sid or not auth_token:
missing = []
if not call_sid:
missing.append("call_sid")
if not account_sid:
missing.append("account_sid")
if not auth_token:
missing.append("auth_token")
raise ValueError(
f"Cannot hang up Twilio call: missing required parameters: {', '.join(missing)}"
)
if (region and not edge) or (edge and not region):
raise ValueError(
"Both edge and region parameters are required if one is set. "
f"Twilio's FQDN format requires both: api.{{edge}}.{{region}}.twilio.com. "
f"Got: region='{region}', edge='{edge}'"
)
self._stream_sid = stream_sid self._stream_sid = stream_sid
self._call_sid = call_sid self._call_sid = call_sid
self._account_sid = account_sid self._account_sid = account_sid
self._auth_token = auth_token self._auth_token = auth_token
self._region = region
self._edge = edge
self._params = params or TwilioFrameSerializer.InputParams() self._params = params or TwilioFrameSerializer.InputParams()
self._twilio_sample_rate = self._params.twilio_sample_rate self._twilio_sample_rate = self._params.twilio_sample_rate
@@ -158,25 +184,14 @@ class TwilioFrameSerializer(FrameSerializer):
account_sid = self._account_sid account_sid = self._account_sid
auth_token = self._auth_token auth_token = self._auth_token
call_sid = self._call_sid call_sid = self._call_sid
region = self._region
edge = self._edge
if not call_sid or not account_sid or not auth_token: region_prefix = f"{region}." if region else ""
missing = [] edge_prefix = f"{edge}." if edge else ""
if not call_sid:
missing.append("call_sid")
if not account_sid:
missing.append("account_sid")
if not auth_token:
missing.append("auth_token")
logger.warning(
f"Cannot hang up Twilio call: missing required parameters: {', '.join(missing)}"
)
return
# Twilio API endpoint for updating calls # Twilio API endpoint for updating calls
endpoint = ( endpoint = f"https://api.{edge_prefix}{region_prefix}twilio.com/2010-04-01/Accounts/{account_sid}/Calls/{call_sid}.json"
f"https://api.twilio.com/2010-04-01/Accounts/{account_sid}/Calls/{call_sid}.json"
)
# Create basic auth from account_sid and auth_token # Create basic auth from account_sid and auth_token
auth = aiohttp.BasicAuth(account_sid, auth_token) auth = aiohttp.BasicAuth(account_sid, auth_token)