Fix parameter name consistency in parse_server_event function
- Change function body to use 'str' parameter consistently - Matches pattern used in OpenAI Realtime Beta service - Fixes bug where parameter was named 'str' but body used 'message_str' - Maintains consistency with existing codebase patterns
This commit is contained in:
@@ -487,13 +487,13 @@ def parse_server_event(str):
|
|||||||
ServerEvent instance if parsing succeeds, None otherwise.
|
ServerEvent instance if parsing succeeds, None otherwise.
|
||||||
"""
|
"""
|
||||||
try:
|
try:
|
||||||
evt_dict = json.loads(message_str)
|
evt_dict = json.loads(str)
|
||||||
evt = ServerEvent.model_validate(evt_dict)
|
evt = ServerEvent.model_validate(evt_dict)
|
||||||
return evt
|
return evt
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error(f"Error parsing server event: {e}")
|
logger.error(f"Error parsing server event: {e}")
|
||||||
# Truncate raw message to avoid logging potentially sensitive or overly long data
|
# Truncate raw message to avoid logging potentially sensitive or overly long data
|
||||||
truncated_message = message_str[:200] + "..." if len(message_str) > 200 else message_str
|
truncated_message = str[:200] + "..." if len(str) > 200 else str
|
||||||
logger.error(f"Raw message (truncated): {truncated_message}")
|
logger.error(f"Raw message (truncated): {truncated_message}")
|
||||||
return None
|
return None
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user