mirror of
https://github.com/SabreTools/MPF.git
synced 2026-02-07 21:30:16 +00:00
[Problem] Checking Redump against a dump fails when Redump is not available #310
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @InternalLoss on GitHub (Jun 20, 2021).
Originally assigned to: @mnadareski on GitHub.
Version
What version are you using?
Build
What runtime version are you using?
Describe the issue
I assume this is an issue with Redump checking as Redump is currently timing out - when completing a dump using the WIP version + Redump credentials, the program hangs and has to be forcibly closed.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
App should gracefully time out (and maybe warn the user that redump.org was not accessible) after 5-10 seconds, and potentially show "Querying redump.org" while this occurs.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
@InternalLoss commented on GitHub (Jun 20, 2021):
cURL logs from trying to GET http://redump.org/ for reproduction.
@mnadareski commented on GitHub (Jun 20, 2021):
Turns out, the default timeout for requests is 100(!) seconds. I wasn't aware of this. I'll set a much more reasonable timeout to fix this.