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

Extend keep-alive message to support round trip time measurements #219

Open
BigETI opened this issue Feb 21, 2021 · 3 comments
Open

Extend keep-alive message to support round trip time measurements #219

BigETI opened this issue Feb 21, 2021 · 3 comments
Labels
API good first issue Good for newcomers hacktoberfest Good first issue for hacktoberfest!

Comments

@BigETI
Copy link
Member

BigETI commented Feb 21, 2021

The keep-alive message is being used to keep connections between clients and servers open. It would be great to extend it to measure round trip time between connections, which is known as "ping". "Ping" messages could have data attached to them, so clients or servers can identify to which requests they belong to.

@Bios-Marcel
Copy link
Member

How would we even properly implement this when server and client have different times? I guess we'd need a time server?

@BigETI
Copy link
Member Author

BigETI commented Feb 26, 2021

Wait what, no. We do not need a "time" server. The contained message could be anything random, just to identify from which message it was replied from, therefore making it possible to measure round trip time relaibly.

A starts a stopwatch and sends a message which contains "ABC", B receives message and responses with "ABC". A receives response and stops stopwatch. Measured time can be used for example to display ping.

@Bios-Marcel
Copy link
Member

nvm, i thought of time syncing

@Bios-Marcel Bios-Marcel added good first issue Good for newcomers hacktoberfest Good first issue for hacktoberfest! labels Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API good first issue Good for newcomers hacktoberfest Good first issue for hacktoberfest!
Projects
None yet
Development

No branches or pull requests

2 participants