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
As indicated by the requestsdocumentation, this call should include a timeout to avoid blocking for long periods in the event of networking trouble:
Nearly all production code should use this parameter [timeout] in nearly all requests. Failure to do so can cause your program to hang indefinitely
For the Redis workloads I typically have, no response from Redis is better than a slow response from Redis. The client should at least allow users to configure a timeout for these requests, and in my opinion should also have a default timeout enabled without user intervention.
The text was updated successfully, but these errors were encountered:
mmangus
changed the title
Include a timeout for _session.post
Include a timeout for session.post in sync_executeApr 12, 2024
In
sync_execute
, commands are executed by making requests to the Upstash REST API like this......where
session
is arequests.Session
instance.As indicated by the
requests
documentation, this call should include atimeout
to avoid blocking for long periods in the event of networking trouble:For the Redis workloads I typically have, no response from Redis is better than a slow response from Redis. The client should at least allow users to configure a timeout for these requests, and in my opinion should also have a default timeout enabled without user intervention.
The text was updated successfully, but these errors were encountered: