Xiao Yijun
Xiao Yijun
@pcarleton thanks for your information, I'll update this PR once the issue is addressed.
Hi @cliffhall @pcarleton , I just rebased this pr and update the implementation, PTAL. You can use mcp-auth sample for testing. 1. Clone the js sdk repo by `git clone...
@cliffhall Thanks for the suggestion! I’ll give that a try and see how it works.
Hi @cliffhall , I just tested this implementation with the simpleStreamableHttp example from the TypeScript SDK server, and it works as expected. However, some setup is required beforehand: 1. In...
> @xiaoyijun Sorry to have let this one slip through the cracks. Could you resolve conflicts, so I can test? Sure,let's do it next monday.
Hi @cliffhall , thanks for your patience, I have updated this PR, you can test it follow what I mentioned above (https://github.com/modelcontextprotocol/inspector/pull/345#issuecomment-2949668632). I also removed the resource config since the...
Hi @cliffhall , thanks for your testing and detailed feedback. I will update it in the next few days.
Hi @cliffhall , I’ve updated the code according to your suggestions. Please take a look when you have time.
Hi @cliffhall, CI failed due to test failures after the rebase. I’ve updated it — PTAL.
> Question, @xiaoyijun could this also have a field for OAuth roles to ask for? We currently don't have a good solution for that. Definitely feel free to push back...