@wazo/sdk 0.37.30 → 0.37.34

package/lib/domain/Features.js

@@ -4024,6 +4024,34 @@ var chatdMethods = (client              , baseUrl        ) => ({
     client.get(`${baseUrl}/users/me/rooms/messages`, options),
 });
 
+//      
+
+                                     
+               
+  
+
+class MeetingStatus {
+               
+
+                
+
+  constructor({
+    full,
+  }         = {}) {
+    this.full = full;
+
+    // Useful to compare instead of instanceof with minified code
+    this.type = 'MeetingStatus';
+  }
+
+  static parse(plain                       )                {
+    return new MeetingStatus({
+      full: plain.full,
+    });
+  }
+
+}
+
 /*       */
 
                   
@@ -4218,6 +4246,8 @@ var calldMethods = (client              , baseUrl        ) => ({
     client.put(`${baseUrl}/users/me/calls/${callId}/record/resume`, null, null,
       ApiRequester.successResponseParser),
 
+  guestGetMeetingStatus: (meetingUuid        )                         =>
+    client.get(`${baseUrl}/guests/me/meetings/${meetingUuid}/status`).then(MeetingStatus.parse),
 });
 
 //      

package/lib/domain/MeetingStatus.js

@@ -0,0 +1,31 @@
+'use strict';
+
+//      
+
+                                     
+               
+  
+
+class MeetingStatus {
+               
+
+                
+
+  constructor({
+    full,
+  }         = {}) {
+    this.full = full;
+
+    // Useful to compare instead of instanceof with minified code
+    this.type = 'MeetingStatus';
+  }
+
+  static parse(plain                       )                {
+    return new MeetingStatus({
+      full: plain.full,
+    });
+  }
+
+}
+
+module.exports = MeetingStatus;

package/lib/domain/Phone/CTIPhone.js

@@ -2051,13 +2051,21 @@ class 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;
+    this.answerTime = value ? this.answerTime || new Date() : null;
   }
 
   get answered()          {
     return !!this.answerTime;
   }
 
+  toJSON() {
+    const jsonObj = { ...this };
+    // $FlowFixMe
+    jsonObj.answered = this.answered;
+
+    return jsonObj;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -4416,6 +4424,34 @@ class IndirectTransfer {
   }
 }
 
+//      
+
+                                     
+               
+  
+
+class MeetingStatus {
+               
+
+                
+
+  constructor({
+    full,
+  }         = {}) {
+    this.full = full;
+
+    // Useful to compare instead of instanceof with minified code
+    this.type = 'MeetingStatus';
+  }
+
+  static parse(plain                       )                {
+    return new MeetingStatus({
+      full: plain.full,
+    });
+  }
+
+}
+
 /*       */
 
                   
@@ -4610,6 +4646,8 @@ var calldMethods = (client              , baseUrl        ) => ({
     client.put(`${baseUrl}/users/me/calls/${callId}/record/resume`, null, null,
       ApiRequester.successResponseParser),
 
+  guestGetMeetingStatus: (meetingUuid        )                         =>
+    client.get(`${baseUrl}/guests/me/meetings/${meetingUuid}/status`).then(MeetingStatus.parse),
 });
 
 //      

package/lib/domain/Phone/WebRTCPhone.js

@@ -399,13 +399,21 @@ class 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;
+    this.answerTime = value ? this.answerTime || new Date() : null;
   }
 
   get answered()          {
     return !!this.answerTime;
   }
 
