Skip to content

Compatible with spring.webflux.base-path#102

Closed
JoshuaCH wants to merge 1 commit into
modelcontextprotocol:mainfrom
JoshuaCH:main
Closed

Compatible with spring.webflux.base-path#102
JoshuaCH wants to merge 1 commit into
modelcontextprotocol:mainfrom
JoshuaCH:main

Conversation

@JoshuaCH

@JoshuaCH JoshuaCH commented Apr 2, 2025

Copy link
Copy Markdown

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@tzolov

tzolov commented Apr 6, 2025

Copy link
Copy Markdown
Contributor

Related with #79 , #80

tzolov pushed a commit that referenced this pull request Apr 6, 2025
…der (#102)

Adds the ability to specify a base URL prefix for message endpoints in the
WebFlux SSE server transport provider. This enhancement allows for proper
URL construction when the server is running behind a proxy or in a context
with a base path.

- Add new constructor with baseUrl parameter
- Add basePath() method to Builder class
- Modify SSE endpoint event to include baseUrl prefix

Signed-off-by: Christian Tzolov <christian.tzolov@broadcom.com>
@tzolov

tzolov commented Apr 6, 2025

Copy link
Copy Markdown
Contributor

Thanks @JoshuaCH
rebased and merged at 3fa4152

@tzolov tzolov closed this Apr 6, 2025
noear added a commit to solonlab/mcp-java-sdk that referenced this pull request May 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants