Conversation
- spin up a local server to handle redirects - create an MCPOauth class to handle oauths
- add storage to store tokens - in mcp connection, add headers
- added a startAuthentication protocol
- add protectedresource key to mcp server status - shift mcp server auth status handling to core
- correct lints in MCPManagerSingleton
dosubot
bot
added
the
size:XL
RomneyDa
left a comment
RomneyDa
left a comment
There was a problem hiding this comment.
Adds suppot for oauth for MCP servers!
|
@uinstinct we noticed that the linear mcp which requires OAuth works with the current extension It seems like maybe the npm |
|
Reopened since the respons linear works is it uses |
|
@uinstinct trying this out, could you tweak to make the tooltip just 
|
|
@uinstinct Fails with the sentry MCP server, appears that it's because of the redirect URL which does not hit anything Noticed this comment which seems relavent
e.g. approval redirects to |
There was a problem hiding this comment.
Tested, works with https://hub.continue.dev/dallin/linear-sse
Note, doesn't work with Sentry MCP (405), but seems to be an exception, can iterate as needed
|
🎉 This PR is included in version 1.5.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.7.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
3 participants
Description
Oauth support for MCP
closes CON-2672
Checklist
Screen recording or screenshot
feat.mp4
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Added OAuth support for MCP by spinning up a local server to handle authentication redirects and integrating the flow with the VS Code extension.