Title: Add warning log for unknown/expired session IDs by nielskaspers · Pull Request #2229 · modelcontextprotocol/python-sdk · GitHub
Open Graph Title: Add warning log for unknown/expired session IDs by nielskaspers · Pull Request #2229 · modelcontextprotocol/python-sdk
X Title: Add warning log for unknown/expired session IDs by nielskaspers · Pull Request #2229 · modelcontextprotocol/python-sdk
Description: Summary Adds logger.warning() in the else branch of _handle_stateful_request() when a request is rejected due to an unknown or expired session ID Adds test assertion verifying the warning is logged with the session ID The other two branches already log at debug/info level, but this branch silently returned 404. This makes it difficult to diagnose connection failures caused by stale session IDs (e.g. after server restarts). Test plan Existing test test_unknown_session_id_returns_404 updated and passing Verified warning includes the session ID string for traceability Closes #2204
Open Graph Description: Summary Adds logger.warning() in the else branch of _handle_stateful_request() when a request is rejected due to an unknown or expired session ID Adds test assertion verifying the warning is logge...
X Description: Summary Adds logger.warning() in the else branch of _handle_stateful_request() when a request is rejected due to an unknown or expired session ID Adds test assertion verifying the warning is logge...
Opengraph URL: https://github.com/modelcontextprotocol/python-sdk/pull/2229
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:c0d8bcd7-19cf-da0c-3835-9d3f4e11e98f |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | BE12:3BBBDD:1639C9F:1FE3F4B:6A58B90B |
| html-safe-nonce | 2f47926ee948c52c5ba75870c9fc4c157a37f80437cc8de5a6ab9547487c31d3 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCRTEyOjNCQkJERDoxNjM5QzlGOjFGRTNGNEI6NkE1OEI5MEIiLCJ2aXNpdG9yX2lkIjoiODczNzUwNDQwMzMyMjc4MDI3IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | defb25624645de35ec710b537ddfb7c7ede019b629592cbae1cf325e07eb75f0 |
| hovercard-subject-tag | pull_request:3362934696 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,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/modelcontextprotocol/python-sdk/pull/2229/files |
| twitter:image | https://avatars.githubusercontent.com/u/153818647?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/153818647?s=400&v=4 |
| og:image:alt | Summary Adds logger.warning() in the else branch of _handle_stateful_request() when a request is rejected due to an unknown or expired session ID Adds test assertion verifying the warning is logge... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f2a0c7865178af3d91dd9f13b0cdfc3c73a2529c873d2780bb4c01271a57ec6 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/modelcontextprotocol/python-sdk git https://github.com/modelcontextprotocol/python-sdk.git |
| octolytics-dimension-user_id | 182288589 |
| octolytics-dimension-user_login | modelcontextprotocol |
| octolytics-dimension-repository_id | 862584018 |
| octolytics-dimension-repository_nwo | modelcontextprotocol/python-sdk |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 862584018 |
| octolytics-dimension-repository_network_root_nwo | modelcontextprotocol/python-sdk |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 8aae7b8d6caacacf5c66eaeb2702d8dc88d85b4a |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width