node-rtp icon indicating copy to clipboard operation
node-rtp copied to clipboard

Absolutely LOVE the code, great job! Help with RFC-2250

Open g00dnatur3 opened this issue 8 years ago • 2 comments

This code is super readable, to the point, and encapsulated perfectly..

The whole implementation is just an object with getters and setters...

sender uses setters, receiver uses getters & the constructor --

so i literally just read the datagram from the socket and throw it to the constructor...

AWESOME..

But my question is, what changes would I need to make to support rfc-2250.

Which getters and setters would need to change to support 2250?

Thanks!!

g00dnatur3 avatar Feb 17 '17 06:02 g00dnatur3

Honestly, when I originally wrote this 6.5 years ago, it was more of a proof-of-concept to be able to send/receive RTP audio packets in node rather than be a real full-fledged module. With that in mind, I don't have any plans to make any changes to this code. There are probably more complete modules available on npm these days.

mscdex avatar Feb 17 '17 11:02 mscdex

@g00dnatur3 I tried to create new RTP module for NodeJS based on RFC3550. If you want you can check it here.

1995parham avatar Jun 02 '17 19:06 1995parham