+  toJSON() {
+    const jsonObj = { ...this };
+    // $FlowFixMe
+    jsonObj.answered = this.answered;
+
+    return jsonObj;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -1513,6 +1521,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     callSession.answerTime = new Date();
+    this._updateCallSession(callSession);
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -1722,6 +1731,12 @@ class WebRTCPhone extends Emitter                  {
       return Promise.resolve(null);
     }
 
+    const sipSessionId = this.getSipSessionIdFromCallSession(callSession);
+    if (sipSessionId && !(sipSessionId in this.callSessions)) {
+      logger.warn('Call session already ended or not found, can\'t accept it.', { sipSessionId });
+      return Promise.resolve(null);
+    }
+
     if (callSession.getId() in this.acceptedSessions) {
       logger.warn('CallSession already accepted.', { id: callSession ? callSession.getId() : 'n/a' });
       return Promise.resolve(callSession.sipCallId);
@@ -1729,6 +1744,7 @@ 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);
@@ -1828,7 +1844,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);
@@ -2039,6 +2055,7 @@ class WebRTCPhone extends Emitter                  {
     }
     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);
 
@@ -2180,7 +2197,7 @@ class WebRTCPhone extends Emitter                  {
     return this.client && this.client.isConference(callSession.sipCallId);
   }
 
