node-datachannel 0.4.0 → 0.4.2

package/README.md

@@ -31,6 +31,7 @@ npm install node-datachannel
 | Node V16 |     +     |      +      |      +           |      +      |      +      |       +        |
 | Node V17 |     +     |      +      |      +           |      +      |      +      |       +        |
 | Node V18 |     +     |      +      |      +           |      +      |      +      |       +        |
+| Node V19 |     +     |      +      |      +           |      +      |      +      |       +        |
 
 1) Please note that; For Linux-arm64 platform we need OpenSSL to be installed locally.
 

package/lib/index.d.ts

@@ -17,7 +17,7 @@ export interface SctpSettings {
 
 // Functions
 export function preload(): void;
-export function initLogger(level: LogLevel): void;
+export function initLogger(level: LogLevel, callback?: (level: LogLevel, message: string) => void): void;
 export function cleanup(): void;
 export function setSctpSettings(settings: SctpSettings): void;
 
@@ -46,9 +46,11 @@ export interface IceServer {
 }
 
 export type TransportPolicy = 'all' | 'relay';
+
 export interface RtcConfig {
     iceServers: (string | IceServer)[];
     proxyServer?: ProxyServer;
+    bindAddress?: string;
     enableIceTcp?: boolean;
     enableIceUdpMux?: boolean;
     disableAutoNegotiation?: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-datachannel",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "description": "libdatachannel node bindings",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",
@@ -10,7 +10,7 @@
   },
   "_from": "node-datachannel@latest",
   "scripts": {
-    "install": "prebuild-install || (npm install --ignore-scripts && npm run _prebuild)",
+    "install": "prebuild-install || (npm install --ignore-scripts --production=false && npm run _prebuild)",
     "install-nice": "npm run clean && cmake-js build --CDUSE_NICE=1",
     "build": "cmake-js build",
     "build:debug": "cmake-js build -D",
@@ -41,7 +41,7 @@
       "url": "https://github.com/paullouisageneau"
     }
   ],
-  "license": "LGPL",
+  "license": "MPL 2.0",
   "gypfile": true,
   "bugs": {
     "url": "https://github.com/murat-dogan/node-datachannel/issues"

package/src/data-channel-wrapper.cpp

@@ -103,7 +103,7 @@ Napi::Value DataChannelWrapper::getId(const Napi::CallbackInfo &info)
         return info.Env().Null();
     }
 
-    return Napi::Number::New(info.Env(), mDataChannelPtr->id());
+    return Napi::Number::New(info.Env(), mDataChannelPtr->id().value_or(-1));
 }
 
 Napi::Value DataChannelWrapper::getProtocol(const Napi::CallbackInfo &info)

package/src/peer-connection-wrapper.cpp

@@ -171,6 +171,10 @@ PeerConnectionWrapper::PeerConnectionWrapper(const Napi::CallbackInfo &info) : N
         rtcConfig.proxyServer = rtc::ProxyServer(type, ip, port, username, password);
     }
 
+    // bind address, libjuice only
+    if(config.Get("bindAddress").IsString())
+        rtcConfig.bindAddress = config.Get("bindAddress").As<Napi::String>().ToString();
+
     // Port Ranges
     if (config.Get("portRangeBegin").IsNumber())
         rtcConfig.portRangeBegin = config.Get("portRangeBegin").As<Napi::Number>().Uint32Value();
@@ -265,7 +269,7 @@ void PeerConnectionWrapper::doDestroy()
 }
 
 void PeerConnectionWrapper::doResetCallbacks()
