Commit graph

1,044 commits

Author SHA1 Message Date
Felix Weinberger
111e282386
feat: add session token authentication to proxy server
- Generate random session token on server startup using crypto.randomBytes
- Add authentication middleware requiring Bearer token for all sensitive endpoints
- Apply auth to /mcp, /stdio, /sse, and /message endpoints
- Display session token in console on startup
- Add DANGEROUSLY_OMIT_AUTH environment variable to disable auth for development
- Update README with authentication documentation

This prevents unauthorized access to the proxy server's ability to execute
local processes and connect to MCP servers.
2025-06-10 18:16:28 +01:00
Felix Weinberger
e8e990987c
fix: bind to localhost by default to prevent DNS rebinding attacks
- Change default binding from 0.0.0.0 (all interfaces) to 127.0.0.1 (localhost only)
- Add HOST environment variable to override binding address when needed
- Update console output to show actual binding address
- Document security change in README with warnings about binding to all interfaces

This prevents remote attackers from accessing the MCP Inspector proxy server,
which has the ability to execute local processes. The proxy should only be
accessible from the local machine unless explicitly configured otherwise.
2025-06-10 18:16:24 +01:00
Cliff Hall
03edbb01ce
Merge pull request #488 from msabramo/tooltip-for-url-field
Add tooltip for URL field
2025-06-06 13:38:44 -04:00
Marc Abramowitz
d5ef137c42 Merge branch 'main' into tooltip-for-url-field 2025-06-06 06:00:37 -07:00
Marc Abramowitz
9a7ba3451a npm run prettier-fix 2025-06-06 05:49:49 -07:00
shaun smith
5ade12d0b1
Merge pull request #478 from cliffhall/normalize-connection-logging
Normalize connection logging
2025-06-05 13:58:49 -07:00
Cliff Hall
cf07937f8e
Merge branch 'main' into normalize-connection-logging 2025-06-05 16:55:27 -04:00
Marc Abramowitz
2ecfdb0f2e Display tooltip only if sseUrl is set. 2025-06-05 00:25:18 -07:00
Ola Hungerford
a2d2cf6f3d
Merge pull request #459 from msabramo/resizable-left-sidebar
Resizable left sidebar
2025-06-04 19:58:59 -07:00
Ola Hungerford
798bc1b793
Merge pull request #489 from cliffhall/bump-version-0.14.0 0.14.0
Bump Inspector to 0.14.0
2025-06-04 13:46:39 -07:00
cliffhall
8ef91e359e Bump Inspector packages to 0.14.0, and sdk to 1.12.1 2025-06-04 16:13:35 -04:00
Cliff Hall
4bffa62516
Merge pull request #485 from dr3s/450-template-queries
fix: Resource template query parameters not appended to request URI
2025-06-04 14:58:22 -04:00
Cliff Hall
cc28a891e9
Merge branch 'main' into 450-template-queries 2025-06-04 14:54:57 -04:00
dr3s
9508cc4627 pretty 2025-06-04 14:44:15 -04:00
Cliff Hall
3772ff30a2
Merge pull request #469 from modelcontextprotocol/pcarleton/auth-debugger-draft
[auth] Support new auth metadata endpoint from draft spec
2025-06-04 13:22:08 -04:00
Marc Abramowitz
5da1dc3256 Add tooltip for URL field
A quick way to see the full URL if it's cut off