-  async close()                {
+  async close(force          = false)                {
     logger.info('WebRTC close');
 
     try {
@@ -2192,7 +2209,7 @@ class WebRTCPhone extends Emitter                  {
       logger.error('WebRTC close, unregister error', e);
     }
 
-    this.client.close();
+    this.client.close(force);
     this.unbind();
 
     this.incomingSessions = [];
@@ -2318,6 +2335,8 @@ class WebRTCPhone extends Emitter                  {
       }
 
       callSession.creationTime = new Date();
+      this._updateCallSession(callSession);
+
       this.eventEmitter.emit(ON_CALL_INCOMING, callSession, wantsToDoVideo);
     });
 
@@ -2342,9 +2361,17 @@ class WebRTCPhone extends Emitter                  {
     });
 
     this.client.on(this.client.ACCEPTED, (sipSession         ) => {
-      logger.info('WebRTC call accepted', { sipId: sipSession.id });
+      const sessionId = this.getSipSessionId(sipSession);
+      const hasSession = sessionId in this.callSessions;
 
-      this._onCallAccepted(sipSession, this.client.hasVideo(this.getSipSessionId(sipSession)));
+      logger.info('WebRTC call accepted', { sipId: sessionId, hasSession });
+
+      if (!hasSession) {
+        logger.warn('Call accepted ignored, session is no longer present in the WebRtcPhone', { sessionId });
+        return;
+      }
+
+      this._onCallAccepted(sipSession, this.client.hasVideo(sessionId));
 
       if (this.audioOutputDeviceId) {
         this.client.changeAudioOutputDevice(this.audioOutputDeviceId);
@@ -2361,6 +2388,8 @@ class WebRTCPhone extends Emitter                  {
       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);
     });
 
@@ -2478,9 +2507,14 @@ class WebRTCPhone extends Emitter                  {
     return this.client.getSipSessionId(sipSession);
   }
 
-  _updateCallSession(callSession             ) {
+  getSipSessionIdFromCallSession(callSession             ) {
     const sipSession = this.findSipSession(callSession);
-    const sipSessionId = sipSession ? this.getSipSessionId(sipSession) : null;
+
+    return sipSession ? this.getSipSessionId(sipSession) : null;
+  }
+
+  _updateCallSession(callSession             ) {
+    const sipSessionId = this.getSipSessionIdFromCallSession(callSession);
 
     if (sipSessionId) {
       this.callSessions[sipSessionId] = callSession;
@@ -2585,6 +2619,7 @@ class WebRTCPhone extends Emitter                  {
       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),

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

@@ -370,13 +370,21 @@ class 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;
+    this.answerTime = value ? this.answerTime || new Date() : null;
   }
 
   get answered()          {
     return !!this.answerTime;
   }
 
+  toJSON() {
+    const jsonObj = { ...this };
+    // $FlowFixMe
+    jsonObj.answered = this.answered;
+
+    return jsonObj;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -402,6 +410,8 @@ class CallSession {
   }
 }
 
+const stringify = cs => JSON.parse(JSON.stringify(cs));
+
 describe('CallSession domain', () => {
   it('should update from another CallSession without data loss', () => {
     const callSession = new CallSession({
@@ -440,7 +450,7 @@ describe('CallSession domain', () => {
     expect(cs2.is(cs1)).toBeTruthy();
   });
 
-  it('should set answerTime when setting answered', () => {
+  it('should set answerTime when setting answered to true', () => {
     const cs1 = new CallSession({});
     cs1.answered = true;
 
@@ -449,11 +459,37 @@ describe('CallSession domain', () => {
 
     const cs3 = new CallSession({ answered: true });
 
-    expect(cs1.answered).toBeTruthy();
+    expect(cs1.answered).toEqual(true);
     expect(cs1.answerTime).not.toBeNull();
-    expect(cs2.answered).toBeTruthy();
+    expect(cs2.answered).toEqual(true);
     expect(cs2.answerTime).not.toBeNull();
-    expect(cs3.answered).toBeTruthy();
+    expect(cs3.answered).toEqual(true);
     expect(cs3.answerTime).not.toBeNull();
+
+    const cs4 = stringify(cs1);
+    const cs5 = stringify(cs2);
+    const cs6 = stringify(cs3);
+
+    expect(cs4.answered).toEqual(true);
+    expect(cs5.answered).toEqual(true);
+    expect(cs6.answered).toEqual(true);
+  });
+
+  it('should reset answerTime when setting answered to false', () => {
+    const cs1 = new CallSession({});
+    cs1.answered = false;
+
+    const cs2 = new CallSession({ answered: false });
+
+    expect(cs1.answered).toEqual(false);
+    expect(cs1.answerTime).toBeNull();
+    expect(cs2.answered).toEqual(false);
+    expect(cs2.answerTime).toBeNull();
+
+    const cs4 = stringify(cs1);
+    const cs5 = stringify(cs2);
+
+    expect(cs4.answered).toEqual(false);
+    expect(cs5.answered).toEqual(false);
   });
 });

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

@@ -370,13 +370,21 @@ class 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;
+    this.answerTime = value ? this.answerTime || new Date() : null;
   }
 
   get answered()          {
     return !!this.answerTime;
   }
 
+  toJSON() {
+    const jsonObj = { ...this };
+    // $FlowFixMe
+    jsonObj.answered = this.answered;
+
+    return jsonObj;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,

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

@@ -370,13 +370,21 @@ class 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;
+    this.answerTime = value ? this.answerTime || new Date() : null;
   }
 
   get answered()          {
     return !!this.answerTime;
   }
 
+  toJSON() {
+    const jsonObj = { ...this };
+    // $FlowFixMe
+    jsonObj.answered = this.answered;
+
+    return jsonObj;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,

package/lib/index.js

@@ -3596,13 +3596,21 @@ class 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;
+    this.answerTime = value ? this.answerTime || new Date() : null;
   }
 
   get answered()          {
     return !!this.answerTime;
   }
 
+  toJSON() {
+    const jsonObj = { ...this };
+    // $FlowFixMe
+    jsonObj.answered = this.answered;
+
+    return jsonObj;
+  }
+
   static parseCall(call      )              {
     return new CallSession({
       callId: call.id,
@@ -4937,6 +4945,34 @@ var chatdMethods = (client              , baseUrl        ) => ({
     client.get(`${baseUrl}/users/me/rooms/messages`, options),
 });
 
+//      
+
+                                     
+               
+  
+
+class MeetingStatus {
+               
+
+                
+
+  constructor({
+    full,
+  }         = {}) {
+    this.full = full;
+
+    // Useful to compare instead of instanceof with minified code
+    this.type = 'MeetingStatus';
+  }
+
+  static parse(plain                       )                {
+    return new MeetingStatus({
+      full: plain.full,
+    });
+  }
+
+}
+
 /*       */
 
                   
@@ -5131,6 +5167,8 @@ var calldMethods = (client              , baseUrl        ) => ({
     client.put(`${baseUrl}/users/me/calls/${callId}/record/resume`, null, null,
       ApiRequester.successResponseParser),
 
+  guestGetMeetingStatus: (meetingUuid        )                         =>
+    client.get(`${baseUrl}/guests/me/meetings/${meetingUuid}/status`).then(MeetingStatus.parse),
 });
 
 //      
@@ -6198,7 +6236,7 @@ const events$1 = [REGISTERED, UNREGISTERED, REGISTRATION_FAILED, INVITE];
 const transportEvents = [CONNECTED, DISCONNECTED, TRANSPORT_ERROR, MESSAGE];
 class CanceledCallError extends Error {}
 
-const MAX_REGISTER_TRIES = 10;
+const MAX_REGISTER_TRIES = 5;
 
                     
                           
@@ -6252,6 +6290,7 @@ class WebRTCClient extends Emitter {
                         
                                                       
                                             
+                       
 
   // sugar
                         
@@ -6314,6 +6353,7 @@ class WebRTCClient extends Emitter {
     this.conferences = {};
     this.networkMonitoringInterval = {};
     this.sessionNetworkStats = {};
+    this.forceClosed = false;
 
     this._boundOnHeartbeat = this._onHeartbeat.bind(this);
     this.heartbeat = new Heartbeat(config.heartbeatDelay, config.heartbeatTimeout, config.maxHeartbeats);
@@ -6435,6 +6475,8 @@ class WebRTCClient extends Emitter {
       skipRegister: this.skipRegister,
     };
 
+    this.forceClosed = false;
+
     if (this.skipRegister) {
       logger$9.info('sdk webrtc skip register...', logInfo);
       return Promise.resolve();
@@ -6459,7 +6501,15 @@ class WebRTCClient extends Emitter {
     const registerOptions = this._isWeb() ? {} : { extraContactHeaderParams: ['mobility=mobile'] };
 
     const onRegisterFailed = () => {
-      logger$9.info('sdk webrtc registering failed', { tries, registerer: !!this.registerer });
+      logger$9.info('sdk webrtc registering failed', {
+        tries,
+        registerer: !!this.registerer,
+        forceClosed: this.forceClosed,
+      });
+      if (this.forceClosed) {
+        return;
+      }
+
       this.connectionPromise = null;
       if (this.registerer && this.registerer.waiting) {
         this.registerer.waitingToggle(false);
@@ -6737,8 +6787,9 @@ class WebRTCClient extends Emitter {
     }
   }
 
-  async close() {
-    logger$9.info('sdk webrtc closing client', { userAgent: !!this.userAgent });
+  async close(force          = false) {
+    logger$9.info('sdk webrtc closing client', { userAgent: !!this.userAgent, force });
+    this.forceClosed = force;
     this._cleanupMedia();
     this.connectionPromise = null;
 
@@ -6757,7 +6808,7 @@ class WebRTCClient extends Emitter {
     this.userAgent.delegate = null;
     this.userAgent.stateChange.removeAllListeners();
 
-    await this._disconnectTransport();
+    await this._disconnectTransport(force);
 
     this._cleanupRegister();
 
@@ -8193,7 +8244,15 @@ class WebRTCClient extends Emitter {
     return userAgent.UserAgent.makeURI(`sip:${target}@${this.config.host}`);
   }
 
-  async _disconnectTransport() {
+  async _disconnectTransport(force          = false) {
+    if (force) {
+      // Bypass sip.js state machine that prevent to close WS with the state `Connecting`
+      this.userAgent.transport.disconnectResolve = () => {};
+      // eslint-disable-next-line
+      this.userAgent.transport._ws.close(1000);
+      return;
+    }
+
     // Check if `disconnectPromise` is not present to avoid `Disconnect promise must not be defined` errors.
     if (this.userAgent && this.userAgent.transport && !this.userAgent.transport.disconnectPromise) {
       try {
@@ -9831,6 +9890,7 @@ class WebRTCPhone extends Emitter                  {
     }
 
     callSession.answerTime = new Date();
+    this._updateCallSession(callSession);
     this.eventEmitter.emit(ON_CALL_ACCEPTED, callSession, cameraEnabled);
 
     return callSession;
@@ -10040,6 +10100,12 @@ class WebRTCPhone extends Emitter                  {
       return Promise.resolve(null);
     }
 
+    const sipSessionId = this.getSipSessionIdFromCallSession(callSession);
+    if (sipSessionId && !(sipSessionId in this.callSessions)) {
+      logger$7.warn('Call session already ended or not found, can\'t accept it.', { sipSessionId });
+      return Promise.resolve(null);
+    }
+
     if (callSession.getId() in this.acceptedSessions) {
       logger$7.warn('CallSession already accepted.', { id: callSession ? callSession.getId() : 'n/a' });
       return Promise.resolve(callSession.sipCallId);
@@ -10047,6 +10113,7 @@ 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);
@@ -10146,7 +10213,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$7.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);
@@ -10357,6 +10424,7 @@ class WebRTCPhone extends Emitter                  {
     }
     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);
 
@@ -10498,7 +10566,7 @@ class WebRTCPhone extends Emitter                  {
     return this.client && this.client.isConference(callSession.sipCallId);
   }
 
-  async close()                {
+  async close(force          = false)                {
     logger$7.info('WebRTC close');
 
     try {
@@ -10510,7 +10578,7 @@ class WebRTCPhone extends Emitter                  {
       logger$7.error('WebRTC close, unregister error', e);
     }
 
-    this.client.close();
+    this.client.close(force);
     this.unbind();
 
     this.incomingSessions = [];
@@ -10636,6 +10704,8 @@ class WebRTCPhone extends Emitter                  {
       }
 
       callSession.creationTime = new Date();
+      this._updateCallSession(callSession);
+
       this.eventEmitter.emit(ON_CALL_INCOMING, callSession, wantsToDoVideo);
     });
 
@@ -10660,9 +10730,17 @@ class WebRTCPhone extends Emitter                  {
     });
 
     this.client.on(this.client.ACCEPTED, (sipSession         ) => {
-      logger$7.info('WebRTC call accepted', { sipId: sipSession.id });
+      const sessionId = this.getSipSessionId(sipSession);
+      const hasSession = sessionId in this.callSessions;
 
-      this._onCallAccepted(sipSession, this.client.hasVideo(this.getSipSessionId(sipSession)));
+      logger$7.info('WebRTC call accepted', { sipId: sessionId, hasSession });
+
+      if (!hasSession) {
+        logger$7.warn('Call accepted ignored, session is no longer present in the WebRtcPhone', { sessionId });
+        return;
+      }
+
+      this._onCallAccepted(sipSession, this.client.hasVideo(sessionId));
 
       if (this.audioOutputDeviceId) {
         this.client.changeAudioOutputDevice(this.audioOutputDeviceId);
@@ -10679,6 +10757,8 @@ class WebRTCPhone extends Emitter                  {
       logger$7.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);
     });
 
@@ -10796,9 +10876,14 @@ class WebRTCPhone extends Emitter                  {
     return this.client.getSipSessionId(sipSession);
   }
 
-  _updateCallSession(callSession             ) {
+  getSipSessionIdFromCallSession(callSession             ) {
     const sipSession = this.findSipSession(callSession);
-    const sipSessionId = sipSession ? this.getSipSessionId(sipSession) : null;
+
+    return sipSession ? this.getSipSessionId(sipSession) : null;
+  }
+
+  _updateCallSession(callSession             ) {
+    const sipSessionId = this.getSipSessionIdFromCallSession(callSession);
 
     if (sipSessionId) {
       this.callSessions[sipSessionId] = callSession;
@@ -10903,6 +10988,7 @@ class WebRTCPhone extends Emitter                  {
       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),