PyPi_GrowattServer icon indicating copy to clipboard operation
PyPi_GrowattServer copied to clipboard

Added timeout for each requests

Open nagycsongor opened this issue 1 year ago • 3 comments

Requests without timeout goes into infinity wait which blocks forever the caller. Timeout helps for interrupting these calls and handling as an error.

nagycsongor avatar Mar 23 '24 21:03 nagycsongor

Sorry for the long wait, i like the idea! Do we want to set a default timeout, or rather keep it up to the user?

indykoning avatar Aug 09 '24 09:08 indykoning

@indykoning I do not know, I guess it ups on the user. In my case the problem was that in case of the server is not accessible stayed there forever and I could not exit from this wait on a normal way.

nagycsongor avatar Aug 09 '24 10:08 nagycsongor

@nagycsongor can you also add the timeout to the new merged Noah Requests? I am at the holidays at the moment.

@indykoning I would say a default Timeout like 30 seconds, but the user can also configure a custom timeout. I think for 90% of the Users a default is fine like all the HomeAssistant Users that don't know that the integration is based on your project.

Rappelkiste98 avatar Aug 09 '24 13:08 Rappelkiste98