You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add an optional argument request_read_timeout_seconds and rename current _read_timeout_seconds into session_read_timeout_seconds if request_read_timeout_seconds is set it would override the session one.
I will publish a small PR in that sense.
The text was updated successfully, but these errors were encountered:
also maybe on another note but I find calling the argument read_timeout_seconds and using a timedelta a bit odd, I would expect an int to be passed for "seconds"
Describe the bug
According to the MCP specs:
But looking at the code the timeouts are set at the session level and not easily configured on a per-request basis as the specs recommend:
see
Suggestion
Add an optional argument
request_read_timeout_seconds
and rename current _read_timeout_seconds intosession_read_timeout_seconds
ifrequest_read_timeout_seconds
is set it would override the session one.I will publish a small PR in that sense.
The text was updated successfully, but these errors were encountered: