@wazo/sdk 0.37.25 → 0.37.29

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 ? 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();
+  });
 });

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

@@ -700,7 +700,7 @@ class Contact {
       this[key] = old[key] || this[key];
     });
 
-    if (old.lineState && !this.lineState) {
+    if (old.lineState) {
       this.lineState = old.lineState;
     }
 

package/lib/domain/__tests__/IndirectTransfer.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 ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,

package/lib/domain/__tests__/Room.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 ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,

package/lib/index.js

@@ -475,7 +475,7 @@ const logger$c = IssueReporter$1 ? IssueReporter$1.loggerFor('api') : console;
 
 // Use a function here to be able to mock it in tests
 const realFetch = () => {
-  if (typeof document !== 'undefined') {
+  if (typeof document !== 'undefined' && window.fetch) {
     // Browser
     return window.fetch;
   }
@@ -1935,7 +1935,7 @@ class Contact {
       this[key] = old[key] || this[key];
     });
 
-    if (old.lineState && !this.lineState) {
+    if (old.lineState) {
       this.lineState = old.lineState;
     }
 
@@ -3260,13 +3260,16 @@ var updateFrom = (instance     , from     ) => {
 
                              
                     
+                     
                             
               
                  
                        
                          
+                       
                                   
                       
+                  
                     
                  
                       
@@ -3283,6 +3286,7 @@ var updateFrom = (instance     , from     ) => {
                          
                      
                            
+                       
   
 
 class CallSession {
@@ -3302,10 +3306,14 @@ class CallSession {
 
                        
 
-                    
+                      
+
+                     // = creationTime
 
                     
 
+                 
+
                     
 
                             
@@ -3341,6 +3349,8 @@ class CallSession {
                      
 
                            
+
+                      
 
   constructor({
     answered,
@@ -3354,6 +3364,8 @@ class CallSession {
     paused,
     ringing,
     startTime,
+    creationTime,
+    endTime,
     cameraEnabled,
     dialedExtension,
     sipCallId,
@@ -3366,12 +3378,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;
@@ -3390,6 +3406,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';
@@ -3420,7 +3438,7 @@ class CallSession {
   }
 
   answer() {
-    this.answered = true;
+    this.answerTime = new Date();
   }
 
   systemAnswer() {
@@ -3576,6 +3594,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 ? new Date() : null;
+  }
+
+  get answered()          {
+    return !!this.answerTime;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -5635,11 +5662,16 @@ class WazoSessionDescriptionHandler extends sessionDescriptionHandler.SessionDes
 
   // $FlowFixMe
   setRemoteSessionDescription(sessionDescription                           )                {
-    const result = super.setRemoteSessionDescription(sessionDescription);
+    try {
+      const result = super.setRemoteSessionDescription(sessionDescription);
 
-    this.eventEmitter.emit('setDescription', sessionDescription);
+      this.eventEmitter.emit('setDescription', sessionDescription);
 
-    return result;
+      return result;
+    } catch (error) {
+      this.logger.error('SessionDescriptionHandler.setRemoteSessionDescription error', error);
+      throw error;
+    }
   }
 
   // Overridden to avoid to fix ice-candidates missing in the SDP in react-native and chrome canary.
@@ -8787,6 +8819,14 @@ var WebSocketClient$1 = /*#__PURE__*/Object.freeze({
   'default': WebSocketClient
 });
 
+/*       */
+class SFUNotAvailableError extends Error {
+  constructor(message         = 'Sorry your user is not configured to support video conference') {
+    super(message);
+    this.name = 'SFUNotAvailableError';
+  }
+}
+
 var COUNTRIES = {
   BELGIUM: 'BE',
   CANADA: 'CA',
@@ -9184,6 +9224,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';
@@ -9228,6 +9269,7 @@ const events = [
   ON_CAMERA_DISABLED,
   ON_CALL_FAILED,
   ON_CALL_ENDED,
+  ON_CALL_REJECTED,
   ON_MESSAGE,
   ON_REINVITE,
   ON_TRACK,
@@ -9584,6 +9626,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);
@@ -9779,6 +9822,7 @@ class WebRTCPhone extends Emitter                  {
       this.removeIncomingSessions(sipSessionId);
     }
 
+    callSession.answerTime = new Date();
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -9836,6 +9880,7 @@ class WebRTCPhone extends Emitter                  {
   _onCallTerminated(sipSession         ) {
     logger$7.info('WebRTC phone - on call terminated', { sipId: sipSession.id });
 
+    const sipSessionId = this.getSipSessionId(sipSession);
     const callSession = this._createCallSession(sipSession);
     const isCurrentSession = this.isCurrentCallSipSession(callSession);
     const isCurrentIncomingCall = callSession.is(this.getIncomingCallSession());
@@ -9853,7 +9898,6 @@ class WebRTCPhone extends Emitter                  {
       }, 5);
     }
 
-    const sipSessionId = this.getSipSessionId(sipSession);
     if (sipSessionId) {
       this.removeIncomingSessions(sipSessionId);
     }
@@ -9994,6 +10038,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     this.shouldSendReinvite = false;
+    callSession.answerTime = new Date();
     this.acceptedSessions[callSession.getId()] = true;
 
     this.eventEmitter.emit(ON_CALL_ANSWERED, callSession);
@@ -10303,6 +10348,7 @@ class WebRTCPhone extends Emitter                  {
       return Promise.resolve(null);
     }
     const callSession = this._createOutgoingCallSession(sipSession, cameraEnabled || false);
+    callSession.creationTime = new Date();
 
     this.eventEmitter.emit(ON_PLAY_PROGRESS_SOUND, this.audioOutputDeviceId, this.audioOutputVolume);
 
@@ -10498,8 +10544,12 @@ class WebRTCPhone extends Emitter                  {
     }
 
     const sessionId = this.incomingSessions[0];
+    const sipSession = this.client.getSipSession(sessionId);
+    if (!sipSession) {
+      return null;
+    }
 
-    return this._createCallSession(this.client.getSipSession(sessionId));
+    return this._createCallSession(sipSession);
   }
 
   sendMessage(sipSession          = null, body        , contentType         = 'text/plain') {
@@ -10577,6 +10627,7 @@ class WebRTCPhone extends Emitter                  {
           callSession);
       }
 
+      callSession.creationTime = new Date();
       this.eventEmitter.emit(ON_CALL_INCOMING, callSession, wantsToDoVideo);
     });
 
@@ -10616,6 +10667,13 @@ class WebRTCPhone extends Emitter                  {
       this.eventEmitter.emit(ON_CALL_ERROR, e);
     });
 
+    this.client.on(this.client.REJECTED, session => {
+      logger$7.info('WebRTC call rejected', session.id);
+      const callSession = this._createCallSession(session);
+      callSession.endTime = new Date();
+      this.eventEmitter.emit(ON_CALL_REJECTED, callSession);
+    });
+
     this.client.on(this.client.UNREGISTERED, () => {
       logger$7.info('WebRTC unregistered');
       this.eventEmitter.emit(ON_UNREGISTERED);
@@ -10828,6 +10886,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,
@@ -10835,6 +10894,8 @@ class WebRTCPhone extends Emitter                  {
       sipStatus: state,
       displayName: identity ? identity.displayName || number : number,
       startTime: fromSession ? fromSession.startTime : new Date(),
+      creationTime: fromSession ? fromSession.creationTime : null,
+      endTime: fromSession ? fromSession.endTime : null,
       answered: state === sessionState.SessionState.Established,
       paused: this.client.isCallHeld(sipSession),
       isCaller: 'incoming' in extra ? !extra.incoming : false,
@@ -10845,6 +10906,7 @@ class WebRTCPhone extends Emitter                  {
       videoMuted: fromSession ? fromSession.isVideoMuted() : false,
       recording: fromSession ? fromSession.isRecording() : false,
       recordingPaused: false, // @TODO
+      sipSession,
       ...extra,
     });
 
@@ -10873,6 +10935,7 @@ var PHONE_EVENTS = /*#__PURE__*/Object.freeze({
   ON_CAMERA_RESUMED: ON_CAMERA_RESUMED,
   ON_CALL_CANCELED: ON_CALL_CANCELED,
   ON_CALL_FAILED: ON_CALL_FAILED,
+  ON_CALL_REJECTED: ON_CALL_REJECTED,
   ON_CALL_ENDED: ON_CALL_ENDED,
   ON_CALL_ENDING: ON_CALL_ENDING,
   ON_MESSAGE: ON_MESSAGE,
@@ -12321,7 +12384,7 @@ class Phone extends Emitter {
 
   checkSfu() {
     if (!this.hasSfu()) {
-      throw new Error('Sorry your user is not configured to support video conference');
+      throw new SFUNotAvailableError();
     }
   }
 
@@ -13882,6 +13945,7 @@ const Wazo = {
   InvalidSubscription,
   InvalidAuthorization,
   CanceledCallError,
+  SFUNotAvailableError,
 };
 
 if (window) {
@@ -13924,6 +13988,7 @@ var index = {
   WazoWebSocketClient: WebSocketClient,
   BadResponse,
   ServerError,
+  SFUNotAvailableError,
   Call,
   CallSession,
   CTIPhone,

package/lib/lib/WazoSessionDescriptionHandler.js

@@ -25,7 +25,7 @@ IssueReporter$1 ? IssueReporter$1.loggerFor('api') : console;
 
 // Use a function here to be able to mock it in tests
 const realFetch = () => {
-  if (typeof document !== 'undefined') {
+  if (typeof document !== 'undefined' && window.fetch) {
     // Browser
     return window.fetch;
   }
@@ -497,11 +497,16 @@ class WazoSessionDescriptionHandler extends sessionDescriptionHandler.SessionDes
 
   // $FlowFixMe
   setRemoteSessionDescription(sessionDescription                           )                {
-    const result = super.setRemoteSessionDescription(sessionDescription);
+    try {
+      const result = super.setRemoteSessionDescription(sessionDescription);
 
-    this.eventEmitter.emit('setDescription', sessionDescription);
+      this.eventEmitter.emit('setDescription', sessionDescription);
 
-    return result;
+      return result;
+    } catch (error) {
+      this.logger.error('SessionDescriptionHandler.setRemoteSessionDescription error', error);
+      throw error;
+    }
   }
 
   // Overridden to avoid to fix ice-candidates missing in the SDP in react-native and chrome canary.

package/lib/service/CallApi.js

@@ -1416,7 +1416,7 @@ class Contact {
       this[key] = old[key] || this[key];
     });
 
-    if (old.lineState && !this.lineState) {
+    if (old.lineState) {
       this.lineState = old.lineState;
     }
 
@@ -2472,7 +2472,7 @@ const logger$1 = IssueReporter$1 ? IssueReporter$1.loggerFor('api') : console;
 
 // Use a function here to be able to mock it in tests
 const realFetch = () => {
-  if (typeof document !== 'undefined') {
+  if (typeof document !== 'undefined' && window.fetch) {
     // Browser
     return window.fetch;
   }