Timeout causing panic about monolith

[snshn]

@TerminusBot I've added a new flag (-t/--timeout) and bumped the timeout to be 30 seconds by default in #114. I believe that should resolve this problem once and for all. Please let me know if 30 seconds is enough.

[TerminusBot]

I just set the timeout 60 and it works very well now.

[snshn]

@TerminusBot I bumped it to 120 seconds, that seems to be what Chrome and FF are doing regarding network request timeout. If you do -t 0 it'll be infinite. Please let me know if there's anything else that I could improve!

[snshn]

Thank you for the heads up, I'll try to increase the default timeout as well as handle this panic better.

[snshn]

Merged #15 in, should be better on slow connections now, but I think the only way to completely resolve this issue would be to handle failing download attempts in a safe manner (e.g. continue retrieving other assets if one fails).

[nscooling]

Thanks, it handling some but still timing out of others

[snshn]

I improved the code to treat all network errors as blank responses, this should help saving pages with resources that timeout or throw other kinds of errors. Please let me know if it got rid of the panic. I'm thinking of adding a -t option for setting extended timeout in case of slow Internet connection.

[TerminusBot]

thread 'main' panicked at 'called Result::unwrap() on an Err value: reqwest::Error { kind: Decode, source: TimedOut }', src/libcore/result.rs:1165:5
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace.

maybe the best solution is retrying if panic happens

[snshn]

@TerminusBot it happens for the very first request, right?

[TerminusBot]

@snshn totally right

[snshn]

I'll look into it this weekend, thank you for the info

[TerminusBot]

@TerminusBot I've added a new flag (-t/--timeout) and bumped the timeout to be 30 seconds by default in #114. I believe that should resolve this problem once and for all. Please let me know if 30 seconds is enough.

Sorry, 60 is good enough for me and it works well in production, so I may not test.But you have added the new parameter and we have an easy way to solve this bug.
You have a great job! Monolith helps me a lot. Thanks again!