Shnezzin / breitbandmessung

Perform breitbandmessung.de speedtests in an automated way
4 stars 2 forks source link

Only downlink OR uplink have to be in spec to trigger Internet OK #3

Closed PA7T closed 2 years ago

PA7T commented 2 years ago

I think in line 132 of speedtest.py the if condition is not correct:

if result_up.text >= MIN_UPLOAD or result_down.text >= MIN_DOWNLOAD:

The OR should be changed to an AND. I think both conditions have to be met, so you can consider your internet ok. Let me know your thoughts.

Shnezzin commented 2 years ago

Thanks for the info. There is an old line left over. It has been adjusted.