phoenix/litellm
1
0
Fork 0
forked from phoenix/litellm-mirror
Code Pull requests Activity

Merge pull request #3384 from BerriAI/litellm_fix_details_to

Browse source 
fix - error sending details to log on sentry
This commit is contained in:
Ishaan Jaff 2024-05-01 11:44:54 -07:00 • committed by GitHub
commit b3161bb20b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 13 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

11
litellm/utils.py
View file

@ -1236,7 +1236,10 @@ class Logging:
print_verbose=print_verbose, print_verbose=print_verbose,
) )
elif callback == "sentry" and add_breadcrumb: elif callback == "sentry" and add_breadcrumb:
try:
details_to_log = copy.deepcopy(self.model_call_details) details_to_log = copy.deepcopy(self.model_call_details)
except:
details_to_log = self.model_call_details
if litellm.turn_off_message_logging: if litellm.turn_off_message_logging:
# make a copy of the _model_Call_details and log it # make a copy of the _model_Call_details and log it
details_to_log.pop("messages", None) details_to_log.pop("messages", None)
@ -1327,8 +1330,10 @@ class Logging:
) )
elif callback == "sentry" and add_breadcrumb: elif callback == "sentry" and add_breadcrumb:
print_verbose("reaches sentry breadcrumbing") print_verbose("reaches sentry breadcrumbing")
try:
details_to_log = copy.deepcopy(self.model_call_details) details_to_log = copy.deepcopy(self.model_call_details)
except:
details_to_log = self.model_call_details
if litellm.turn_off_message_logging: if litellm.turn_off_message_logging:
# make a copy of the _model_Call_details and log it # make a copy of the _model_Call_details and log it
details_to_log.pop("messages", None) details_to_log.pop("messages", None)
@ -2635,7 +2640,11 @@ def function_setup(
dynamic_success_callbacks = kwargs.pop("success_callback") dynamic_success_callbacks = kwargs.pop("success_callback")
if add_breadcrumb: if add_breadcrumb:
try:
details_to_log = copy.deepcopy(kwargs) details_to_log = copy.deepcopy(kwargs)
except:
details_to_log = kwargs
if litellm.turn_off_message_logging: if litellm.turn_off_message_logging:
# make a copy of the _model_Call_details and log it # make a copy of the _model_Call_details and log it
details_to_log.pop("messages", None) details_to_log.pop("messages", None)