Skip to content

Conversation

@IAmSurajBobade
Copy link
Contributor

mcp/sse: add support to provide for options for SSE transport

  • Add SSEOptions struct to define SSE handler options.
  • Update Signature of NewSSEHandler to accept SSEOptions
  • Ensure Logger is never nill, using ensureLogger helper method
  • update unit tests
  • update example tests

Fixes #503

@jba
Copy link
Contributor

jba commented Sep 19, 2025

The SSEOptions work has to be separate from the logging.
We want the options in today, but the logging can wait another day or two.
Please send a separate PR.

@jba
Copy link
Contributor

jba commented Sep 19, 2025

Refer to #507.

@IAmSurajBobade
Copy link
Contributor Author

closing as duplicate of #508

@IAmSurajBobade IAmSurajBobade deleted the issue/503 branch September 19, 2025 17:05
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.

SSE: Add support for SSE handler options

2 participants