[Updater.cpp] Increasing stream timeout to avoid read timeout during OTA flash write

[cyberman54]

If using a nginx webserver to feed a https OTA write flash stream with ~1,5MB binary data, i'm getting reproducely _error #6 (stream read timeout) on a ESP32 Pycom LoPy with CPU revision 0.

I was able to stable fix this by applying a patch to Updater.cpp, following a hint for ESP 8266 Update.cpp, found at the end of this issue:

esp8266/Arduino#1157

This PR is a quick&dirty fix. Problem should be further analysed and then solved at the root cause. Eventually this is a TCP related issue.

[Jeroen88]

I am not in favor of this PR. It would be better to use setTimeout on the Stream before you pass it into ::writeStream, outside of UpdateClass. This PR would always set the timeout to 10,000 ms without any possibility to change it.

[cyberman54]

@Jeroen88 is right. I close this.

[cyberman54]

I reopen this, because i did not find a working solution to set the timeout for the stream of class WifiClient before calling Update.writeStream(), see my corresponding issue here

[cyberman54]

Closing this again, since the problem should be solved with pending PR #1979

[haimiko]

Tested this "fix". Doesn't help. Still getting "ERROR[6]: Stream Read Timeout". Even up'ped the timeout to 60 seconds with no change. This issue is more prevalent on larger bins.

BTW. This fix seems to help on local networks but if you're trying to update over the internet you'll still experience the same issue.

[Jeroen88]

@haimiko Maybe you could try my PR #1979? If you look at my comments in that PR you can see that there are several timeout issues in the current libraries that should be solved with that PR.