Inspired by @cliffhall's
[comment](https://github.com/modelcontextprotocol/inspector/pull/459#issuecomment-2932277785)
2025-06-03 22:54:19 -07:00
Marc Abramowitz
13d31596af Merge branch 'main' into resizable-left-sidebar 2025-06-03 22:29:15 -07:00
Marc Abramowitz
93553d591f Revert "Add tip to README.md"
This reverts commit 0e5e94393a.
2025-06-03 22:24:17 -07:00
cliffhall
c485e57a39 Add reporting of the Proxy <-> Server sessionId. It's only possible to get the session id for this leg of the proxy if it is a StreamableHttp connection.
* In server/src/index.ts
  - slight format tweak of the Client <-> Proxy message so that the two messages will line up when displayed
* In server/src/mcpProxy.ts
  - add reportedServerSession boolean initialized to false
  - in transportToServer.onmessage,
    - if reportedServerSession isn't set yet, report the sessionId if present on the transport, and then set reportedServerSession to true
2025-06-03 17:46:57 -04:00
dr3s
21b86d409c add test 2025-06-03 13:58:33 -04:00
dr3s
26f744866a formatting 2025-06-03 13:58:33 -04:00
dr3s
07f3e4593b remove logs 2025-06-03 13:58:33 -04:00
dr3s
f1650d06e3 use UriTemplate variable names 2025-06-03 13:58:33 -04:00
dr3s
2f50a4ba0e fix: Resource template query parameters not appended to request URI #450 2025-06-03 13:58:33 -04:00
Paul Carleton
a387c6ad7e
hook warning 2025-06-03 17:29:22 +01:00
Paul Carleton
e2d3989bb2
use scopes from PRM if available 2025-06-03 11:05:45 +01:00
Paul Carleton
9371006838
less noisy errors 2025-06-03 11:02:34 +01:00
Paul Carleton
0f4a289a4d
Merge branch 'main' into pcarleton/auth-debugger-draft 2025-06-02 16:41:44 +01:00
Paul Carleton
fa01fafa59
version bump + lint 2025-06-02 16:21:55 +01:00
Paul Carleton
98668507c7
version bump 2025-06-02 16:20:36 +01:00
Paul Carleton
2d7936d10b
fix type errors 2025-06-02 16:20:16 +01:00
Cliff Hall
789c90cc0b
Merge branch 'main' into normalize-connection-logging 2025-06-02 10:25:30 -04:00
Cliff Hall
7df0ac40c5
Merge pull request #470 from cliffhall/send-delete-on-streamable-http-disconnect
Send HTTP DELETE on streamable transport disconnect
2025-06-02 10:05:30 -04:00
Ola Hungerford
bf85cebb2f
Merge pull request #476 from olaservo/clarify-proxy-role-in-readme
Add simple architecture overview with note on proxy role
2025-05-30 16:44:28 -07:00
Cliff Hall
1919a0e9b2
Merge pull request #400 from Littly/fix/header
feat: support custom authorization headers, fix #395
2025-05-30 18:42:50 -04:00
Cliff Hall
021e6a423b
Merge branch 'main' into fix/header 2025-05-30 18:39:01 -04:00
Cliff Hall
291c59eaf9
Merge branch 'main' into clarify-proxy-role-in-readme 2025-05-30 18:37:40 -04:00
Cliff Hall
e55cd75e82
Merge branch 'main' into send-delete-on-streamable-http-disconnect 2025-05-30 18:36:10 -04:00
Cliff Hall
52dc2f6db6
Merge branch 'main' into normalize-connection-logging 2025-05-30 17:51:37 -04:00
cliffhall
b732ab4085 prettier 2025-05-30 17:51:22 -04:00
cliffhall
2df723804b Normalize the reporting of connection setup and connection errors. The logging is verbose and different for every server type. This is an attempt to make the console output cleaner and more readable.
* In server/index.js
  - in createTransport()
    - stringify query params to put them on one line
    - remove logging of transport creation (done in callers)
  - in /mcp POST handler
    - report POST request if session id is present
    - otherwise, report new StreamableHttp connection request
    - report creation of server transport
    - report creation of client transport
  - in /stdio GET handler
    - report new STDIO connection request
    - report creation of server transport
    - report creation of client transport
    - report 401 as authentication error without stack
    - in stderr data handler
      - if it includes "MODULE_NOT_FOUND"
        - report "Command not found"
        - remove transports
      - otherwise send to client as usual
  - in /sse GET handler
    - report SSE connection request
    - report 401 as authentication error without stack
    - report 404 as server that may not support SSE
    - report error with "ECONNREFUSED" as "Connection refused" without stack
    - only create, map, and proxy transports if serverTransport was successfully created
  - in /message POST handler
    - report received POST message

* In mcpProxy.ts
  - in onServerError()
    - if error message includes 404 OR error.cause includes ECONNREFUSED, report "Connection refused."
2025-05-30 17:13:53 -04:00
Ola Hungerford
2c527caaae
Merge pull request #462 from dbsanfte/patch-1
Add quick start example
2025-05-30 13:35:41 -07:00
Cliff Hall
438390ecd5
Merge branch 'main' into send-delete-on-streamable-http-disconnect 2025-05-30 11:58:02 -04:00
Cliff Hall
e8d5e09b62
Merge pull request #428 from cliffhall/allow-multiple-client-connections
Allow multiple instances of the Inspector to connect to the same MCP server
2025-05-30 11:44:21 -04:00
olaservo
11757ef615 Add simple architecture overview 2025-05-30 07:46:28 -07:00
xiaoli
3510e1a561 fix: preserve capitalization in custom headers
Express converts all header names to lowercase, which prevents custom headers with uppercase characters from being properly passed to the target service. This fix ensures that custom headers preserve their original capitalization when forwarded through the proxy.
2025-05-30 21:34:40 +08:00
Xiaoli
f87884900e
Merge branch 'main' into fix/header 2025-05-30 10:14:42 +08:00
xiaoli
b51ee625cd style: fix code style to pass CI checks 2025-05-30 10:14:03 +08:00
cliffhall
02aacb46a2 Send HTTP/DELETE when disconnecting client
* In server/src/index.ts
  - add delete handler for /mcp endpoint
    - gets the server transport for the sessionId
    - calls terminateSession on it
    - removes the webapp and server transports for the sessionId from the maps
    - returns status 200 to the client.

* In client/src/lib/hooks/useConnection.ts
  - import Transport
  - add useState for clientTransport, type Transport or null initialized to null
  - in connect() function
    - move creation of client transport down a ways, just before calling client.connect with it
    - immendiately efter calling client.connect, call setClientTransport with the connected transport (has to happen after, so that what's saved has the abort controller, etc. otherwise it doesn't work to save it prior.
    - in disconnect() function
      - immediately call clientTransport.terminateSession if transportType is "streamable-http"
      - setClientTransport to null
2025-05-29 19:54:36 -04:00
Cliff Hall
a7aa72655a
Merge branch 'main' into allow-multiple-client-connections 2025-05-29 16:05:36 -04:00