Skip to Content.
Sympa Menu

ndt-dev - Re: [ndt-dev] patches for the HTML5-frontend ?

Subject: NDT-DEV email list created

List archive

Re: [ndt-dev] patches for the HTML5-frontend ?


Chronological Thread 
  • From: Aaron Brown <>
  • To: Athanasios Douitsis <>
  • Cc: , Angela Dimitriou <>
  • Subject: Re: [ndt-dev] patches for the HTML5-frontend ?
  • Date: Thu, 22 Oct 2015 12:57:08 -0400



Hi,

On Oct 21, 2015, at 11:06 AM, Athanasios Douitsis <> wrote:

Hello,

In case one desires to submit a small patch (e.g. for the ndt-browser-client.js) under HTML5-frontend, would a pull request on https://github.com/ndt-project/ndt be appropriate?

You stated it wouldn't be easy but if you do come up with one, that's the easiest way.

I have run across an mlab ndt server that for some reason does not respond normally to the websocket version of the protocol. While this sounds bad, it actually gave me the opportunity to realize that, upon creation of the websocket in ndt-browser-client.js, the onclose and onerror events are not handled, so if the connection is dropped abnormally or any other sort of error occurs, the application using NDTjs doesn't learn it and can hang in there indefinitely.

If I may be permitted, I'd like to submit a small patch to fix that.

Also, another question, in my tests I am using Workers under Firefox quite successfully. It seems that the bug saying that WebSockets were not available in Workers (*) is nowadays fixed. Anyone aware whether there is any issue concerning the accuracy of measurements? Because as far as I can tell it works beautifully.

Sweet. I guess it's a more recent version. If you'd be interested in patching that issue, we'd be happy to accept it. Otherwise, I'm happy to take a look at fixing it.

Cheers,
Aaron


(*) https://bugzilla.mozilla.org/show_bug.cgi?id=504553

Thanks,
--
Athanasios Douitsis





Archive powered by MHonArc 2.6.16.

Top of Page