Title: Reconsider PR #850: Wrong Fix for a Potentially Invalid Use Case · Issue #929 · modelcontextprotocol/java-sdk · GitHub
Open Graph Title: Reconsider PR #850: Wrong Fix for a Potentially Invalid Use Case · Issue #929 · modelcontextprotocol/java-sdk
X Title: Reconsider PR #850: Wrong Fix for a Potentially Invalid Use Case · Issue #929 · modelcontextprotocol/java-sdk
Description: References: PR #850, Issue #260 Rationale 1: charset=utf-8 is non-conformant RFC 8259 §8.1 and IANA do not recognize charset as a valid parameter for application/json. This might result in status 415 with strict servers. csharp-sdk issue...
Open Graph Description: References: PR #850, Issue #260 Rationale 1: charset=utf-8 is non-conformant RFC 8259 §8.1 and IANA do not recognize charset as a valid parameter for application/json. This might result in status 4...
X Description: References: PR #850, Issue #260 Rationale 1: charset=utf-8 is non-conformant RFC 8259 §8.1 and IANA do not recognize charset as a valid parameter for application/json. This might result in status 4...
Opengraph URL: https://github.com/modelcontextprotocol/java-sdk/issues/929
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Reconsider PR #850: Wrong Fix for a Potentially Invalid Use Case","articleBody":"**References:** PR #850, Issue #260\n\n### Rationale 1: `charset=utf-8` is non-conformant\n\n[RFC 8259 §8.1](https://datatracker.ietf.org/doc/html/rfc8259#section-11) and [IANA](https://www.iana.org/assignments/media-types/application/json) do not recognize `charset` as a valid parameter for `application/json`.\n\nThis might result in status 415 with strict servers.\n[csharp-sdk issue](https://github.com/modelcontextprotocol/csharp-sdk/issues/1527)\n\n### Rationale 2: Non-ASCII tool names are spec-violating\n\nThe [MCP spec](https://modelcontextprotocol.io/specification/2025-11-25/server/tools) and accepted [SEP-986](https://github.com/modelcontextprotocol/modelcontextprotocol/issues/986) both restrict tool names to `[a-zA-Z0-9_\\-.]`. Non-ASCII characters like `天気予報` are outside this set (`SHOULD`, not `MUST` — but still discouraged).\n\n\n### Proposed Actions\n1. Revert `Content-Type` header change\n2. Add tool name validation at registration\n\nHappy to submit a PR if maintainers agree.\n\n@Kehrlann ","author":{"url":"https://github.com/jayaraman-venkatesan","@type":"Person","name":"jayaraman-venkatesan"},"datePublished":"2026-04-21T17:18:27.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/929/java-sdk/issues/929"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:86462de5-88ad-7c23-a76d-3b742df3093d |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | C6E8:3D821D:767CFD:A46194:6A604314 |
| html-safe-nonce | f84b6f8f53ff8d4a946f7856ece535b8b354632794951f3d382cca7cc98679c2 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNkU4OjNEODIxRDo3NjdDRkQ6QTQ2MTk0OjZBNjA0MzE0IiwidmlzaXRvcl9pZCI6IjgwMTE5MDIxMDU2NzE2MDU5NiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | f2c2f8b6c73f87d6e49b85e608d33aed44f690c7b11a49f0d86bde0a0d0a3274 |
| hovercard-subject-tag | issue:4304225099 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/modelcontextprotocol/java-sdk/929/issue_layout |
| twitter:image | https://opengraph.githubassets.com/01c7fbacf8a9eb61845654135ee2c7f8eaa4e701acf8c3d76308628daa7b9785/modelcontextprotocol/java-sdk/issues/929 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/01c7fbacf8a9eb61845654135ee2c7f8eaa4e701acf8c3d76308628daa7b9785/modelcontextprotocol/java-sdk/issues/929 |
| og:image:alt | References: PR #850, Issue #260 Rationale 1: charset=utf-8 is non-conformant RFC 8259 §8.1 and IANA do not recognize charset as a valid parameter for application/json. This might result in status 4... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | jayaraman-venkatesan |
| hostname | github.com |
| expected-hostname | github.com |
| None | 60da8c2a42fa2bbf5f7567474990ec467836a84444262a58e200fa91b7f3d2d0 |
| turbo-cache-control | no-preview |
| go-import | github.com/modelcontextprotocol/java-sdk git https://github.com/modelcontextprotocol/java-sdk.git |
| octolytics-dimension-user_id | 182288589 |
| octolytics-dimension-user_login | modelcontextprotocol |
| octolytics-dimension-repository_id | 919609219 |
| octolytics-dimension-repository_nwo | modelcontextprotocol/java-sdk |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 919609219 |
| octolytics-dimension-repository_network_root_nwo | modelcontextprotocol/java-sdk |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 9824515e740d83d5eb82168a089b806ab0fe04a1 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width