-{  
+{
     mOnLocalDescriptionCallback.reset();
     mOnLocalCandidateCallback.reset();
     mOnStateChangeCallback.reset();
@@ -513,10 +517,10 @@ Napi::Value PeerConnectionWrapper::createDataChannel(const Napi::CallbackInfo &i
                 switch (reliability.Get("type").As<Napi::Number>().Uint32Value())
                 {
                 case 1:
-                    init.reliability.rexmit = reliability.Get("rexmit").As<Napi::Number>();
+                    init.reliability.rexmit = static_cast<int>(reliability.Get("rexmit").As<Napi::Number>().ToNumber());
                     break;
                 case 2:
-                    init.reliability.rexmit = std::chrono::milliseconds(reliability.Get("rexmit").As<Napi::Number>());
+                    init.reliability.rexmit = std::chrono::milliseconds(reliability.Get("rexmit").As<Napi::Number>().Uint32Value());
                     break;
                 }
             }
@@ -661,7 +665,7 @@ void PeerConnectionWrapper::onStateChange(const Napi::CallbackInfo &info)
 
     mRtcPeerConnPtr->onStateChange([&](rtc::PeerConnection::State state) {
         if (mOnStateChangeCallback)
-            mOnStateChangeCallback->call([this, state](Napi::Env env, std::vector<napi_value> &args) {                
+            mOnStateChangeCallback->call([this, state](Napi::Env env, std::vector<napi_value> &args) {
                 if(instances.find(this) == instances.end())
                     throw ThreadSafeCallback::CancelException();
 

package/src/rtc-wrapper.cpp

@@ -61,7 +61,42 @@ void RtcWrapper::initLogger(const Napi::CallbackInfo &info)
 
     try
     {
-        rtc::InitLogger(logLevel);
+        if (length < 2)
+        {
+            rtc::InitLogger(logLevel);
+        }
+        else
+        {
+            if (!info[1].IsFunction())
+            {
+                Napi::TypeError::New(env, "Function expected").ThrowAsJavaScriptException();
+                return;
+            }
+            logCallback = std::make_unique<ThreadSafeCallback>(info[1].As<Napi::Function>());
+            rtc::InitLogger(logLevel, [&](rtc::LogLevel level, std::string message) {
+                if (logCallback)
+                    logCallback->call([level, message = std::move(message)](Napi::Env env, std::vector<napi_value> &args) {
+                        // This will run in main thread and needs to construct the
+                        // arguments for the call
+
+                        std::string logLevel;
+                        if (level == rtc::LogLevel::Verbose)
+                            logLevel = "Verbose";
+                        if (level == rtc::LogLevel::Debug)
+                            logLevel = "Debug";
+                        if (level == rtc::LogLevel::Info)
+                            logLevel = "Info";
+                        if (level == rtc::LogLevel::Warning)
+                            logLevel = "Warning";
+                        if (level == rtc::LogLevel::Error)
+                            logLevel = "Error";
+                        if (level == rtc::LogLevel::Fatal)
+                            logLevel = "Fatal";
+                        args = {Napi::String::New(env, logLevel), Napi::String::New(env, message)};
+                    });
+            });
+        }
+
     }
     catch (std::exception &ex)
     {
@@ -85,6 +120,8 @@ void RtcWrapper::cleanup(const Napi::CallbackInfo &info)
         
         // Clear Callbacks    
         PeerConnectionWrapper::ResetCallbacksAll();
+
+        if (logCallback) logCallback.reset();
     }        
     catch (std::exception &ex)
     {

package/src/rtc-wrapper.h

@@ -8,6 +8,8 @@
 
 #include "rtc/rtc.hpp"
 
+#include "thread-safe-callback.h"
+
 class RtcWrapper
 {
 public:
@@ -16,6 +18,8 @@ public:
     static void initLogger(const Napi::CallbackInfo &info);
     static void cleanup(const Napi::CallbackInfo &info);
     static void setSctpSettings(const Napi::CallbackInfo &info);
+private:
+    static inline std::unique_ptr<ThreadSafeCallback> logCallback = nullptr;
 };
 
 #endif // RTC_WRAPPER_H

package/test/connectivity.js

@@ -11,6 +11,7 @@ let dc2 = null;
 // export interface RtcConfig {
 //     iceServers: string[];
 //     proxyServer?: ProxyServer;
+//     bindAddress?: string;
 //     enableIceTcp?: boolean;
 //     portRangeBegin?: number;
 //     portRangeEnd?: number;