What is wrong with this JavaScript code ?

Cryptolator

hero member

Activity: 508

Merit: 500

Quote from: olsn on February 06, 2014, 04:22:34 PM

In line 167 you have a timeout of 100milliseconds, the API states though that you should limit your queries to once every 10 seconds,
you should change the timeout from '100' to '10000' (10000milliseconds -> 10 seconds)

Just to be on the save side, I'd change it to 11000 or 12000

Not sure if that was all, but that's what cought my attention at the first glance Wink

- hope this helps!

Thanks for that, didn't know about the 10 seconds minimum for queries. I changed it for 12 sec like you said. Might be it !

Thanks again !

gogodr

sr. member

Activity: 434

Merit: 250

yep, pretty much that.
Also the fact that you don't wait for your request to finish before sending another one.
you can handle GET requests way more easier using Jquery and you can use a callback when you get a response.
That would be a better way to make an asynchronous recursive function like the one you want.

olsn

newbie

Activity: 55

Merit: 0