@wazo/sdk 0.37.0 → 0.37.4

package/esm/__tests__/api-client.test.js

@@ -309,7 +309,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {
@@ -2487,6 +2493,65 @@ class ExternalApp {
   }
 }
 
+//      
+
+                                       
+                                  
+                   
+               
+                             
+               
+               
+                
+ 
+
+class Meeting {
+               
+
+                                
+                   
+               
+               
+               
+                 
+                            
+
+  static parse(plain                         )          {
+    return new Meeting({
+      guestSipAuthorization: plain.guest_sip_authorization,
+      hostname: plain.hostname,
+      name: plain.name,
+      ownerUuids: plain.owner_uuids,
+      port: plain.port,
+      uuid: plain.uuid,
+      number: plain.number,
+    });
+  }
+
+  static newFrom(meeting         ) {
+    return newFrom(meeting, Meeting);
+  }
+
+  constructor({ uuid, name, guestSipAuthorization, ownerUuids, port, hostname, number }         = {}) {
+    this.guestSipAuthorization = guestSipAuthorization;
+    this.hostname = hostname;
+    this.name = name;
+    this.ownerUuids = ownerUuids;
+    this.port = port;
+    this.uuid = uuid;
+    this.number = number;
+
+    // Useful to compare instead of instanceof with minified code
+    this.type = 'Meeting';
+  }
+
+  getGuestSipCredentials() {
+    // eslint-disable-next-line no-undef
+    const [username, secret] = atob(this.guestSipAuthorization).split(':');
+    return { username, secret };
+  }
+}
+
 /*       */
 
 var confdMethods = (client              , baseUrl        ) => ({
@@ -2551,6 +2616,18 @@ var confdMethods = (client              , baseUrl        ) => ({
       return null;
     }
   },
+
+  createMyMeeting: (name        )                   =>
+    client.post(`${baseUrl}/users/me/meetings`, { name }).then(Meeting.parse),
+
+  deleteMyMeeting: (meetingUuid        )                   =>
+    client.delete(`${baseUrl}/users/me/meetings/${meetingUuid}`, null),
+
+  getMeeting: (meetingUuid        )                   =>
+    client.get(`${baseUrl}/meetings/${meetingUuid}`, null).then(Meeting.parse),
+
+  guestGetMeeting: (meetingUuid        )                   =>
+    client.get(`${baseUrl}/guests/me/meetings/${meetingUuid}`, null).then(Meeting.parse),
 });
 
 //      
@@ -3135,7 +3212,11 @@ class CallLog {
       answered: plain.answered,
       callDirection: plain.call_direction,
       destination: {
-        extension: plain.requested_extension || plain.destination_extension,
+        // @TEMP: Temporarily assuming empty numbers are meetings
+        // which is admittedly a very dangerous assumption. Did i mention it was temporary?
+        extension: plain.requested_extension
+          || plain.destination_extension
+          || `meeting-${plain.requested_name || plain.destination_name || ''}`,
         name: plain.requested_name || plain.destination_name || '',
       },
       source: {
@@ -3151,27 +3232,11 @@ class CallLog {
   }
 
   static parseNew(plain                 , session         )          {
-    return new CallLog({
-      answer: plain.answer ? moment(plain.answer).toDate() : null,
-      answered: plain.answered,
-      callDirection: plain.call_direction,
-      destination: {
-        extension: plain.requested_extension || plain.destination_extension,
-        name: plain.requested_name || plain.destination_name || '',
-      },
-      source: {
-        extension: plain.source_extension,
-        name: plain.source_name,
-      },
-      id: plain.id,
-      duration: (plain.duration || 0) * 1000, // duration is in seconds
-      start: moment(plain.start).toDate(),
-      end: plain.end ? moment(plain.end).toDate() : null,
-      recordings: Recording.parseMany(plain.recordings),
-      // @TODO: FIXME add verification declined vs missed call
-      newMissedCall: session
-        && session.hasExtension(plain.requested_extension || plain.destination_extension) && !plain.answered,
-    });
+    const callLog          = CallLog.parse(plain);
+    // @TODO: FIXME add verification declined vs missed call
+    callLog.newMissedCall = session
+      && session.hasExtension(plain.requested_extension || plain.destination_extension) && !plain.answered;
+    return callLog;
   }
 
   static newFrom(profile         ) {
@@ -3928,6 +3993,9 @@ var calldMethods = (client              , baseUrl        ) => ({
   getConferenceParticipantsAsUser: async (conferenceId        )                  =>
     client.get(`${baseUrl}/users/me/conferences/${conferenceId}/participants`),
 
+  getMeetingParticipantsAsUser: async (meetingUuid        )                  =>
+    client.get(`${baseUrl}/users/me/meetings/${meetingUuid}/participants`),
+
   listTrunks: () => client.get(`${baseUrl}/trunks`),
 
   mute: (callId        ) =>

package/esm/__tests__/web-rtc-client.test.js

@@ -591,7 +591,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {
@@ -763,7 +769,9 @@ class WazoSessionDescriptionHandler extends SessionDescriptionHandler {
       .then(() => {
         if (isConference && (options.constraints && !options.constraints.video) && !('hold' in options) && !audioOnly) {
           // Add a video an empty bundle to be able to replaceTrack when joining a conference without video
-          this.peerConnection.addTransceiver('video', { streams: [this._localMediaStream], direction: 'sendrecv' });
+          if (this.peerConnection.addTransceiver) {
+            this.peerConnection.addTransceiver('video', { streams: [this._localMediaStream], direction: 'sendrecv' });
+          }
         }
       })
       .then(() => this.createLocalOfferOrAnswer(options))
@@ -790,12 +798,6 @@ class WazoSessionDescriptionHandler extends SessionDescriptionHandler {
         };
       })
       .then((sessionDescription) => this.applyModifiers(sessionDescription, modifiers))
-      .then((sessionDescription) => {
-        // Set new constraints to avoid the constraints check issue in `sdh.getLocalMediaStream` later.
-        this.localMediaStreamConstraints = options.constraints;
-
-        return sessionDescription;
-      })
       .then((sessionDescription) => ({ body: sessionDescription.sdp, contentType: 'application/sdp' }))
       .catch((error) => {
         wazoLogger.error('error when creating media', error);
@@ -990,6 +992,9 @@ class WazoSessionDescriptionHandler extends SessionDescriptionHandler {
 
         // set the transceiver direction to the answer direction
         this._peerConnection.getTransceivers().forEach((transceiver) => {
+          if (transceiver.stopped) {
+            return;
+          }
           if (transceiver.direction /* guarding, but should always be true */) {
             const { receiver } = transceiver;
             if (isConference && audioOnly && receiver.track && receiver.track.kind === 'video') {
@@ -1034,10 +1039,15 @@ class WazoSessionDescriptionHandler extends SessionDescriptionHandler {
       if (kind !== 'audio' && kind !== 'video') {
         throw new Error(`Unknown new track kind ${kind}.`);
       }
-      const sender = pc.getSenders().find((otherSender) => otherSender.track && otherSender.track.kind === kind);
+      const sender = pc.getSenders && pc.getSenders().find((otherSender) =>
+        otherSender.track && otherSender.track.kind === kind);
 
       // Do not reuse sender video tracks in SFU
       if (sender && (!sfu || newTrack.kind === 'audio')) {
+        if (sender.track) {
+          // eslint-disable-next-line no-param-reassign
+          newTrack.enabled = sender.track.enabled;
+        }
         trackUpdates.push(
           new Promise((resolve) => {
             this.logger.debug(`SessionDescriptionHandler.setLocalMediaStream - replacing sender ${kind} track`);
@@ -1071,7 +1081,11 @@ class WazoSessionDescriptionHandler extends SessionDescriptionHandler {
             // Review: could make streamless tracks a configurable option?
             // https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/addTrack#Usage_notes
             try {
-              pc.addTrack(newTrack, localStream);
+              if (pc.addTrack) {
+                pc.addTrack(newTrack, localStream);
+              } else {
+                pc.addStream(localStream);
+              }
             } catch (error) {
               this.logger.error(`SessionDescriptionHandler.setLocalMediaStream - failed to add sender ${kind} track`);
               throw error;
@@ -3028,6 +3042,65 @@ class ExternalApp {
   }
 }
 
+//      
+
+                                       
+                                  
+                   
+               
+                             
+               
+               
+                
+ 
+
+class Meeting {
+               
+
+                                
+                   
+               
+               
+               
+                 
+                            
+
+  static parse(plain                         )          {
+    return new Meeting({
+      guestSipAuthorization: plain.guest_sip_authorization,
+      hostname: plain.hostname,
+      name: plain.name,
+      ownerUuids: plain.owner_uuids,
+      port: plain.port,
+      uuid: plain.uuid,
+      number: plain.number,
+    });
+  }
+
+  static newFrom(meeting         ) {
+    return newFrom(meeting, Meeting);
+  }
+
+  constructor({ uuid, name, guestSipAuthorization, ownerUuids, port, hostname, number }         = {}) {
+    this.guestSipAuthorization = guestSipAuthorization;
+    this.hostname = hostname;
+    this.name = name;
+    this.ownerUuids = ownerUuids;
+    this.port = port;
+    this.uuid = uuid;
+    this.number = number;
+
+    // Useful to compare instead of instanceof with minified code
+    this.type = 'Meeting';
+  }
+
+  getGuestSipCredentials() {
+    // eslint-disable-next-line no-undef
+    const [username, secret] = atob(this.guestSipAuthorization).split(':');
+    return { username, secret };
+  }
+}
+
 /*       */
 
 var confdMethods = (client              , baseUrl        ) => ({
@@ -3092,6 +3165,18 @@ var confdMethods = (client              , baseUrl        ) => ({
       return null;
     }
   },
+
+  createMyMeeting: (name        )                   =>
+    client.post(`${baseUrl}/users/me/meetings`, { name }).then(Meeting.parse),
+
+  deleteMyMeeting: (meetingUuid        )                   =>
+    client.delete(`${baseUrl}/users/me/meetings/${meetingUuid}`, null),
+
+  getMeeting: (meetingUuid        )                   =>
+    client.get(`${baseUrl}/meetings/${meetingUuid}`, null).then(Meeting.parse),
+
+  guestGetMeeting: (meetingUuid        )                   =>
+    client.get(`${baseUrl}/guests/me/meetings/${meetingUuid}`, null).then(Meeting.parse),
 });
 
 //      
@@ -3676,7 +3761,11 @@ class CallLog {
       answered: plain.answered,
       callDirection: plain.call_direction,
       destination: {
-        extension: plain.requested_extension || plain.destination_extension,
+        // @TEMP: Temporarily assuming empty numbers are meetings
+        // which is admittedly a very dangerous assumption. Did i mention it was temporary?
+        extension: plain.requested_extension
+          || plain.destination_extension
+          || `meeting-${plain.requested_name || plain.destination_name || ''}`,
         name: plain.requested_name || plain.destination_name || '',
       },
       source: {
@@ -3692,27 +3781,11 @@ class CallLog {
   }
 
   static parseNew(plain                 , session         )          {
-    return new CallLog({
-      answer: plain.answer ? moment(plain.answer).toDate() : null,
-      answered: plain.answered,
-      callDirection: plain.call_direction,
-      destination: {
-        extension: plain.requested_extension || plain.destination_extension,
-        name: plain.requested_name || plain.destination_name || '',
-      },
-      source: {
-        extension: plain.source_extension,
-        name: plain.source_name,
-      },
-      id: plain.id,
-      duration: (plain.duration || 0) * 1000, // duration is in seconds
-      start: moment(plain.start).toDate(),
-      end: plain.end ? moment(plain.end).toDate() : null,
-      recordings: Recording.parseMany(plain.recordings),
-      // @TODO: FIXME add verification declined vs missed call
-      newMissedCall: session
-        && session.hasExtension(plain.requested_extension || plain.destination_extension) && !plain.answered,
-    });
+    const callLog          = CallLog.parse(plain);
+    // @TODO: FIXME add verification declined vs missed call
+    callLog.newMissedCall = session
+      && session.hasExtension(plain.requested_extension || plain.destination_extension) && !plain.answered;
+    return callLog;
   }
 
   static newFrom(profile         ) {
@@ -4469,6 +4542,9 @@ var calldMethods = (client              , baseUrl        ) => ({
   getConferenceParticipantsAsUser: async (conferenceId        )                  =>
     client.get(`${baseUrl}/users/me/conferences/${conferenceId}/participants`),
 
+  getMeetingParticipantsAsUser: async (meetingUuid        )                  =>
+    client.get(`${baseUrl}/users/me/meetings/${meetingUuid}/participants`),
+
   listTrunks: () => client.get(`${baseUrl}/trunks`),
 
   mute: (callId        ) =>
@@ -5101,6 +5177,7 @@ const MAX_REGISTER_TRIES = 10;
                            
                          
                            
+                        
                         
   
 
@@ -5127,6 +5204,7 @@ class WebRTCClient extends Emitter {
                          
                                      
                                      
+                        
 
   // sugar
                         
@@ -5165,6 +5243,8 @@ class WebRTCClient extends Emitter {
     super();
     this.uaConfigOverrides = uaConfigOverrides;
     this.config = config;
+    this.skipRegister = config.skipRegister;
+
     this._buildConfig(config, session).then((newConfig              ) => {
       this.config = newConfig;
       this.userAgent = this.createUserAgent(uaConfigOverrides);
@@ -5292,14 +5372,22 @@ class WebRTCClient extends Emitter {
   }
 
   register(tries         = 0)               {
-    logger.info('sdk webrtc registering...', {
+    const logInfo = {
       userAgent: !!this.userAgent,
       registered: this.isRegistered(),
       connectionPromise: !!this.connectionPromise,
       registerer: !!this.registerer,
       waiting: this.registerer && this.registerer.waiting,
       tries,
-    });
+      skipRegister: this.skipRegister,
+    };
+
+    if (this.skipRegister) {
+      logger.info('sdk webrtc skip register...', logInfo);
+      return Promise.resolve();
+    }
+
+    logger.info('sdk webrtc registering...', logInfo);
 
     if (!this.userAgent) {
       logger.info('sdk webrtc recreating User Agent');
@@ -5630,12 +5718,18 @@ class WebRTCClient extends Emitter {
     }
 
     if (pc.getSenders) {
+      if (!pc.getSenders().length) {
+        return false;
+      }
       pc.getSenders().forEach(sender => {
         if (sender && sender.track && sender.track.kind === 'audio') {
           muted = muted && !sender.track.enabled;
         }
       });
     } else {
+      if (!pc.getLocalStreams().length) {
+        return false;
+      }
       pc.getLocalStreams().forEach(stream => {
         stream.getAudioTracks().forEach(track => {
           muted = muted && !track.enabled;
@@ -5692,6 +5786,9 @@ class WebRTCClient extends Emitter {
     if (!this._isWeb()) {
       options.sessionDescriptionHandlerModifiers = [holdModifier];
     }
+    options.sessionDescriptionHandlerOptions = {
+      constraints: options.constraints,
+    };
 
     // Send re-INVITE
     return session.invite(options);
@@ -5731,6 +5828,9 @@ class WebRTCClient extends Emitter {
       // (eg: holdModifier)
       options.sessionDescriptionHandlerModifiers = [];
     }
+    options.sessionDescriptionHandlerOptions = {
+      constraints: options.constraints,
+    };
 
     // Send re-INVITE
     return session.invite(options);
@@ -5913,7 +6013,8 @@ class WebRTCClient extends Emitter {
     }
     const sdh = session.sessionDescriptionHandler;
     const pc = sdh.peerConnection;
-    const localStream = this.getLocalStream(this.getSipSessionId(session));
+    const sessionId = this.getSipSessionId(session);
+    const localStream = this.getLocalStream(sessionId);
 
     // Release old video stream
     if (localStream) {
@@ -5939,11 +6040,8 @@ class WebRTCClient extends Emitter {
 
       // let's update the local stream
       this.eventEmitter.emit('onVideoInputChange', stream);
-      try {
-        await sdh.setLocalMediaStream(stream);
-      } catch (_) {
-        // Nothing to do
-      }
+      this.setLocalMediaStream(sessionId, stream);
+
       return stream;
     });
   }
@@ -6245,9 +6343,11 @@ class WebRTCClient extends Emitter {
       remoteStream.removeTrack(track);
     });
 
-    pc.getReceivers().forEach(receiver => {
-      remoteStream.addTrack(receiver.track);
-    });
+    if (pc.getReceivers) {
+      pc.getReceivers().forEach(receiver => {
+        remoteStream.addTrack(receiver.track);
+      });
+    }
   }
 
   getMediaConfiguration(enableVideo         , conference          = false, constraints          = null)         {
@@ -6282,6 +6382,25 @@ class WebRTCClient extends Emitter {
     return sessionId in this.conferences;
   }
 
+  createAudioElementFor(sessionId        ) {
+    const audio      = document.createElement('audio');
+    audio.setAttribute('id', `audio-${sessionId}`);
+
+    if (audio.setSinkId && this.audioOutputDeviceId) {
+      audio.setSinkId(this.audioOutputDeviceId);
+    }
+
+    if (document.body) {
+      document.body.appendChild(audio);
+    }
+    this.audioElements[sessionId] = audio;
+
+    audio.volume = this.audioOutputVolume;
+    audio.autoplay = true;
+
+    return audio;
+  }
+
   _onTransportError() {
     this.eventEmitter.emit(TRANSPORT_ERROR);
     this.attemptReconnection();
@@ -6511,7 +6630,7 @@ class WebRTCClient extends Emitter {
     let hadRemoteVideo = false;
 
     // Monkey patch `onInviteRequest` to be able to know if there was a remote video stream before `onInvite` is called
-    // Becase when `onInvite` is called we already got the video track
+    // Because when `onInvite` is called we already got the video track
     session.onInviteRequest = (request) => {
       hadRemoteVideo = this.hasARemoteVideoTrack(sipSessionId);
 
@@ -6586,20 +6705,10 @@ class WebRTCClient extends Emitter {
     const isConference = this.isConference(sessionId);
 
     if (this._hasAudio() && this._isWeb() && !(sessionId in this.audioElements)) {
-      const audio      = document.createElement('audio');
-      audio.setAttribute('id', `audio-${sessionId}`);
-
-      if (audio.setSinkId && this.audioOutputDeviceId) {
-        audio.setSinkId(this.audioOutputDeviceId);
-      }
-
-      if (document.body) {
-        document.body.appendChild(audio);
-      }
-      this.audioElements[sessionId] = audio;
+      this.createAudioElementFor(sessionId);
     }
 
-    const audioElement = this.audioElements[this.getSipSessionId(session)];
+    const audioElement = this.audioElements[sessionId];
     const stream = newStream || this.getRemoteStream(sessionId);
 
     if (!this._isWeb() || !stream) {

package/esm/api/agentd.js

@@ -305,7 +305,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {

package/esm/api/amid.js

@@ -244,7 +244,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {

package/esm/api/application.js

@@ -305,7 +305,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {

package/esm/api/auth.js

@@ -306,7 +306,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {

package/esm/api/call-logd.js

@@ -245,7 +245,13 @@ class IssueReporter {
   }
 
   _isLevelAbove(level1        , level2        ) {
-    return LOG_LEVELS.indexOf(level1) > LOG_LEVELS.indexOf(level2);
+    const index1 = LOG_LEVELS.indexOf(level1);
+    const index2 = LOG_LEVELS.indexOf(level2);
+    if (index1 === -1 || index2 === -1) {
+      return false;
+    }
+
+    return index1 > index2;
   }
 
   _makeCategory(category        ) {
@@ -444,7 +450,11 @@ class CallLog {
       answered: plain.answered,
       callDirection: plain.call_direction,
       destination: {
-        extension: plain.requested_extension || plain.destination_extension,
+        // @TEMP: Temporarily assuming empty numbers are meetings
+        // which is admittedly a very dangerous assumption. Did i mention it was temporary?
+        extension: plain.requested_extension
+          || plain.destination_extension
+          || `meeting-${plain.requested_name || plain.destination_name || ''}`,
         name: plain.requested_name || plain.destination_name || '',
       },
       source: {
@@ -460,27 +470,11 @@ class CallLog {
   }
 
   static parseNew(plain                 , session         )          {
-    return new CallLog({
-      answer: plain.answer ? moment(plain.answer).toDate() : null,
-      answered: plain.answered,
-      callDirection: plain.call_direction,
-      destination: {
-        extension: plain.requested_extension || plain.destination_extension,
-        name: plain.requested_name || plain.destination_name || '',
-      },
-      source: {
-        extension: plain.source_extension,
-        name: plain.source_name,
-      },
-      id: plain.id,
-      duration: (plain.duration || 0) * 1000, // duration is in seconds
-      start: moment(plain.start).toDate(),
-      end: plain.end ? moment(plain.end).toDate() : null,
-      recordings: Recording.parseMany(plain.recordings),
-      // @TODO: FIXME add verification declined vs missed call
-      newMissedCall: session
-        && session.hasExtension(plain.requested_extension || plain.destination_extension) && !plain.answered,
-    });
+    const callLog          = CallLog.parse(plain);
+    // @TODO: FIXME add verification declined vs missed call
+    callLog.newMissedCall = session
+      && session.hasExtension(plain.requested_extension || plain.destination_extension) && !plain.answered;
+    return callLog;
   }
 
   static newFrom(profile         ) {