Skip to content

Conversation

krrishdholakia
Copy link
Contributor

@krrishdholakia krrishdholakia commented Oct 5, 2025

Title

allows user to debug why mcp server is not working as expected

Related to #15180

Relevant issues

We were initially swallowing MCP server errors, making it hard for clients to debug why their MCP servers were not working via LiteLLM.

so the change is to just re-raise the exception to the client, for easier debugging

Screenshot 2025-10-04 at 6 17 21 PM

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

allows user to debug why mcp server is not working as expected

Related to #15180
Copy link

vercel bot commented Oct 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
litellm Error Error Oct 7, 2025 3:16am

Allow admin to configure static custom headers to send on each mcp request to a backend server

Closes LIT-1186
@krrishdholakia krrishdholakia changed the title fix(mcp/): re-raise exception from server to client fix(mcp/): re-raise exception from server to client + MCP - add static_headers for configuring custom headers on yaml to backend serve r Oct 6, 2025
@krrishdholakia krrishdholakia changed the title fix(mcp/): re-raise exception from server to client + MCP - add static_headers for configuring custom headers on yaml to backend serve r fix(mcp/): re-raise exception from server to client + MCP - add static_headers for configuring custom headers on yaml to backend server Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant