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

fix: replace bare except handlers with specific exception types

Browse Source
This commit is contained in:
kigland
2026-03-06 23:05:02 +08:00
parent 3199168d3e
commit 848f35f5df
7 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
examples/foundational/39-mcp-stdio.py
View File

@@ -70,7 +70,7 @@ class UrlToImageProcessor(FrameProcessor):
return data["artObject"]["webImage"]["url"] return data["artObject"]["webImage"]["url"]
if "artworks" in data and len(data["artworks"]): if "artworks" in data and len(data["artworks"]):
return data["artworks"][0]["webImage"]["url"] return data["artworks"][0]["webImage"]["url"]
except: except (json.JSONDecodeError, KeyError, TypeError):
pass pass
return None return None

2
examples/foundational/39c-multiple-mcp.py
View File

@@ -72,7 +72,7 @@ class UrlToImageProcessor(FrameProcessor):
return data["artObject"]["webImage"]["url"] return data["artObject"]["webImage"]["url"]
if "artworks" in data and len(data["artworks"]): if "artworks" in data and len(data["artworks"]):
return data["artworks"][0]["webImage"]["url"] return data["artworks"][0]["webImage"]["url"]
except: except (json.JSONDecodeError, KeyError, TypeError):
pass pass
async def run_image_process(self, image_url: str): async def run_image_process(self, image_url: str):

2
src/pipecat/adapters/services/bedrock_adapter.py
View File

@@ -209,7 +209,7 @@ class AWSBedrockLLMAdapter(BaseLLMAdapter[AWSBedrockLLMInvocationParams]):
tool_result_content = [{"json": content_json}] tool_result_content = [{"json": content_json}]
else: else:
tool_result_content = [{"text": message["content"]}] tool_result_content = [{"text": message["content"]}]
except: except (json.JSONDecodeError, ValueError):
tool_result_content = [{"text": message["content"]}] tool_result_content = [{"text": message["content"]}]
return { return {

2
src/pipecat/serializers/telnyx.py
View File

@@ -198,7 +198,7 @@ class TelnyxFrameSerializer(FrameSerializer):
f"Telnyx call {call_control_id} was already terminated" f"Telnyx call {call_control_id} was already terminated"
) )
return return
except: except Exception:
pass # Fall through to log the raw error pass # Fall through to log the raw error
# Log other 422 errors # Log other 422 errors

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

@@ -212,7 +212,7 @@ class TwilioFrameSerializer(FrameSerializer):
if error_data.get("code") == 20404: if error_data.get("code") == 20404:
logger.debug(f"Twilio call {call_sid} was already terminated") logger.debug(f"Twilio call {call_sid} was already terminated")
return return
except: except Exception:
pass # Fall through to log the raw error pass # Fall through to log the raw error
# Log other 404 errors # Log other 404 errors

2
src/pipecat/services/aws/llm.py
View File

@@ -369,7 +369,7 @@ class AWSBedrockLLMContext(OpenAILLMContext):
tool_result_content = [{"json": content_json}] tool_result_content = [{"json": content_json}]
else: else:
tool_result_content = [{"text": message["content"]}] tool_result_content = [{"text": message["content"]}]
except: except (json.JSONDecodeError, ValueError):
tool_result_content = [{"text": message["content"]}] tool_result_content = [{"text": message["content"]}]
return { return {

2
src/pipecat/services/mcp_service.py
View File

@@ -298,7 +298,7 @@ class MCPClient(BaseObject):
try: try:
logger.debug(f"Found {len(available_tools)} available tools") logger.debug(f"Found {len(available_tools)} available tools")
except: except Exception:
pass pass
for tool in available_tools.tools: for tool in available_tools.tools: