@stream-io/video-client 1.11.9 → 1.11.11

package/dist/src/types.d.ts

@@ -174,8 +174,6 @@ export type CallLeaveOptions = {
     /**
      * If true, the caller will get a `call.rejected` event.
      * Has an effect only if the call is in the `ringing` state.
-     *
-     * @default `false`.
      */
     reject?: boolean;
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stream-io/video-client",
-  "version": "1.11.9",
+  "version": "1.11.11",
   "packageManager": "yarn@3.2.4",
   "main": "dist/index.cjs.js",
   "module": "dist/index.es.js",

package/src/Call.ts

@@ -496,7 +496,7 @@ export class Call {
    * Leave the call and stop the media streams that were published by the call.
    */
   leave = async ({
-    reject = false,
+    reject,
     reason = 'user is leaving the call',
   }: CallLeaveOptions = {}) => {
     await withoutConcurrency(this.joinLeaveConcurrencyTag, async () => {
@@ -516,13 +516,14 @@ export class Call {
         await waitUntilCallJoined();
       }
 
-      if (callingState === CallingState.RINGING) {
+      if (callingState === CallingState.RINGING && reject !== false) {
         if (reject) {
           await this.reject(reason);
         } else {
+          // if reject was undefined, we still have to cancel the call automatically
+          // when I am the creator and everyone else left the call
           const hasOtherParticipants = this.state.remoteParticipants.length > 0;
           if (this.isCreatedByMe && !hasOtherParticipants) {
-            // I'm the one who started the call, so I should cancel it when there are no other participants.
             await this.reject('cancel');
           }
         }

package/src/devices/InputMediaDeviceManager.ts

@@ -215,7 +215,6 @@ export abstract class InputMediaDeviceManager<
       await this.applySettingsToStream();
     } catch (error) {
       this.state.setDevice(prevDeviceId);
-      await this.applySettingsToStream();
       throw error;
     }
   }

package/src/devices/devices.ts

@@ -184,7 +184,7 @@ export const getAudioStream = async (
   const constraints: MediaStreamConstraints = {
     audio: {
       ...audioDeviceConstraints.audio,
-      ...normalizeContraints(trackConstraints),
+      ...trackConstraints,
     },
   };
 
@@ -195,6 +195,20 @@ export const getAudioStream = async (
     });
     return await getStream(constraints);
   } catch (error) {
+    if (
+      error instanceof DOMException &&
+      error.name === 'OverconstrainedError' &&
+      trackConstraints?.deviceId
+    ) {
+      const { deviceId, ...relaxedContraints } = trackConstraints;
+      getLogger(['devices'])(
+        'warn',
+        'Failed to get audio stream, will try again with relaxed contraints',
+        { error, constraints, relaxedContraints },
+      );
+      return getAudioStream(relaxedContraints);
+    }
+
     getLogger(['devices'])('error', 'Failed to get audio stream', {
       error,
       constraints,
@@ -217,7 +231,7 @@ export const getVideoStream = async (
   const constraints: MediaStreamConstraints = {
     video: {
       ...videoDeviceConstraints.video,
-      ...normalizeContraints(trackConstraints),
+      ...trackConstraints,
     },
   };
   try {
@@ -227,6 +241,20 @@ export const getVideoStream = async (
     });
     return await getStream(constraints);
   } catch (error) {
+    if (
+      error instanceof DOMException &&
+      error.name === 'OverconstrainedError' &&
+      trackConstraints?.deviceId
+    ) {
+      const { deviceId, ...relaxedContraints } = trackConstraints;
+      getLogger(['devices'])(
+        'warn',
+        'Failed to get video stream, will try again with relaxed contraints',
+        { error, constraints, relaxedContraints },
+      );
+      return getVideoStream(relaxedContraints);
+    }
+
     getLogger(['devices'])('error', 'Failed to get video stream', {
       error,
       constraints,
@@ -235,20 +263,6 @@ export const getVideoStream = async (
   }
 };
 
-function normalizeContraints(constraints: MediaTrackConstraints | undefined) {
-  if (
-    constraints?.deviceId === 'default' ||
-    (typeof constraints?.deviceId === 'object' &&
-      'exact' in constraints.deviceId &&
-      constraints.deviceId.exact === 'default')
-  ) {
-    const { deviceId, ...contraintsWithoutDeviceId } = constraints;
-    return contraintsWithoutDeviceId;
-  }
-
-  return constraints;
-}
-
 /**
  * Prompts the user for a permission to share a screen.
  * If the user grants the permission, a screen sharing stream is returned. Throws otherwise.

package/src/events/call.ts

@@ -79,9 +79,11 @@ export const watchCallEnded = (call: Call) => {
       callingState !== CallingState.IDLE &&
       callingState !== CallingState.LEFT
     ) {
-      call.leave({ reason: 'call.ended event received' }).catch((err) => {
-        call.logger('error', 'Failed to leave call after call.ended ', err);
-      });
+      call
+        .leave({ reason: 'call.ended event received', reject: false })
+        .catch((err) => {
+          call.logger('error', 'Failed to leave call after call.ended ', err);
+        });
     }
   };
 };

package/src/types.ts

@@ -215,8 +215,6 @@ export type CallLeaveOptions = {
   /**
    * If true, the caller will get a `call.rejected` event.
    * Has an effect only if the call is in the `ringing` state.
-   *
-   * @default `false`.
    */
   reject?: boolean;