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

fix(proxy_server.py): enable sending test connections when slack alerting added to proxy via ui

Browse source
This commit is contained in:
Krrish Dholakia 2024-01-04 17:00:07 +05:30
parent c7644915f9
commit 6b708347f3
1 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
litellm/proxy/proxy_server.py
View file

@ -2286,7 +2286,7 @@ async def update_config(config_info: ConfigYAML):
Currently supports modifying General Settings + LiteLLM settings Currently supports modifying General Settings + LiteLLM settings
""" """
global llm_router, llm_model_list, general_settings, proxy_config global llm_router, llm_model_list, general_settings, proxy_config, proxy_logging_obj
try: try:
# Load existing config # Load existing config
config = await proxy_config.get_config() config = await proxy_config.get_config()
@ -2323,7 +2323,14 @@ async def update_config(config_info: ConfigYAML):
} }
# Save the updated config # Save the updated config
config = await proxy_config.save_config(new_config=config) await proxy_config.save_config(new_config=config)
# Test new connections
## Slack
if "slack" in config.get("general_settings", {}).get("alerting", []):
await proxy_logging_obj.alerting_handler(
message="This is a test", level="Low"
)
return {"message": "Config updated successfully"} return {"message": "Config updated successfully"}
except HTTPException as e: except HTTPException as e:
raise e raise e