feat: add url property to RequestInfo interface#1353
feat: add url property to RequestInfo interface#1353valentinbeggi wants to merge 1 commit intomodelcontextprotocol:v1.xfrom
Conversation
🦋 Changeset detectedLatest commit: b8cb152 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
d717684 to
f385229
Compare
commit: |
|
Unfortunately the node server cc. @KKonstantinov EDIT: i read the comment fully. I think I would prefer URL type and unify it across transports. |
f385229 to
b8cb152
Compare
|
@mattzcarey updated ✅ |
|
@mattzcarey Can we please merge this ? This is much needed now Claude requires the resources meta to embed an hash of the path as domain (MCP server can be served on multiple endpoints). |
|
This is coming in v2 but maybe we should back port @felixweinberger ? |
Summary
urlproperty toRequestInfointerface exposing the request URLCloses #1351
Test plan
requestInfotests to verifyurlproperty