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

Small fixes #73

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Small fixes #73

wants to merge 5 commits into from

Conversation

Istador
Copy link
Contributor

@Istador Istador commented Oct 24, 2024

No description provided.

There are TCP timeouts, but in situations where it fails instantly there would otherwise be no delay.
(E.g. firewall rejection, or when in flight mode.)
This change only makes it work when in offline/airplane mode.
When connected to a server it still crashes.
Likely due to a different part of the code.

Thanks to https://github.com/piplup55 for pointing out to me that this worked in old versions.
I then tested a few older builds and found out that it started to not work with version `v1.1.0`.
Using `git bisect` I was able to find out that commit 34691af introduced the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant