Make getIP call async
Description
Make getIP call async
Why it should be implemented
In standard operation, getIP is the only outgoing connection made by the server.
This call can take a long time or fail.
Optional: implementation suggestions
Also, when call fails (error code != 200), standard message should be displayed instead of the reported error message. (got some raw html for 504 gateway timeout displayed in place of ip infos)
I guess you are talking about the node branch, right?
The node version is being completely rewritten currently and uses more async / await than the current one. I will add a better error handling.
I think he's talking about the JS, the test doesn't start until we get a response from getIP, and if there's some server side problem it can take a long time.
@adolfintel yes, was talking about the JS.
Has there been a solution to this? My speed test server functions fine when using a private IP but experimenting with using public IPs to run a speed test and I get a LONG delay after hitting the "Start" button and then get the HTML code showing up on the screen that is essentially the 504 Gateway Timeout error. I'd like to get this functioning properly but my knowledge of Javascript is a bit limited.
@RonRN18 You can disable IP/ISP detection if your use case doesn't require it or you can try to investigate the problem.
See here for a list of parameters that can be changed: https://github.com/librespeed/speedtest/wiki/Making-a-custom-front-end