-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Ability to double ping measurement. #34
Comments
Try asking him how he solved it. My guess is that it was incorrect server config. |
...or you can just remove the /2 at line 325 in worker.js |
@adolfintel Why do you divide the time by 2 anyway? |
Because of how HTTP works. When you make a XHR request, here's what typically happens:
In total, that's 2 RTT, hence the /2. |
Yes, that's probably a good idea. |
You might also want to mention the limitations of the ping test more clearly: I came here after a hosting provider got called out for false advertising on a forum because the ping results from his speedtest page were half of what you would get from normal ping commands. Disclaimer: I'm not that provider, just a curious bystander. ;) |
I'm sure I had that in the old version of the documentation. |
So in #33 it is mentioned that the ping amount is incorrect, and then the user says it's solved but not explaining how.
I am not very tech savvy, but is there anyway to double the ping amount shown on the page?
Speedtest site: https://speedtest.mineswine.com
The text was updated successfully, but these errors were encountered: