ovh / overthebox-feeds

OverTheBox - LEDE/OpenWrt feed
GNU General Public License v3.0
106 stars 41 forks source link

overthebox: loop 5 times max to handle timeouts #1126

Closed gdepeyrot closed 3 years ago

gdepeyrot commented 3 years ago

Proposed change loops five times max to handle speedtest timeout when determining upload or download speed

wesraph commented 3 years ago

I tested the commit on my development machine and it works as planned

wesraph commented 3 years ago

The commit (not the PR) needs to have the name prepended with overthebox. You can also squash your two lasts commits together (https://www.git-tower.com/learn/git/faq/git-squash/) You can do as follow https://asciinema.org/a/pFsc9VGxeJ3rGPbyRoSbt3iK1

gdepeyrot commented 3 years ago

Thanks for the squash guide ;-)