From 4d8c79522b623d0564064f9557f07bae13179864 Mon Sep 17 00:00:00 2001
From: Miroslav Pejic <miroslav.pejic.85@gmail.com>
Date: Thu, 12 Jun 2025 18:01:49 +0200
Subject: [PATCH] [mirotalksfu] - #200 refactor socket.io, update dep

---
 app/src/Room.js         |   3 +-
 app/src/Server.js       |  17 ++-
 package.json            |   8 +-
 public/js/Brand.js      |   2 +-
 public/js/Room.js       |  28 +++-
 public/js/RoomClient.js | 330 +++++++++++++++++++++-------------------
 public/views/Room.html  |   6 +-
 7 files changed, 223 insertions(+), 171 deletions(-)

diff --git a/app/src/Room.js b/app/src/Room.js
index 3358b134..a392af26 100644
--- a/app/src/Room.js
+++ b/app/src/Room.js
@@ -651,6 +651,7 @@ module.exports = class Room {
         });
 
         const { peer_name } = peer;
+        const { iceConsentTimeout = 35 } = this.webRtcTransport;
 
         transport.observer.on('newproducer', (producer) => {
             log.debug('---> new producer created [id:%s]', producer.id);
@@ -678,7 +679,7 @@ module.exports = class Room {
                         log.warn(`Closing transport ${transport.id} due to prolonged ICE disconnection`);
                         transport.close();
                     }
-                }, 5000); // Wait 5 seconds before closing
+                }, iceConsentTimeout * 1000); // Wait iceConsentTimeout seconds before closing
             } else if (iceState === 'closed') {
                 log.warn(`ICE state closed for transport ${transport.id}`);
                 transport.close();
diff --git a/app/src/Server.js b/app/src/Server.js
index 61e57509..2ceb67c5 100644
--- a/app/src/Server.js
+++ b/app/src/Server.js
@@ -64,7 +64,7 @@ dev dependencies: {
  * @license For commercial or closed source, contact us at license.mirotalk@gmail.com or purchase directly via CodeCanyon
  * @license CodeCanyon: https://codecanyon.net/item/mirotalk-sfu-webrtc-realtime-video-conferences/40769970
  * @author  Miroslav Pejic - miroslav.pejic.85@gmail.com
- * @version 1.8.63
+ * @version 1.8.64
  *
  */
 
@@ -1742,6 +1742,21 @@ function startServer() {
                 return cb('isBanned');
             }
 
+            // Remove old peer with same socket.id before adding new one
+            const existingPeer = room.getPeer(socket.id);
+            if (existingPeer) {
+                room.removePeer(socket.id);
+            }
+
+            // If you want to check by peer_uuid as well, you can do:
+            if (peer_uuid) {
+                for (const peer of room.getPeers()) {
+                    if (peer.peer_info?.peer_uuid === peer_uuid) {
+                        room.removePeer(peer.id);
+                    }
+                }
+            }
+
             room.addPeer(new Peer(socket.id, data));
 
             const activeRooms = getActiveRooms();
diff --git a/package.json b/package.json
index af546db1..3d7fac71 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "mirotalksfu",
-    "version": "1.8.63",
+    "version": "1.8.64",
     "description": "WebRTC SFU browser-based video calls",
     "main": "Server.js",
     "scripts": {
@@ -57,11 +57,11 @@
         "node": ">=18"
     },
     "dependencies": {
-        "@aws-sdk/client-s3": "^3.826.0",
-        "@aws-sdk/lib-storage": "^3.826.0",
+        "@aws-sdk/client-s3": "^3.828.0",
+        "@aws-sdk/lib-storage": "^3.828.0",
         "@mattermost/client": "10.8.0",
         "@ngrok/ngrok": "1.5.1",
-        "@sentry/node": "^9.28.0",
+        "@sentry/node": "^9.28.1",
         "async-mutex": "^0.5.0",
         "axios": "^1.9.0",
         "chokidar": "^4.0.3",
diff --git a/public/js/Brand.js b/public/js/Brand.js
index 7666d0cd..5aa86454 100644
--- a/public/js/Brand.js
+++ b/public/js/Brand.js
@@ -64,7 +64,7 @@ let BRAND = {
     },
     about: {
         imageUrl: '../images/mirotalk-logo.gif',
-        title: '<strong>WebRTC SFU v1.8.63</strong>',
+        title: '<strong>WebRTC SFU v1.8.64</strong>',
         html: `
             <button 
                 id="support-button" 
diff --git a/public/js/Room.js b/public/js/Room.js
index d4f8b9c3..4c36c18d 100644
--- a/public/js/Room.js
+++ b/public/js/Room.js
@@ -11,7 +11,7 @@ if (location.href.substr(0, 5) !== 'https') location.href = 'https' + location.h
  * @license For commercial or closed source, contact us at license.mirotalk@gmail.com or purchase directly via CodeCanyon
  * @license CodeCanyon: https://codecanyon.net/item/mirotalk-sfu-webrtc-realtime-video-conferences/40769970
  * @author  Miroslav Pejic - miroslav.pejic.85@gmail.com
- * @version 1.8.63
+ * @version 1.8.64
  *
  */
 
@@ -34,9 +34,23 @@ const thisInfo = getInfo();
 const isEmbedded = window.self !== window.top;
 const showDocumentPipBtn = !isEmbedded && 'documentPictureInPicture' in window;
 
+/**
+ * Initializes a Socket.IO client instance with custom connection and reconnection options.
+ *
+ * @property {string[]} transports - The transport mechanisms to use. Default: ['polling', 'websocket']. Here, only ['websocket'] is used.
+ * @property {boolean} reconnection - Whether to automatically reconnect if connection is lost. Default: true.
+ * @property {number} reconnectionAttempts - Maximum number of reconnection attempts before giving up. Default: Infinity. Here, set to 10.
+ * @property {number} reconnectionDelay - How long to initially wait before attempting a new reconnection (in ms). Default: 1000. Here, set to 3000.
+ * @property {number} reconnectionDelayMax - Maximum amount of time to wait between reconnections (in ms). Default: 5000. Here, set to 15000.
+ * @property {number} timeout - Connection timeout before an error is emitted (in ms). Default: 20000.
+ */
 const socket = io({
     transports: ['websocket'],
-    reconnection: isDesktopDevice,
+    reconnection: true,
+    reconnectionAttempts: 10,
+    reconnectionDelay: 3000,
+    reconnectionDelayMax: 15000,
+    timeout: 20000,
 });
 
 let survey = {
@@ -202,6 +216,8 @@ const isMediaStreamTrackAndTransformerSupported = virtualBackground.checkSupport
 // DYNAMIC SETTINGS
 // ####################################################
 
+let preventExit = false;
+
 let virtualBackgroundBlurLevel;
 let virtualBackgroundSelectedImage;
 let virtualBackgroundTransparent;
@@ -2570,7 +2586,7 @@ function handleSelects() {
         localStorageSettings.dominant_speaker_focus = e.currentTarget.checked;
         lS.setSettings(localStorageSettings);
         e.target.blur();
-    }
+    };
     switchPushToTalk.onchange = async (e) => {
         const producerExist = rc.producerExist(RoomClient.mediaType.audio);
         if (!producerExist && !isPushToTalkActive) {
@@ -5402,8 +5418,6 @@ function showError(errorElement, message, delay = 5000) {
 // HANDLE SESSION EXIT
 // ####################################################
 
-let preventExit = false;
-
 // Call this when the session starts (e.g., after joining a room)
 function startRoomSession() {
     preventExit = true;
@@ -5447,7 +5461,7 @@ window.addEventListener('popstate', (event) => {
 
 // Intercept tab close, refresh, or direct URL navigation
 window.addEventListener('beforeunload', (e) => {
-    if (!preventExit) return;
+    if (!preventExit || window.localStorage.isReconnected === 'true') return;
     // Modern browsers ignore custom messages, but this triggers the prompt
     e.preventDefault();
     e.returnValue = '';
@@ -5465,7 +5479,7 @@ function showAbout() {
         position: 'center',
         imageUrl: BRAND.about?.imageUrl && BRAND.about.imageUrl.trim() !== '' ? BRAND.about.imageUrl : image.about,
         customClass: { image: 'img-about' },
-        title: BRAND.about?.title && BRAND.about.title.trim() !== '' ? BRAND.about.title : 'WebRTC SFU v1.8.63',
+        title: BRAND.about?.title && BRAND.about.title.trim() !== '' ? BRAND.about.title : 'WebRTC SFU v1.8.64',
         html: `
             <br />
             <div id="about">
diff --git a/public/js/RoomClient.js b/public/js/RoomClient.js
index 20f4a156..86192c89 100644
--- a/public/js/RoomClient.js
+++ b/public/js/RoomClient.js
@@ -9,7 +9,7 @@
  * @license For commercial or closed source, contact us at license.mirotalk@gmail.com or purchase directly via CodeCanyon
  * @license CodeCanyon: https://codecanyon.net/item/mirotalk-sfu-webrtc-realtime-video-conferences/40769970
  * @author  Miroslav Pejic - miroslav.pejic.85@gmail.com
- * @version 1.8.63
+ * @version 1.8.64
  *
  */
 
@@ -225,6 +225,18 @@ class RoomClient {
         transcription,
         successCallback
     ) {
+        this.room_id = room_id;
+        this.peer_id = socket.id;
+        this.peer_name = peer_name;
+        this.peer_uuid = peer_uuid;
+        this.peer_info = peer_info;
+        this.peer_avatar = peer_info.peer_avatar;
+
+        // Device type
+        this.isDesktopDevice = peer_info.is_desktop_device;
+        this.isMobileDevice = peer_info.is_mobile_device;
+        this.isMobileSafari = this.isMobileDevice && peer_info.browser_name.toLowerCase().includes('safari');
+
         this.localAudioEl = localAudioEl;
         this.remoteAudioEl = remoteAudioEl;
         this.videoMediaContainer = videoMediaContainer;
@@ -233,29 +245,18 @@ class RoomClient {
 
         // Handle Socket
         this.socket = socket;
-        this.reconnectAttempts = 0;
+        this.reconnectAlert = null;
         this.maxReconnectAttempts = 5;
         this.reconnectInterval = 3000;
         this.maxReconnectInterval = 15000;
-        this.silentReconnect = false;
+        this.serverAwayShown = false;
+        this.silentReconnect = false; // If true, no popup will be shown on reconnect
 
         // Handle ICE
         this.iceRestarting = false;
         this.iceProducerRestarting = false;
         this.iceConsumerRestarting = false;
 
-        this.room_id = room_id;
-        this.peer_id = socket.id;
-        this.peer_name = peer_name;
-        this.peer_avatar = peer_info.peer_avatar;
-        this.peer_uuid = peer_uuid;
-        this.peer_info = peer_info;
-
-        // Device type
-        this.isDesktopDevice = peer_info.is_desktop_device;
-        this.isMobileDevice = peer_info.is_mobile_device;
-        this.isMobileSafari = this.isMobileDevice && peer_info.browser_name.toLowerCase().includes('safari');
-
         // RTMP selected file name
         this.selectedRtmpFilename = '';
 
@@ -1062,8 +1063,12 @@ class RoomClient {
     // ####################################################
 
     initSockets() {
+        this.socket.io.on('reconnect_attempt', this.handleSocketReconnectAttempt);
+        this.socket.io.on('reconnect', this.handleSocketReconnect);
+        this.socket.io.on('reconnect_failed', this.handleSocketReconnectFailed);
         this.socket.on('connect', this.handleSocketConnect);
         this.socket.on('disconnect', this.handleSocketDisconnect);
+        this.socket.on('connect_error', this.handleSocketConnectionError);
         this.socket.on('consumerClosed', this.handleConsumerClosed);
         this.socket.on('setVideoOff', this.handleSetVideoOff);
         this.socket.on('removeMe', this.handleRemoveMe);
@@ -1104,7 +1109,6 @@ class RoomClient {
 
     handleSocketConnect = () => {
         console.log('SocketOn Connected to signaling server!');
-        this.handleConnect();
     };
 
     handleSocketDisconnect = (reason) => {
@@ -1112,6 +1116,25 @@ class RoomClient {
         this.handleDisconnect(reason);
     };
 
+    handleSocketConnectionError = (err) => {
+        console.log(`SocketOn Disconnect Error: ${err.message}`);
+    };
+
+    handleSocketReconnectAttempt = (attempt) => {
+        console.log(`SocketOn Reconnect Attempt: ${attempt}`);
+        this.handleReconnectAttempt(attempt);
+    };
+
+    handleSocketReconnect = () => {
+        console.log('SocketOn Reconnected to signaling server!');
+        this.handleReconnect();
+    };
+
+    handleSocketReconnectFailed = () => {
+        console.error('SocketOn Reconnect failed');
+        this.handleReconnectFailed();
+    };
+
     handleConsumerClosed = ({ consumer_id, consumer_kind }) => {
         console.log('SocketOn Closing consumer', { consumer_id, consumer_kind });
         this.removeConsumer(consumer_id, consumer_kind);
@@ -1303,146 +1326,109 @@ class RoomClient {
     };
 
     // ####################################################
-    // SOCKET CONNECT/DISCONNECT
+    // SOCKET RECONNECT/DISCONNECT
     // ####################################################
 
-    handleConnect() {
-        this._isConnected = true;
-        this.refreshBrowser();
+    showReconnectAlert(reason) {
+        if (this.silentReconnect) return;
+        this.reconnectAlert = Swal.fire({
+            background: swalBackground,
+            allowOutsideClick: false,
+            allowEscapeKey: false,
+            showDenyButton: false,
+            showConfirmButton: false,
+            position: 'top',
+            icon: 'warning',
+            title: 'Lost connection',
+            text: `${reason}, trying to reconnect...`,
+            showClass: { popup: 'animate__animated animate__fadeInDown' },
+            hideClass: { popup: 'animate__animated animate__fadeOutUp' },
+        });
+    }
+
+    showMaxAttemptsAlert() {
+        if (this.silentReconnect) return;
+        Swal.fire({
+            allowOutsideClick: false,
+            allowEscapeKey: false,
+            showDenyButton: false,
+            showConfirmButton: true,
+            background: swalBackground,
+            position: 'top',
+            icon: 'warning',
+            title: 'Unable to reconnect',
+            text: 'Please check your internet connection!',
+            icon: 'error',
+            confirmButtonText: 'Join Room',
+            confirmButtonColor: '#18392B',
+            showClass: { popup: 'animate__animated animate__fadeInDown' },
+            hideClass: { popup: 'animate__animated animate__fadeOutUp' },
+        }).then((result) => {
+            if (result.isConfirmed) {
+                this.refreshBrowser();
+            }
+        });
+    }
+
+    showServerAwayMessage() {
+        if (this.serverAwayShown) return;
+        this.serverAwayShown = true;
+        console.warn('Server away or in maintenance, please wait...');
+        this.ServerAway();
+        this.exit(true);
+    }
+
+    attemptReconnect(attempt) {
+        if (this._isConnected || attempt >= this.maxReconnectAttempts) return;
+
+        const delay = Math.min(this.reconnectInterval * attempt, this.maxReconnectInterval);
+
+        if (this.reconnectAlert) {
+            this.reconnectAlert.update({
+                title: 'Reconnecting',
+                text: `Reconnection attempt in ${delay / 1000} seconds...`,
+            });
+        }
     }
 
     handleDisconnect(reason) {
         endRoomSession();
 
         window.localStorage.isReconnected = true;
-
         console.log('Disconnected. Attempting to reconnect...');
-        this.exit(true);
 
-        let reconnectAlert = null;
+        // Immediately save recording if active
+        if (this.isRecording()) {
+            this.saveRecording('Socket disconnected');
+        }
 
-        // Helper functions
-        const showReconnectAlert = () => {
-            if (this.silentReconnect) return;
-            this.sound('reconnect');
-            reconnectAlert = Swal.fire({
-                background: swalBackground,
-                allowOutsideClick: false,
-                allowEscapeKey: false,
-                showDenyButton: false,
-                showConfirmButton: false,
-                icon: 'warning',
-                title: `Lost connection\n(${reason})`,
-                text: `The server may be under maintenance or your connection may have changed. Please ${this.isMobileDevice ? 'wait...' : 'check your connection.'}`,
-                showClass: { popup: 'animate__animated animate__fadeInDown' },
-                hideClass: { popup: 'animate__animated animate__fadeOutUp' },
-            });
-        };
+        this.serverAwayShown = false;
+        this._isConnected = false;
 
-        const showMaxAttemptsAlert = () => {
-            if (this.silentReconnect) return;
-            this.sound('alert');
-            Swal.fire({
-                allowOutsideClick: false,
-                allowEscapeKey: false,
-                showDenyButton: false,
-                showConfirmButton: true,
-                background: swalBackground,
-                title: 'Unable to reconnect',
-                text: 'Please check your internet connection!',
-                icon: 'error',
-                confirmButtonText: 'Join Room',
-                confirmButtonColor: '#18392B',
-                showClass: { popup: 'animate__animated animate__fadeInDown' },
-                hideClass: { popup: 'animate__animated animate__fadeOutUp' },
-            }).then((result) => {
-                if (result.isConfirmed) {
-                    this.refreshBrowser();
-                }
-            });
-        };
+        this.showReconnectAlert(reason);
+    }
 
-        const showServerAwayMessage = () => {
-            console.warn('Server away or in maintenance, please wait...');
-            this.ServerAway();
-        };
+    handleReconnectAttempt(attempt) {
+        attempt < this.maxReconnectAttempts ? this.attemptReconnect(attempt) : this.showServerAwayMessage();
+    }
 
-        // Show reconnect alert or save recording if necessary
-        this.isRecording() ? this.saveRecording('Socket disconnected') : showReconnectAlert();
+    handleReconnect() {
+        this._isConnected = true;
+        this.closeReconnectAlert();
+        this.refreshBrowser();
+    }
 
-        let serverAwayShown = false;
-        let reconnect;
-        let reconnectTimer;
-        let waitingTime = this.isMobileDevice ? 30000 : 6000;
+    handleReconnectFailed() {
+        if (!this._isConnected) {
+            this.closeReconnectAlert();
+            this.showMaxAttemptsAlert();
+        }
+    }
 
-        // Clear existing timers if any
-        const clearTimers = () => {
-            clearTimeout(reconnect);
-            clearTimeout(reconnectTimer);
-        };
-
-        // Handle connect_error events
-        this.socket.once('connect_error', () => {
-            clearTimers();
-        });
-        this.socket.on('connect_error', () => {
-            if (this.reconnectAttempts < this.maxReconnectAttempts) {
-                this.reconnectAttempts++;
-                if (reconnectAlert) {
-                    reconnectAlert.update({
-                        title: 'Reconnect',
-                        text: `Attempting to reconnect ${this.reconnectAttempts}...`,
-                    });
-                }
-            } else {
-                if (!serverAwayShown) {
-                    showServerAwayMessage();
-                    serverAwayShown = true;
-                }
-            }
-        });
-
-        // Attempt reconnect after delay
-        const attemptReconnect = async () => {
-            if (this.reconnectAttempts < this.maxReconnectAttempts) {
-                this.reconnectAttempts++;
-                console.log(`Reconnection attempt ${this.reconnectAttempts}...`);
-
-                const delay = Math.min(this.reconnectInterval * this.reconnectAttempts, this.maxReconnectInterval);
-
-                if (reconnectAlert) {
-                    reconnectAlert.update({
-                        title: 'Reconnect',
-                        text: `Attempting to reconnect in ${delay / 1000}s...`,
-                    });
-                }
-
-                reconnectTimer = setTimeout(() => {
-                    this.socket.connect();
-                    attemptReconnect();
-                }, delay);
-            } else {
-                console.log('Reconnection limit reached!');
-                if (reconnectAlert) {
-                    reconnectAlert.close();
-                }
-                showMaxAttemptsAlert();
-            }
-        };
-
-        // Handle successful reconnection
-        this.socket.once('connect', () => {
-            console.log('Reconnected!');
-            this.reconnectAttempts = 0;
-            startRoomSession();
-            clearTimers();
-        });
-
-        //Start reconnect logic after a brief delay
-        reconnect = setTimeout(() => {
-            showReconnectAlert();
-            attemptReconnect();
-        }, waitingTime);
+    closeReconnectAlert() {
+        if (this.reconnectAlert) {
+            this.reconnectAlert.close();
+        }
     }
 
     // ####################################################
@@ -1457,7 +1443,8 @@ class RoomClient {
             showDenyButton: false,
             showConfirmButton: false,
             background: swalBackground,
-            imageUrl: image.poster,
+            position: 'top',
+            icon: 'warning',
             title: 'Server away',
             text: 'The server seems away or in maintenance, please wait until it come back up.',
             denyButtonText: `Leave room`,
@@ -1470,13 +1457,48 @@ class RoomClient {
         });
     }
 
+    removePeerInfoFromLocalStorage() {
+        try {
+            localStorage.removeItem('sfu_peer_info');
+        } catch (e) {
+            console.warn('Unable to remove sfu_peer_info from localStorage:', e);
+        }
+    }
+
+    updatePeerInfoInLocalStorage() {
+        try {
+            localStorage.setItem('sfu_peer_info', JSON.stringify(this.peer_info));
+        } catch (e) {
+            console.warn('Unable to save peer_info to localStorage:', e);
+        }
+    }
+
+    getPeerInfoFromLocalStorage() {
+        try {
+            const sfu_peer_info = localStorage.getItem('sfu_peer_info');
+            return sfu_peer_info ? JSON.parse(sfu_peer_info) : null;
+        } catch (e) {
+            console.warn('Unable to get sfu_peer_info from localStorage:', e);
+            return null;
+        }
+    }
+
     refreshBrowser() {
-        this.exit(true);
-        getPeerName() ? location.reload() : openURL(this.getReconnectDirectJoinURL());
+        endRoomSession();
+        this.updatePeerInfoInLocalStorage();
+        const reconnectDirectJoinURL = this.getReconnectDirectJoinURL();
+        setTimeout(() => {
+            this.exit(true);
+            openURL(reconnectDirectJoinURL);
+            this.removePeerInfoFromLocalStorage();
+        }, 100);
     }
 
     getReconnectDirectJoinURL() {
-        const { peer_audio, peer_video, peer_screen, peer_token } = this.peer_info;
+        const sfu_peer_info = this.getPeerInfoFromLocalStorage();
+        const { peer_presenter, peer_audio, peer_video, peer_screen, peer_token } = sfu_peer_info
+            ? sfu_peer_info
+            : this.peer_info;
         const baseUrl = `${window.location.origin}/join`;
         const queryParams = {
             room: this.room_id,
@@ -1486,7 +1508,7 @@ class RoomClient {
             video: peer_video,
             screen: peer_screen,
             notify: 0,
-            isPresenter: isPresenter || false,
+            isPresenter: peer_presenter || isPresenter,
         };
         if (peer_token) queryParams.token = peer_token;
         const url = `${baseUrl}?${Object.entries(queryParams)
@@ -1608,9 +1630,7 @@ class RoomClient {
             await this.produce(mediaType.screen, null, false, true);
             console.log('11 ----> START SCREEN MEDIA');
         }
-        // if (this.isScreenAllowed) {
-        //     this.shareScreen();
-        // }
+
         console.log('[startLocalMedia] - PRODUCER LABEL', this.producerLabel);
     }
 
@@ -1639,17 +1659,18 @@ class RoomClient {
             case mediaType.audio:
                 this.isAudioAllowed = true;
                 mediaConstraints = this.getAudioConstraints(deviceId);
+                this.peer_info.peer_audio = true;
                 audio = true;
                 break;
             case mediaType.video:
                 this.isVideoAllowed = true;
-                swapCamera
-                    ? (mediaConstraints = this.getCameraConstraints())
-                    : (mediaConstraints = this.getVideoConstraints(deviceId));
+                mediaConstraints = swapCamera ? this.getCameraConstraints() : this.getVideoConstraints(deviceId);
+                this.peer_info.peer_video = true;
                 video = true;
                 break;
             case mediaType.screen:
                 mediaConstraints = this.getScreenConstraints();
+                this.peer_info.peer_screen = true;
                 screen = true;
                 break;
             default:
@@ -2601,7 +2622,6 @@ class RoomClient {
         switch (type) {
             case mediaType.audio:
                 this.event(_EVENTS.pauseAudio);
-                this.setIsAudio(this.peer_id, false);
                 break;
             case mediaType.video:
                 this.event(_EVENTS.pauseVideo);
@@ -2632,7 +2652,6 @@ class RoomClient {
         switch (type) {
             case mediaType.audio:
                 this.event(_EVENTS.resumeAudio);
-                this.setIsAudio(this.peer_id, true);
                 break;
             case mediaType.video:
                 this.event(_EVENTS.resumeVideo);
@@ -2710,7 +2729,6 @@ class RoomClient {
             default:
                 break;
         }
-
         this.sound('left');
     }
 
@@ -9186,7 +9204,7 @@ class RoomClient {
                     break;
                 case 'hand':
                     this.peer_info.peer_hand = status;
-                    let peer_hand = this.getPeerHandBtn(peer_id);
+                    const peer_hand = this.getPeerHandBtn(peer_id);
                     if (status) {
                         if (peer_hand) peer_hand.style.display = 'flex';
                         this.event(_EVENTS.raiseHand);
@@ -9219,7 +9237,7 @@ class RoomClient {
                 case 'screen':
                     break;
                 case 'hand':
-                    let peer_hand = this.getPeerHandBtn(peer_id);
+                    const peer_hand = this.getPeerHandBtn(peer_id);
                     if (status) {
                         if (peer_hand) peer_hand.style.display = 'flex';
                         this.userLog(
diff --git a/public/views/Room.html b/public/views/Room.html
index 624000e3..6566b40d 100644
--- a/public/views/Room.html
+++ b/public/views/Room.html
@@ -730,7 +730,11 @@ access to use this app.
                                     </td>
                                     <td>
                                         <div class="form-check form-switch form-switch-md title">
-                                            <input id="switchDominantSpeakerFocus" class="form-check-input" type="checkbox" />
+                                            <input
+                                                id="switchDominantSpeakerFocus"
+                                                class="form-check-input"
+                                                type="checkbox"
+                                            />
                                         </div>
                                     </td>
                                 </tr>