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

FIx Endlessloop if connection lost #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlpixTM
Copy link

@AlpixTM AlpixTM commented Nov 22, 2024

Returns it in seconds as float with microsecond precision. We need ms here to compare to timeout ms. So multiply by 1000.

Before it was the number of thousand seconds since Unix.

Waiting (WaitForData() ) for a delta of 3000 (timeout) resulted to wait for 3000 000 seconds (practical infinite loop), now it's 3000ms.

Now a connection times out.

Did not test for side effects, only looked fast over it, but they are now also in ms not thousand seconds since Unix.

// Returns it in seconds as float with micosecond precision. We need ms here to compare to timeout ms. So multiply by 1000.
@AlpixTM AlpixTM changed the title Update snap_sysutils.php FIx Endlessloop if connection lost Nov 25, 2024
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