@wazo/sdk 0.37.27 → 0.37.31

package/lib/checker/checks/webrtc-transport.js

@@ -5353,6 +5353,7 @@ const MAX_REGISTER_TRIES = 10;
                       
                
                 
+                         
                              
                     
               
@@ -6969,6 +6970,13 @@ class WebRTCClient extends Emitter {
   }
 
   _createWebRTCConfiguration(configOverrides         = {}) {
+    let { host } = this.config;
+    let port = this.config.port || 443;
+
+    if (this.config.websocketSip) {
+      [host, port = 443] = this.config.websocketSip.split(':');
+    }
+
     const config         = {
       authorizationUsername: this.config.authorizationUser,
       authorizationPassword: this.config.password,
@@ -7004,7 +7012,7 @@ class WebRTCClient extends Emitter {
       },
       transportOptions: {
         traceSip: configOverrides.traceSip || false,
-        wsServers: `wss://${this.config.host}:${this.config.port || 443}/api/asterisk/ws`,
+        wsServers: `wss://${host}:${port}/api/asterisk/ws`,
       },
       sessionDescriptionHandlerFactoryOptions: {
         modifiers: [replaceLocalIpModifier],

package/lib/checker/checks/webrtc.js

@@ -5353,6 +5353,7 @@ const MAX_REGISTER_TRIES = 10;
                       
                
                 
+                         
                              
                     
               
@@ -6969,6 +6970,13 @@ class WebRTCClient extends Emitter {
   }
 
   _createWebRTCConfiguration(configOverrides         = {}) {
+    let { host } = this.config;
+    let port = this.config.port || 443;
+
+    if (this.config.websocketSip) {
+      [host, port = 443] = this.config.websocketSip.split(':');
+    }
+
     const config         = {
       authorizationUsername: this.config.authorizationUser,
       authorizationPassword: this.config.password,
@@ -7004,7 +7012,7 @@ class WebRTCClient extends Emitter {
       },
       transportOptions: {
         traceSip: configOverrides.traceSip || false,
-        wsServers: `wss://${this.config.host}:${this.config.port || 443}/api/asterisk/ws`,
+        wsServers: `wss://${host}:${port}/api/asterisk/ws`,
       },
       sessionDescriptionHandlerFactoryOptions: {
         modifiers: [replaceLocalIpModifier],

package/lib/domain/CallSession.js

@@ -34,13 +34,16 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
                        
                          
+                       
                                   
                       
+                  
                     
                  
                       
@@ -57,6 +60,7 @@ var updateFrom = (instance     , from     ) => {
                          
                      
                            
+                       
   
 
 class CallSession {
@@ -76,10 +80,14 @@ class CallSession {
 
                        
 
-                    
+                      
+
+                     // = creationTime
 
                     
 
+                 
+
                     
 
                             
@@ -115,6 +123,8 @@ class CallSession {
                      
 
                            
+
+                      
 
   constructor({
     answered,
@@ -128,6 +138,8 @@ class CallSession {
     paused,
     ringing,
     startTime,
+    creationTime,
+    endTime,
     cameraEnabled,
     dialedExtension,
     sipCallId,
@@ -140,12 +152,16 @@ class CallSession {
     recording,
     recordingPaused,
     videoRemotelyDowngraded,
+    sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
     this.displayName = displayName;
     this.number = number;
+    this.creationTime = creationTime;
     this.startTime = startTime;
+    this.endTime = endTime;
     this.isCaller = isCaller;
     this.answered = answered;
     this.answeredBySystem = answeredBySystem;
@@ -164,6 +180,8 @@ class CallSession {
     this.recording = recording || false;
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
+    this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -194,7 +212,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -350,6 +368,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? this.answerTime || new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,

package/lib/domain/Phone/CTIPhone.js

@@ -1715,13 +1715,16 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
                        
                          
+                       
                                   
                       
+                  
                     
                  
                       
@@ -1738,6 +1741,7 @@ var updateFrom = (instance     , from     ) => {
                          
                      
                            
+                       
   
 
 class CallSession {
@@ -1757,10 +1761,14 @@ class CallSession {
 
                        
 
-                    
+                      
+
+                     // = creationTime
 
                     
 
+                 
+
                     
 
                             
@@ -1796,6 +1804,8 @@ class CallSession {
                      
 
                            
+
+                      
 
   constructor({
     answered,
@@ -1809,6 +1819,8 @@ class CallSession {
     paused,
     ringing,
     startTime,
+    creationTime,
+    endTime,
     cameraEnabled,
     dialedExtension,
     sipCallId,
@@ -1821,12 +1833,16 @@ class CallSession {
     recording,
     recordingPaused,
     videoRemotelyDowngraded,
+    sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
     this.displayName = displayName;
     this.number = number;
+    this.creationTime = creationTime;
     this.startTime = startTime;
+    this.endTime = endTime;
     this.isCaller = isCaller;
     this.answered = answered;
     this.answeredBySystem = answeredBySystem;
@@ -1845,6 +1861,8 @@ class CallSession {
     this.recording = recording || false;
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
+    this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -1875,7 +1893,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -2031,6 +2049,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? this.answerTime || new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,

package/lib/domain/Phone/WebRTCPhone.js

@@ -63,13 +63,16 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
                        
                          
+                       
                                   
                       
+                  
                     
                  
                       
@@ -86,6 +89,7 @@ var updateFrom = (instance     , from     ) => {
                          
                      
                            
+                       
   
 
 class CallSession {
@@ -105,10 +109,14 @@ class CallSession {
 
                        
 
-                    
+                      
+
+                     // = creationTime
 
                     
 
+                 
+
                     
 
                             
@@ -144,6 +152,8 @@ class CallSession {
                      
 
                            
+
+                      
 
   constructor({
     answered,
@@ -157,6 +167,8 @@ class CallSession {
     paused,
     ringing,
     startTime,
+    creationTime,
+    endTime,
     cameraEnabled,
     dialedExtension,
     sipCallId,
@@ -169,12 +181,16 @@ class CallSession {
     recording,
     recordingPaused,
     videoRemotelyDowngraded,
+    sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
     this.displayName = displayName;
     this.number = number;
+    this.creationTime = creationTime;
     this.startTime = startTime;
+    this.endTime = endTime;
     this.isCaller = isCaller;
     this.answered = answered;
     this.answeredBySystem = answeredBySystem;
@@ -193,6 +209,8 @@ class CallSession {
     this.recording = recording || false;
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
+    this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -223,7 +241,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -379,6 +397,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? this.answerTime || new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -887,6 +914,7 @@ const ON_CAMERA_DISABLED = 'onCameraDisabled';
 const ON_CAMERA_RESUMED = 'onCameraResumed';
 const ON_CALL_CANCELED = 'onCallCanceled';
 const ON_CALL_FAILED = 'onCallFailed';
+const ON_CALL_REJECTED = 'onCallRejected';
 const ON_CALL_ENDED = 'onCallEnded';
 const ON_CALL_ENDING = 'onCallEnding';
 const ON_MESSAGE = 'onMessage';
@@ -931,6 +959,7 @@ const events = [
   ON_CAMERA_DISABLED,
   ON_CALL_FAILED,
   ON_CALL_ENDED,
+  ON_CALL_REJECTED,
   ON_MESSAGE,
   ON_REINVITE,
   ON_TRACK,
@@ -1287,6 +1316,7 @@ class WebRTCPhone extends Emitter                  {
 
           // Should be called before `_onCallTerminated` or the callCount will not decrement...
           const callSession = this._createCallSession(sipSession);
+          callSession.endTime = new Date();
           const wasCurrentSession = this._onCallTerminated(sipSession);
 
           return this.eventEmitter.emit(ON_CALL_ENDED, callSession, wasCurrentSession);
@@ -1482,6 +1512,8 @@ class WebRTCPhone extends Emitter                  {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
+    this._updateCallSession(callSession);
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -1697,6 +1729,8 @@ class WebRTCPhone extends Emitter                  {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
+    this._updateCallSession(callSession);
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);
@@ -1796,7 +1830,7 @@ class WebRTCPhone extends Emitter                  {
     // Hold current session if different from the current one (we don't want 2 sessions active at the same time).
     if (this.currentSipSession && this.currentSipSession.id !== sipSession.id) {
       logger.info('WebRTC hold call after resume', { id: this.currentSipSession.id });
-      this.holdSipSession(this.currentSipSession, callSession);
+      this.holdSipSession(this.currentSipSession, this.currentCallSession);
     }
 
     const promise = this.unholdSipSession(sipSession, callSession, withEvent, isConference);
@@ -2006,6 +2040,8 @@ class WebRTCPhone extends Emitter                  {
       return Promise.resolve(null);
     }
     const callSession = this._createOutgoingCallSession(sipSession, cameraEnabled || false);
+    callSession.creationTime = new Date();
+    this._updateCallSession(callSession);
 
     this.eventEmitter.emit(ON_PLAY_PROGRESS_SOUND, this.audioOutputDeviceId, this.audioOutputVolume);
 
@@ -2284,6 +2320,9 @@ class WebRTCPhone extends Emitter                  {
           callSession);
       }
 
+      callSession.creationTime = new Date();
+      this._updateCallSession(callSession);
+
       this.eventEmitter.emit(ON_CALL_INCOMING, callSession, wantsToDoVideo);
     });
 
@@ -2323,6 +2362,15 @@ class WebRTCPhone extends Emitter                  {
       this.eventEmitter.emit(ON_CALL_ERROR, e);
     });
 
+    this.client.on(this.client.REJECTED, session => {
+      logger.info('WebRTC call rejected', session.id);
+      const callSession = this._createCallSession(session);
+      callSession.endTime = new Date();
+      this._updateCallSession(callSession);
+
+      this.eventEmitter.emit(ON_CALL_REJECTED, callSession);
+    });
+
     this.client.on(this.client.UNREGISTERED, () => {
       logger.info('WebRTC unregistered');
       this.eventEmitter.emit(ON_UNREGISTERED);
@@ -2535,6 +2583,7 @@ class WebRTCPhone extends Emitter                  {
     const number = identity ? identity.uri._normal.user : null;
     const { state } = sipSession || {};
     const sessionId = this.getSipSessionId(sipSession);
+    fromSession = fromSession || this.callSessions[sessionId];
 
     const callSession = new CallSession({
       callId: fromSession && fromSession.callId,
@@ -2542,6 +2591,9 @@ class WebRTCPhone extends Emitter                  {
       sipStatus: state,
       displayName: identity ? identity.displayName || number : number,
       startTime: fromSession ? fromSession.startTime : new Date(),
+      creationTime: fromSession ? fromSession.creationTime : null,
+      answerTime: fromSession ? fromSession.answerTime : null,
+      endTime: fromSession ? fromSession.endTime : null,
       answered: state === sessionState.SessionState.Established,
       paused: this.client.isCallHeld(sipSession),
       isCaller: 'incoming' in extra ? !extra.incoming : false,
@@ -2552,6 +2604,7 @@ class WebRTCPhone extends Emitter                  {
       videoMuted: fromSession ? fromSession.isVideoMuted() : false,
       recording: fromSession ? fromSession.isRecording() : false,
       recordingPaused: false, // @TODO
+      sipSession,
       ...extra,
     });
 
@@ -2575,6 +2628,7 @@ exports.ON_CALL_HELD = ON_CALL_HELD;
 exports.ON_CALL_INCOMING = ON_CALL_INCOMING;
 exports.ON_CALL_MUTED = ON_CALL_MUTED;
 exports.ON_CALL_OUTGOING = ON_CALL_OUTGOING;
+exports.ON_CALL_REJECTED = ON_CALL_REJECTED;
 exports.ON_CALL_RESUMED = ON_CALL_RESUMED;
 exports.ON_CALL_UNHELD = ON_CALL_UNHELD;
 exports.ON_CALL_UNMUTED = ON_CALL_UNMUTED;

package/lib/domain/SFUNotAvailableError.js

@@ -0,0 +1,11 @@
+'use strict';
+
+/*       */
+class SFUNotAvailableError extends Error {
+  constructor(message         = 'Sorry your user is not configured to support video conference') {
+    super(message);
+    this.name = 'SFUNotAvailableError';
+  }
+}
+
+module.exports = SFUNotAvailableError;

package/lib/domain/__tests__/CallSession.test.js

@@ -34,13 +34,16 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
                        
                          
+                       
                                   
                       
+                  
                     
                  
                       
@@ -57,6 +60,7 @@ var updateFrom = (instance     , from     ) => {
                          
                      
                            
+                       
   
 
 class CallSession {
@@ -76,10 +80,14 @@ class CallSession {
 
                        
 
-                    
+                      
+
+                     // = creationTime
 
                     
 
+                 
+
                     
 
                             
@@ -115,6 +123,8 @@ class CallSession {
                      
 
                            
+
+                      
 
   constructor({
     answered,
@@ -128,6 +138,8 @@ class CallSession {
     paused,
     ringing,
     startTime,
+    creationTime,
+    endTime,
     cameraEnabled,
     dialedExtension,
     sipCallId,
@@ -140,12 +152,16 @@ class CallSession {
     recording,
     recordingPaused,
     videoRemotelyDowngraded,
+    sipSession,
+    answerTime,
   }                      ) {
     this.callId = callId;
     this.sipCallId = sipCallId;
     this.displayName = displayName;
     this.number = number;
+    this.creationTime = creationTime;
     this.startTime = startTime;
+    this.endTime = endTime;
     this.isCaller = isCaller;
     this.answered = answered;
     this.answeredBySystem = answeredBySystem;
@@ -164,6 +180,8 @@ class CallSession {
     this.recording = recording || false;
     this.recordingPaused = recordingPaused || false;
     this.videoRemotelyDowngraded = videoRemotelyDowngraded;
+    this.sipSession = sipSession;
+    this.answerTime = answerTime || this.answerTime;
 
     // Useful to compare instead of instanceof with minified code
     this.type = 'CallSession';
@@ -194,7 +212,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -350,6 +368,15 @@ class CallSession {
     return newFrom(callSession, CallSession);
   }
 
+  // Retro-compatibility: `answered` was a boolean before. We can reproduce the behaviour with a getter/setter
+  set answered(value         ) {
+    this.answerTime = value ? this.answerTime || new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -412,4 +439,21 @@ describe('CallSession domain', () => {
     expect(cs1.is(cs2)).toBeTruthy();
     expect(cs2.is(cs1)).toBeTruthy();
   });
+
+  it('should set answerTime when setting answered', () => {
+    const cs1 = new CallSession({});
+    cs1.answered = true;
+
+    const cs2 = new CallSession({});
+    cs2.answer();
+
+    const cs3 = new CallSession({ answered: true });
+
+    expect(cs1.answered).toBeTruthy();
+    expect(cs1.answerTime).not.toBeNull();
+    expect(cs2.answered).toBeTruthy();
+    expect(cs2.answerTime).not.toBeNull();
+    expect(cs3.answered).toBeTruthy();
+    expect(cs3.answerTime).not.toBeNull();
+  });
 });