You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for taking a look at this. I know that #349, #415, #403, and #310 are all this same thing, but they were also all closed a number of years ago. Now, with great libraries like node-webrtc, which, according to its README, "is a Node.js Native Addon that provides bindings to WebRTC", it is now possible to use WebRTC in Node.
However, it is still not supported by this library. I would really appreciate this feature.
My suggestion for how to implement this is to use a new parameter in JsSIP.UA() called "wrtc" [or something similar], as simple-peer already does. This parameter would be optional, and if left undefined, would default to the "window.*" APIs.
I'm still unsure if solving #427 would solve this problem.
The text was updated successfully, but these errors were encountered:
We won't do that because if we did that we would automatically become responsible of how such a node-webrtc library (that BTW we don't know) behaves, and from my past experience it does not behave well at all.
So instead of that, we are gonna remove WebRTC entirely from JsSIP and make the app responsible of it.
BTW please use the mailing list for questions or suggestions.
Thanks for taking a look at this. I know that #349, #415, #403, and #310 are all this same thing, but they were also all closed a number of years ago. Now, with great libraries like node-webrtc, which, according to its README, "is a Node.js Native Addon that provides bindings to WebRTC", it is now possible to use WebRTC in Node.
However, it is still not supported by this library. I would really appreciate this feature.
My suggestion for how to implement this is to use a new parameter in JsSIP.UA() called "wrtc" [or something similar], as simple-peer already does. This parameter would be optional, and if left undefined, would default to the "window.*" APIs.
I'm still unsure if solving #427 would solve this problem.
The text was updated successfully, but these errors were encountered: