[remove] ecdhCurve option from tls.connect for https links#24
[remove] ecdhCurve option from tls.connect for https links#24DELUUXE wants to merge 1 commit intogabek:masterfrom
Conversation
Fixes: The "options.ecdhCurve" property must be of type string. Received type boolean (false)
|
Can this please be merged? Its a blocker for using this for my project. |
|
Hi there. Admittedly I haven't had a use for this library lately. I've moved to writing everything in Go 😃 I'd be happy to merge this in and publish a new version, but maybe there should be a longer term plan. I didn't realize this was being actively used. Happy to hear any thoughts! |
|
Hi @gabek, I think this library works great as it is. The If you do have some additional time; making the loglevel configurable would be nice. It now has some console.trace statements that can't be disabled. Which is polluting the stdout when you are retrieving a bunch of streams. |
Fixes
The "options.ecdhCurve" property must be of type string. Received type boolean (false)when using https urlsMay solve issue #21
Node version: v15.7.0
Radio tested: icecast (azuracast)