getting corrupted data in one end point about node-rudp HOT 5 OPEN

Thanks for bringing up this issue. I will be looking into it on Saturday.

It's an insanely busy week for me.

Cheers.

from node-rudp.

I think there is a missing 'var' behind 'bools' on line 16 of Packet.js btw

from node-rudp.

Hey
An update on the problem.
I tried sending an html file from my laptop to a server using rudp and the server got the file with no problem ( file size: 200K)
But when I tried the library with a large file ( a pdf file 9M) the received file had different size every time I tried.
So I think it does not work with large files apparently.
I would appreciate if you could take a look at this.
In the meantime, I'm trying to read the code and to see if I can find something.

Thanks brother

from node-rudp.

Thanks for bringing up this issue. I will be looking into it on Saturday.

It's an insanely busy week for me.

Cheers.

Hey,
After banging my head on my wall enough I found the problem.
The problem was when we have enough windows to make the random function to generate the baseSequenceNumber same for two consecutive windows.

if (packet.getSequenceNumber() === this._syncSequenceNumber) { this._packetSender.send(Packet.createAcknowledgementPacket(packet.getSequenceNumber())); return; }

In this case, the server will stay desynced and send the ack for the sync packet. The client after seeing the ack for sync packet would continue sending data. Since the server is desynced, the server will ignore the incoming packets which make the system break!

Fix:
I will submit a pull request soon.
I fixed this if and also changed a couple of small things.
I would appreciate if you take a look at it.

Cheers <3

from node-rudp.

I made the pull request.

btw since I read the whole code I think I can write documentation for this library like what are the classes and what does each one do.

from node-rudp.