@stream-io/video-client 0.7.5 → 0.7.7

package/dist/src/Call.d.ts

@@ -79,6 +79,7 @@ export declare class Call {
     private sfuClient?;
     private reconnectAttempts;
     private maxReconnectAttempts;
+    private isLeaving;
     /**
      * A list hooks/functions to invoke when the call is left.
      * A typical use case is to clean up some global event handlers.

package/package.json

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

package/src/Call.ts

@@ -216,6 +216,7 @@ export class Call {
   private sfuClient?: StreamSfuClient;
   private reconnectAttempts = 0;
   private maxReconnectAttempts = 10;
+  private isLeaving = false;
 
   /**
    * A list hooks/functions to invoke when the call is left.
@@ -479,6 +480,8 @@ export class Call {
       await this.assertCallJoined();
     }
 
+    this.isLeaving = true;
+
     if (this.ringing) {
       // I'm the one who started the call, so I should cancel it.
       const hasOtherParticipants = this.state.remoteParticipants.length > 0;
@@ -887,6 +890,9 @@ export class Call {
         // unregister the "goAway" handler, as we won't need it anymore for this connection.
         // the upcoming re-join will register a new handler anyway
         unregisterGoAway();
+        // when the user has initiated "call.leave()" operation, we shouldn't
+        // care for the WS close code and we shouldn't ever attempt to reconnect
+        if (this.isLeaving) return;
         // do nothing if the connection was closed on purpose
         if (e.code === StreamSfuClient.NORMAL_CLOSURE) return;
         // do nothing if the connection was closed because of a policy violation
@@ -1933,9 +1939,8 @@ export class Call {
     if (rating < 1 || rating > 5) {
       throw new Error('Rating must be between 1 and 5');
     }
-    const userSessionId = this.sfuClient?.sessionId;
     const callSessionId = this.state.session?.id;
-    if (!callSessionId || !userSessionId) {
+    if (!callSessionId) {
       throw new Error(
         'Feedback can be submitted only in the context of a call session',
       );
@@ -1945,6 +1950,9 @@ export class Call {
       getClientDetails(),
     );
 
+    // user sessionId is not available once the call has been left
+    // until we relax the backend validation, we'll send N/A
+    const userSessionId = this.sfuClient?.sessionId ?? 'N/A';
     const endpoint = `${this.streamClientBasePath}/feedback/${callSessionId}`;
     return this.streamClient.post<
       CollectUserFeedbackResponse,