search

latysheff / node-sctp

SCTP userspace sockets for Node.js
MIT License
59 stars 10 forks source link

Missing protocol #1

Closed misitoth closed 6 years ago

misitoth commented 7 years ago

Hi,

Can you define LCSAP protocol in lib/defs.js payload_protocol_identifier? LCSAP identifier 29 according to iana SCTP Payload Protocol Identifiers.

Thanks

latysheff commented 7 years ago

Hi, never thought someone will need this :) OK, in next update I will add all know protocols. Meanwhile, just use integer 29.

misitoth commented 7 years ago

Hi,

If you can add all know protocol, there is not gonna be problem in the future. I want to use your package in my test environment with E-SMLC, that is the reason why I desperately need this protocol :-)

Thanks anyway.

latysheff commented 7 years ago

Hey, there's no problem at all, because: 1) SCTP RFC doesn't require setting any protocol identifier (default zero should be OK) 2) you can set it like this (API may change): socket.setWriteOptions({protocol:29});

By the way, are you going to implement LCS-AP on top of node-sctp?

misitoth commented 7 years ago

Hi,

I will use your option2, I hope documentation will be extended with more examples. I am planning LCS-AP in my environment, when sctp layer establish association properly.

latysheff commented 7 years ago

Do you have problems with establishing connection?

latysheff commented 6 years ago

resolved in new version