Skip to content

doc/mixed auth example#325

Open
valentinbeggi wants to merge 4 commits intomainfrom
doc/auth-example
Open

doc/mixed auth example#325
valentinbeggi wants to merge 4 commits intomainfrom
doc/auth-example

Conversation

@valentinbeggi
Copy link
Copy Markdown
Collaborator

@valentinbeggi valentinbeggi commented Jan 16, 2026

Mixed auth example

Note that it doesn't work very well in chatGPT. chatGPT should show UI for login in case the user is unauthenticated, except that they require that the tools return a securitySchemes key, which is not yet in the MCP spec nor in TS sdk

modelcontextprotocol/modelcontextprotocol#1488

#301

Comment on lines +5 to +6
"source.organizeImports.biome": "explicit",
"source.fixAll.biome": "explicit"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you need to manually set those 2 ?

Comment on lines +12 to +15
"server:build": "tsc -p tsconfig.server.json",
"server:start": "node dist/index.js",
"web:build": "tsc -b web && vite build -c web/vite.config.ts",
"web:preview": "vite preview -c web/vite.config.ts"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

those are legacy, you can remove

@qchuchu
Copy link
Copy Markdown
Contributor

qchuchu commented Feb 10, 2026

@valentinbeggi @neil-ac something is blocked here ? :)

@valentinbeggi
Copy link
Copy Markdown
Collaborator Author

@qchuchu yes it's not working 🦧

cc modelcontextprotocol/modelcontextprotocol#1488

@qchuchu qchuchu added the blocked When PR cannot be merged because of external dependencies label Feb 19, 2026
@qchuchu qchuchu changed the title doc/auth example doc/mixed auth example Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked When PR cannot be merged because of external dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants