Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report delayed errors #66

Merged
merged 8 commits into from
Oct 14, 2024
Merged

Report delayed errors #66

merged 8 commits into from
Oct 14, 2024

Conversation

sfan5
Copy link
Contributor

@sfan5 sfan5 commented Oct 13, 2024

fixes #65

note: I am running this in prod, just want to keep an eye on bugs before merging.

@sfan5 sfan5 added the feature label Oct 13, 2024
@sfan5
Copy link
Contributor Author

sfan5 commented Oct 13, 2024

how it looks on the client:

2024-10-13 10:27:16: ACTION[Server]: Announcing start to servers.minetest.net
2024-10-13 10:32:16: ERROR[CurlFetch]: HTTPFetch for servers.minetest.net/announce returned response code 409
2024-10-13 10:32:16: ERROR[CurlFetch]: Response body:
2024-10-13 10:32:16: ERROR[CurlFetch]: Request has been filed, but the previous request encountered the following error:
2024-10-13 10:32:16: ERROR[CurlFetch]: Server x.x.x.x port 30000 did not respond to ping

server.py Outdated Show resolved Hide resolved
server.py Outdated Show resolved Hide resolved
@sfan5 sfan5 merged commit 3d08cd4 into master Oct 14, 2024
2 checks passed
@sfan5 sfan5 deleted the delayed_err branch October 14, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Report problems during asyncFinishThread back to user
2 participants