Skip to content

Make "Stateless" mode sessionless - #760

Merged
halter73 merged 2 commits into
mainfrom
halter73/sessionless
Oct 2, 2025
Merged

Make "Stateless" mode sessionless#760
halter73 merged 2 commits into
mainfrom
halter73/sessionless

Conversation

@halter73

@halter73halter73 commented Sep 10, 2025

Copy link
Copy Markdown
Contributor
  • This removes the requirement to configure data protection which is the motivating reason for this change
  • This means that McpServer.ClientInfo will always be null in stateless mode

We could also consider removing McpServerOptions.KnownClientInfo and StreamableHttpServerTransport.OnInitRequestReceived since the StreamableHttpHandler no longer uses them, but they still could be useful to people trying to use StreamableHttpServerTransport directly.

I'll be on vacation the next two weeks. If someone wants to go ahead and merge this before I get back, that's fine by me. Otherwise, I'll address any feedback once I'm back from vacation.

Fixes#682

@halter73
Make "Stateless" mode sessionless
a3b519b
- This removes the requirement to configure data protection which is the motivating reason for this change
- This means that McpServer.ClientInfo will always be null in stateless mode
eiriktsarpalis
eiriktsarpalis reviewed Sep 30, 2025
View reviewed changes
Comment threadsrc/ModelContextProtocol.AspNetCore/StreamableHttpHandler.cs
@halter73halter73 mentioned this pull request Oct 1, 2025
Closed
@halter73
Merge branch 'main' into halter73/sessionless
2815ee9
# Conflicts:
#	src/ModelContextProtocol.AspNetCore/StreamableHttpHandler.cs
#	src/ModelContextProtocol.Core/Server/McpServerExtensions.cs
@halter73
halter73 merged commit 47b9582 into mainOct 2, 2025
9 of 10 checks passed
@halter73
halter73 deleted the halter73/sessionless branch October 2, 2025 22:57
@ghostghost mentioned this pull request Oct 4, 2025
Closed
@jozkeejozkee mentioned this pull request Oct 8, 2025
Merged
22 tasks
@AarononthewebAaronontheweb mentioned this pull request Oct 10, 2025
Merged
This was referenced Oct 13, 2025
Closed
Closed
This was referenced Oct 20, 2025
Open
Open
Merged
@TianqiZhang

TianqiZhang commented Nov 26, 2025

Copy link
Copy Markdown

@halter73 our MCP server is a remote stateless server, and logging client info is very important to us. With this change, we found it impossible to know the origin client where a tool call comes from.

Is there a reason why we stopped encoding client info into session id in stateless mode? Any workaround you can recommend?

The reason we chose stateless mode is that we deploy to multi-instance azure app service and we don't have a good session affinity solution. Any suggestion on that? If we can use stateful mode we will be happy to do so. //cc @mikekistler

@halter73halter73 mentioned this pull request Nov 26, 2025
Closed
@dependabotdependabotBot mentioned this pull request Dec 15, 2025
Merged
@jeffhandleyjeffhandley added the breaking-change This issue or PR introduces a breaking change label Feb 20, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eiriktsarpaliseiriktsarpaliseiriktsarpalis approved these changes
@stephentoubstephentoubAwaiting requested review from stephentoub
@MackinnonBuckMackinnonBuckAwaiting requested review from MackinnonBuck
@mikekistlermikekistlerAwaiting requested review from mikekistler
@PederHPPederHPAwaiting requested review from PederHP

Assignees

No one assigned

Labels

breaking-changeThis issue or PR introduces a breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Stateless Mode with Scaling Not Working

4 participants

@halter73@TianqiZhang@eiriktsarpalis@jeffhandley