Skip to content

more efficient server handling #156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
edwardcapriolo opened this issue Apr 14, 2025 · 2 comments
Open

more efficient server handling #156

edwardcapriolo opened this issue Apr 14, 2025 · 2 comments

Comments

@edwardcapriolo
Copy link

Please do a quick search on GitHub issues first, the feature you are about to request might have already been requested.

Expected Behavior

Efficient correct HTTP
bad user data should return 400

Current Behavior

Bad data returns 5xx
Extrac copies of JSON messages are created in memory

Context

More memory pressure,
harder to debug
edwardcapriolo added a commit to edwardcapriolo/mcp-java-sdk that referenced this issue Apr 14, 2025
@edwardcapriolo
More efficient http modelcontextprotocol#156
372e813
edwardcapriolo added a commit to edwardcapriolo/mcp-java-sdk that referenced this issue Apr 14, 2025
@edwardcapriolo
More efficient http modelcontextprotocol#156
8010c9b
edwardcapriolo added a commit to edwardcapriolo/mcp-java-sdk that referenced this issue Apr 15, 2025
@edwardcapriolo
More efficient http modelcontextprotocol#156
2976a72
@edwardcapriolo
Copy link
Author

Hello. Do you mind giving this a look. I am on vacation this week so if you want to get some cleanups out of me this is the week :)

This was referenced Apr 15, 2025
Closed
Open
edwardcapriolo added a commit to edwardcapriolo/mcp-java-sdk that referenced this issue Apr 15, 2025
@edwardcapriolo
More efficient http modelcontextprotocol#156
6a95378
edwardcapriolo added a commit to edwardcapriolo/mcp-java-sdk that referenced this issue Apr 16, 2025
@edwardcapriolo
More efficient http modelcontextprotocol#156
5a59a32
@edwardcapriolo
Copy link
Author

Third rebase as others include fixes inside this PR. Please have a look thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@edwardcapriolo