Skip to content

Fix: Prevent exceptions when disabling MCP Server #2899

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.

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

Closed
wants to merge 1 commit into from

Conversation

YunKuiLu
Copy link
Contributor

  • Prevent projects depending on mcp-server-webmvc or mcp-server-webflux from exceptions when spring.ai.mcp.server.enabled is set to false.
  • Add unit tests to verify.

When the project depends on the spring-ai-starter-mcp-server-webmvc package but attempts to temporarily disable the MCP server (via spring.ai.mcp.server.enabled=false), the following exception is thrown:
image

@YunKuiLu
Copy link
Contributor Author

Hi @tzolov , as the main author of MCP Server, could you review this PR?
Thank you!

- Prevent projects depending on `mcp-server-webmvc` or `mcp-server-webflux` from exceptions when `spring.ai.mcp.server.enabled` is set to `false`.
- Add unit tests to verify.

Signed-off-by: YunKui Lu <[email protected]>
@markpollack markpollack added bug Something isn't working MCP labels Apr 29, 2025
@markpollack markpollack added this to the 1.0.0-RC1 milestone Apr 29, 2025
@tzolov tzolov self-assigned this May 1, 2025
@tzolov
Copy link
Contributor

tzolov commented May 1, 2025

Thanks you for catching and fixing this @YunKuiLu ! Bonus points for the tests ;)

Rebased and merged at c2843f1

@tzolov tzolov closed this May 1, 2025
@YunKuiLu YunKuiLu deleted the fix-mcpserver branch May 19, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working MCP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants