Skip to content

fix: re-raise exception instead of silently dropping MCP team permissions #30477

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Sameerlite merged 2 commits into from
Jun 16, 2026
Merged

fix: re-raise exception instead of silently dropping MCP team permissions #30477

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
65c55c2
fix: re-raise exception instead of silently
factnn Jun 15, 2026
de90bd3
fix: log full traceback when MCP permission lookup fails
factnn Jun 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions litellm/proxy/_experimental/mcp_server/mcp_server_manager.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1421,8 +1421,11 @@ async def get_allowed_mcp_servers(
"No allowed MCP Servers found for user api key auth."
)
return list(combined_servers)
except Exception as e:
verbose_logger.warning(f"Failed to get allowed MCP servers: {str(e)}.")
except Exception:
verbose_logger.exception(
"Failed to get allowed MCP servers; team-level object_permission "
"grants may be dropped. Falling back to global servers only."
)
return allow_all_server_ids

async def resolve_toolset_tool_permissions(
Expand Down
Loading