Restarting Alert Stream on Connection Loss or Timeout about pyhik

[mezz64]

Thanks for digging into this. I only recently realized (thanks to a recent PR) that the timeout being set on the requests session object is doing nothing. With that knowledge the reconnect issues certainly make a lot more sense.

I see in your fork you added the timeout parameter to the init function, do you have a use case for this? I'd rather just set constants for connection and read timeout (5 and 60) and pass them into the appropriate functions as needed. I think these values are fairly safe and it's unlikely they would need tweaked higher.

[shaibt]

In my use case I have different sensitivities to camera reconnect times so I initially added to init() so will be configurable but on second thought its not that important and setting constants is good enough

[mezz64]

Suggestions have been implemented. I'm leaving the watchdog for now, but may remove it later. As you've mentioned with the timeouts properly in place it really shouldn't be needed.