-
Notifications
You must be signed in to change notification settings - Fork 466
Specify transport type on startup #255
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
Comments
Hey @aaronpowell. This seems reasonable. Would you like to add a PR? |
I think there would be a level of intersection with #258 - maybe when the |
Could we not rectify the situation in the client app, so that if you have supplied an http(s) url it switches the type to SSE? |
Since the client app gets its info on what to put in the transport drop down + input fields by calling the Like I suggested, expanding on the solution in #258 |
Is your feature request related to a problem? Please describe.
When you first run the inspector it defaults to
stdio
as the transport, unless previously specified. So if you run:npx -y @modelcontextprotocol/inspector http://localhost:1234/sse
it will default tostdio
and upon changing the dropdown, set it to the default ofhttp:''localhost:3001/sse
, removing your default value.Describe the solution you'd like
When running inspector you should be able to specify the transport type to use, or it should be inferred from the server provided (eg: if it's
http(s)
prefixed, using SSE, otherwise use stdio).Describe alternatives you've considered
I can't find any way in the code that this can be set.
Additional context
N/A
The text was updated successfully, but these errors were encountered: