Closed rstoneback closed 3 years ago
This looks good. Is removing sys.stdout.flush()
intentional?
This looks good. Is removing
sys.stdout.flush()
intentional?
Yep. I originally added that back when all feedback was via print to stdout. With logging I don't think flushing stdout has any effect.
Addresses #68
Improves download robustness by ensuring that requests closes each connection. Multiple calls to download in a short timeframe can result in requests reusing a connection but then SSL gets out of sync.