LH-265: Update client

README.md

@@ -22,17 +22,20 @@
 2. Run the `npm start:prod` command to start the server in production mode.
 (To connect to the terminal, use `pm2 log video-server`)
 
----
+
+### Web client
 
 - The server will start by default on port 3000, and the ssl certificates will have to be configured
 - The web client can be accessed using the /sfu path
-ex: http://localhost:3000/sfu/?assetId=1&&accountId=1&producer=true&assetName=Adi&assetType=linx
+ex: https://HOST/sfu/?assetId=1&&accountId=1&producer=true&dest_asset_id=75&assetName=Adi
 assetId = asset id of the unit on which you are doing the test
 accountId = account id of the unit on which you are doing the test
 producer = it will always be true because you are the producer
 (it's possible to put false, but then you have to have another client with producer true)
 assetName = asset name of the unit on which you are doing the test
-assetType = asset type of the unit on which you are doing the test
+dest_asset_id= the addressee with whom the call is made
+- To make a call using this client, you need a microphone and permission to use it
+- For any changes related to the client, the command `npm run watch' will have to be used to generate the bundle.js used by the web client
 
 ### Demo project
   The demo project used initially and then modified for our needs `https://github.com/jamalag/mediasoup2`

app.js

@@ -364,83 +364,31 @@ peers.on('connection', async socket => {
       const callId = socketDetails[socket.id];
       console.log('[consume] callId', callId);
 
-      // Check if the router can consume the specified producer
-      if (
-        videoCalls[callId].router.canConsume({
-          producerId: videoCalls[callId].producerVideo.id,
-          rtpCapabilities
-        }) &&
-        videoCalls[callId].router.canConsume({
-          producerId: videoCalls[callId].producerAudio.id,
-          rtpCapabilities
-        })
-      ) {
-        console.log('[consume] Can consume', callId);
-        // Transport can now consume and return a consumer
-        videoCalls[callId].consumerVideo = await videoCalls[callId].consumerTransport.consume({
-          producerId: videoCalls[callId].producerVideo.id,
-          rtpCapabilities,
-          paused: true,
-        });
+      const canConsumeVideo = !!videoCalls[callId].producerVideo && !!videoCalls[callId].router.canConsume({
+        producerId: videoCalls[callId].producerVideo.id,
+        rtpCapabilities
+      })
 
-        // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-transportclose
-        videoCalls[callId].consumerVideo.on('transportclose', () => {
-          const callId = socketDetails[socket.id];
-          console.log('transport close from consumer', callId);
-          closeCall(callId);
-        });
+      const canConsumeAudio = !!videoCalls[callId].producerAudio && !!videoCalls[callId].router.canConsume({
+        producerId: videoCalls[callId].producerAudio.id,
+        rtpCapabilities
+      })
 
-        // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-producerclose
-        videoCalls[callId].consumerVideo.on('producerclose', () => {
-          const callId = socketDetails[socket.id];
-          console.log('producer of consumer closed', callId);
-          closeCall(callId);
-        });
+      console.log('[consume] canConsumeVideo', canConsumeVideo);
+      console.log('[consume] canConsumeAudio', canConsumeAudio);
 
-        videoCalls[callId].consumerAudio = await videoCalls[callId].consumerTransport.consume({
-          producerId: videoCalls[callId].producerAudio.id,
-          rtpCapabilities,
-          paused: true,
-        });
-
-        // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-transportclose
-        videoCalls[callId].consumerAudio.on('transportclose', () => {
-          const callId = socketDetails[socket.id];
-          console.log('transport close from consumer', callId);
-          closeCall(callId);
-        });
-
-        // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-producerclose
-        videoCalls[callId].consumerAudio.on('producerclose', () => {
-          const callId = socketDetails[socket.id];
-          console.log('producer of consumer closed', callId);
-          closeCall(callId);
-        });
-
-        console.log('🟠 videoCalls[callId].consumerVideo', videoCalls[callId].consumerVideo);
-        console.log('🟠 videoCalls[callId].consumerAudio', videoCalls[callId].consumerAudio);
-
-        // From the consumer extract the following params to send back to the Client
-        const videoParams = {
-          id: videoCalls[callId].consumerVideo.id,
-          producerId: videoCalls[callId].producerVideo.id,
-          kind: 'video',
-          rtpParameters: videoCalls[callId].consumerVideo.rtpParameters,
-        };
-
-        const audioParams = {
-          id: videoCalls[callId].consumerAudio.id,
-          producerId: videoCalls[callId].producerAudio.id,
-          kind: 'audio',
-          rtpParameters: videoCalls[callId].consumerAudio.rtpParameters,
-        };
-
-        console.log('[consume] videoParams', videoParams);
-        console.log('[consume] audioParams', audioParams);
-        // Send the parameters to the client
+      if (canConsumeVideo && !canConsumeAudio) {
+        console.log('1');
+        const videoParams = await consumeVideo(callId, rtpCapabilities)
+        console.log('videoParams', videoParams);
+        callback({ videoParams, audioParams: null });
+      } else if (canConsumeVideo && canConsumeAudio) {
+        console.log('2');
+        const videoParams = await consumeVideo(callId, rtpCapabilities)
+        const audioParams = await consumeAudio(callId, rtpCapabilities)
         callback({ videoParams, audioParams });
       } else {
-        console.log(`[canConsume] Can't consume | callId ${callId}`);
+        console.log(`[consume] Can't consume | callId ${callId}`);
         callback(null);
       }
     } catch (error) {
@@ -465,6 +413,63 @@ peers.on('connection', async socket => {
   });
 });
 
+const consumeVideo = async (callId, rtpCapabilities) => {
+  videoCalls[callId].consumerVideo = await videoCalls[callId].consumerTransport.consume({
+    producerId: videoCalls[callId].producerVideo.id,
+    rtpCapabilities,
+    paused: true,
+  });
+
+  // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-transportclose
+  videoCalls[callId].consumerVideo.on('transportclose', () => {
+    const callId = socketDetails[socket.id];
+    console.log('transport close from consumer', callId);
+    closeCall(callId);
+  });
+
+  // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-producerclose
+  videoCalls[callId].consumerVideo.on('producerclose', () => {
+    const callId = socketDetails[socket.id];
+    console.log('producer of consumer closed', callId);
+    closeCall(callId);
+  });
+
+  return {
+    id: videoCalls[callId].consumerVideo.id,
+    producerId: videoCalls[callId].producerVideo.id,
+    kind: 'video',
+    rtpParameters: videoCalls[callId].consumerVideo.rtpParameters,
+  }
+}
+
+const consumeAudio = async (callId, rtpCapabilities) => {
+  videoCalls[callId].consumerAudio = await videoCalls[callId].consumerTransport.consume({
+    producerId: videoCalls[callId].producerAudio.id,
+    rtpCapabilities,
+    paused: true,
+  });
+
+  // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-transportclose
+  videoCalls[callId].consumerAudio.on('transportclose', () => {
+    const callId = socketDetails[socket.id];
+    console.log('transport close from consumer', callId);
+    closeCall(callId);
+  });
+
+  // https://mediasoup.org/documentation/v3/mediasoup/api/#consumer-on-producerclose
+  videoCalls[callId].consumerAudio.on('producerclose', () => {
+    const callId = socketDetails[socket.id];
+    console.log('producer of consumer closed', callId);
+    closeCall(callId);
+  });
+  return {
+    id: videoCalls[callId].consumerAudio.id,
+    producerId: videoCalls[callId].producerAudio.id,
+    kind: 'audio',
+    rtpParameters: videoCalls[callId].consumerAudio.rtpParameters,
+  }
+}
+
 /*
   - Called from at event 'createWebRtcTransport' and assigned to the consumer or producer transport
   - It will return parameters, these are required for the client to create the RecvTransport 

public/bundle.js

@@ -20373,6 +20373,36 @@ console.log('[URL] ASSET_ID', ASSET_ID, '| ACCOUNT_ID', ACCOUNT_ID, '| callId',
 console.log('🟩 config', config)
 
 let socket, hub
+let device
+let rtpCapabilities
+let producerTransport
+let consumerTransport
+let producerVideo
+let producerAudio
+let consumer
+let originAssetId
+
+// https://mediasoup.org/documentation/v3/mediasoup-client/api/#ProducerOptions
+// https://mediasoup.org/documentation/v3/mediasoup-client/api/#transport-produce
+let videoParams = {
+  encodings: [
+    { scaleResolutionDownBy: 4, maxBitrate: 500000 },
+    { scaleResolutionDownBy: 2, maxBitrate: 1000000 },
+    { scaleResolutionDownBy: 1, maxBitrate: 5000000 },
+    { scalabilityMode: 'S3T3_KEY' }
+  ],
+  codecOptions: {
+    videoGoogleStartBitrate: 1000
+  }
+}
+
+let audioParams = {
+  codecOptions :
+  {
+    opusStereo : true,
+    opusDtx    : true
+  }
+}
 
 setTimeout(() => {
   hub = io(config.hubAddress)
@@ -20448,60 +20478,6 @@ setTimeout(() => {
 
 }, 1600);
 
-
-let device
-let rtpCapabilities
-let producerTransport
-let consumerTransport
-let producerVideo
-let producerAudio
-let consumer
-let originAssetId
-
-// https://mediasoup.org/documentation/v3/mediasoup-client/api/#ProducerOptions
-// https://mediasoup.org/documentation/v3/mediasoup-client/api/#transport-produce
-let videoParams = {
-  encodings: [
-    { scaleResolutionDownBy: 4, maxBitrate: 500000 },
-    { scaleResolutionDownBy: 2, maxBitrate: 1000000 },
-    { scaleResolutionDownBy: 1, maxBitrate: 5000000 },
-    { scalabilityMode: 'S3T3_KEY' }
-  ],
-  codecOptions: {
-    videoGoogleStartBitrate: 1000
-  }
-  // encodings: [
-  //   {
-  //     rid: 'r0',
-  //     maxBitrate: 100000,
-  //     scalabilityMode: 'S1T3',
-  //   },
-  //   {
-  //     rid: 'r1',
-  //     maxBitrate: 300000,
-  //     scalabilityMode: 'S1T3',
-  //   },
-  //   {
-  //     rid: 'r2',
-  //     maxBitrate: 900000,
-  //     scalabilityMode: 'S1T3',
-  //   },
-  // ],
-  // // https://mediasoup.org/documentation/v3/mediasoup-client/api/#ProducerCodecOptions
-  // codecOptions: {
-  //   videoGoogleStartBitrate: 1000
-  // }
-}
-
-let audioParams = {
-  codecOptions :
-  {
-    opusStereo : true,
-    opusDtx    : true
-  }
-}
-
-
 const streamSuccess = (stream) => {
   console.log('[streamSuccess] device', device);
   localVideo.srcObject = stream
@@ -20511,8 +20487,6 @@ const streamSuccess = (stream) => {
 
   videoParams = {
     track: videoTrack,
-    // codec : device.rtpCapabilities.codecs.find((codec) => codec.mimeType.toLowerCase() === 'video/vp9'),
-    // codec : 'video/vp9',
     ...videoParams
   }
 
@@ -20523,14 +20497,6 @@ const streamSuccess = (stream) => {
 
   console.log('[streamSuccess] videoParams', videoParams, ' | audioParams', audioParams);
   goConnect()
-  // console.log('[streamSuccess]');
-  // localVideo.srcObject = stream
-  // const track = stream.getVideoTracks()[0]
-  // videoParams = {
-  //   track,
-  //   ...videoParams
-  // }
-  // goConnect()
 }
 
 const getLocalStream = () => {
@@ -20543,23 +20509,18 @@ const getLocalStream = () => {
       hd   : { width: { ideal: 1280 }, height: { ideal: 720 } }
     }
   })
-  // navigator.mediaDevices.getUserMedia({
-  //   audio: false,
-  //   video: {
-  //     width: {
-  //       min: 640,
-  //       max: 1920,
-  //     },
-  //     height: {
-  //       min: 400,
-  //       max: 1080,
-  //     }
-  //   }
-  // })
   .then(streamSuccess)
   .catch(error => {
     console.log(error.message)
   })
+
+  navigator.permissions.query(
+    { name: 'microphone' }
+  ).then((permissionStatus) =>{
+    console.log('🟨 [PERMISSION] permissionStatus', permissionStatus); // granted, denied, prompt
+    // It will block the code from execution and display "Permission denied" if we don't have microphone permissions
+  })
+
 }
 
 const goConnect = () => {
@@ -20576,7 +20537,6 @@ const goCreateTransport = () => {
 // server side to send/recive media
 const createDevice = async () => {
   try {
-    console.log('[createDevice] 1 device', device);
     device = new mediasoupClient.Device()
 
     // https://mediasoup.org/documentation/v3/mediasoup-client/api/#device-load
@@ -20587,7 +20547,7 @@ const createDevice = async () => {
     })
 
     console.log('Device RTP Capabilities', device.rtpCapabilities)
-    console.log('[createDevice] 2 device', device);
+    console.log('[createDevice] device', device);
     
     // once the device loads, create transport
     goCreateTransport()
@@ -20659,7 +20619,7 @@ const createSendTransport = () => {
       console.log('[produce] parameters', parameters)
 
       try {
-        // tell the server to create a Producer
+        // Tell the server to create a Producer
         // with the following parameters and produce
         // and expect back a server side producer id
         // see server's socket.on('transport-produce', ...)
@@ -20685,14 +20645,16 @@ const connectSendTransport = async () => {
 
   console.log('[connectSendTransport] producerTransport');
 
-  // we now call produce() to instruct the producer transport
+  // We now call produce() to instruct the producer transport
   // to send media to the Router
   // https://mediasoup.org/documentation/v3/mediasoup-client/api/#transport-produce
   // this action will trigger the 'connect' and 'produce' events above
   
-  console.log('videoParams', videoParams);
+  // Produce video
   producerVideo = await producerTransport.produce(videoParams)
+  console.log('videoParams', videoParams);
   console.log('producerVideo', producerVideo);
+
   producerVideo.on('trackended', () => {
     console.log('track ended')
     // close video track
@@ -20703,24 +20665,20 @@ const connectSendTransport = async () => {
     // close video track
   })    
   
-    console.log('audioParams', audioParams);
-    producerAudio = await producerTransport.produce(audioParams)
-    console.log('producerAudio', producerAudio);
-    producerAudio.on('trackended', () => {
-      console.log('track ended')
-      // close video track
-    })
-  
-    producerAudio.on('transportclose', () => {
-      console.log('transport ended')
-  
-      // close video track
-    })
-  
-
-
+  // Produce audio
+  producerAudio = await producerTransport.produce(audioParams)
+  console.log('audioParams', audioParams);
+  console.log('producerAudio', producerAudio);
 
+  producerAudio.on('trackended', () => {
+    console.log('track ended')
+    // close audio track
+  })
 
+  producerAudio.on('transportclose', () => {
+    console.log('transport ended')
+    // close audio track
+  })
 
   const answer = {
     origin_asset_id: ASSET_ID,
@@ -20730,7 +20688,7 @@ const connectSendTransport = async () => {
     origin_asset_type_name: ASSET_TYPE,
     origin_asset_name: ASSET_NAME,
     video_call_id: callId,
-    answer: 'accepted', // answer: 'rejected'
+    answer: 'accepted', // answer: accepted/rejected
   };
   console.log('SEND answer', answer);
 
@@ -20746,7 +20704,7 @@ const connectSendTransport = async () => {
 
 const createRecvTransport = async () => {
   console.log('createRecvTransport');
-  // see server's socket.on('consume', sender?, ...)
+  // See server's socket.on('consume', sender?, ...)
   // this is a call from Consumer, so sender = false
   await socket.emit('createWebRtcTransport', { sender: false, callId }, ({ params }) => {
     // The server sends back params needed 
@@ -20758,13 +20716,13 @@ const createRecvTransport = async () => {
 
     console.log('[createRecvTransport] params', params)
 
-    // creates a new WebRTC Transport to receive media
+    // Creates a new WebRTC Transport to receive media
     // based on server's consumer transport params
     // https://mediasoup.org/documentation/v3/mediasoup-client/api/#device-createRecvTransport
     consumerTransport = device.createRecvTransport(params)
 
     // https://mediasoup.org/documentation/v3/communication-between-client-and-server/#producing-media
-    // this event is raised when a first call to transport.produce() is made
+    // This event is raised when a first call to transport.produce() is made
     // see connectRecvTransport() below
     consumerTransport.on('connect', async ({ dtlsParameters }, callback, errback) => {
       try {
@@ -20798,7 +20756,7 @@ const resetCallSettings = () => {
 
 const connectRecvTransport = async () => {
   console.log('connectRecvTransport');
-  // for consumer, we need to tell the server first
+  // For consumer, we need to tell the server first
   // to create a consumer based on the rtpCapabilities and consume
   // if the router can consume, it will send back a set of params as below
   await socket.emit('consume', {
@@ -20810,7 +20768,7 @@ const connectRecvTransport = async () => {
       return
     }
 
-    // then consume with the local consumer transport
+    // Then consume with the local consumer transport
     // which creates a consumer
     consumer = await consumerTransport.consume({
       id: params.id,
@@ -20853,6 +20811,7 @@ const closeCall = () => {
   resetCallSettings()
 }
 
+
 btnLocalVideo.addEventListener('click', getLocalStream)
 btnRecvSendTransport.addEventListener('click', goConnect)
 btnCloseCall.addEventListener('click', closeCall)

public/index.js

@@ -15,6 +15,36 @@ console.log('[URL] ASSET_ID', ASSET_ID, '| ACCOUNT_ID', ACCOUNT_ID, '| callId',
 console.log('🟩 config', config)
 
 let socket, hub
+let device
+let rtpCapabilities
+let producerTransport
+let consumerTransport
+let producerVideo
+let producerAudio
+let consumer
+let originAssetId
+
+// https://mediasoup.org/documentation/v3/mediasoup-client/api/#ProducerOptions
+// https://mediasoup.org/documentation/v3/mediasoup-client/api/#transport-produce
+let videoParams = {
+  encodings: [
+    { scaleResolutionDownBy: 4, maxBitrate: 500000 },
+    { scaleResolutionDownBy: 2, maxBitrate: 1000000 },
+    { scaleResolutionDownBy: 1, maxBitrate: 5000000 },
+    { scalabilityMode: 'S3T3_KEY' }
+  ],
+  codecOptions: {
+    videoGoogleStartBitrate: 1000
+  }
+}
+
+let audioParams = {
+  codecOptions :
+  {
+    opusStereo : true,
+    opusDtx    : true
+  }
+}
 
 setTimeout(() => {
   hub = io(config.hubAddress)
@@ -90,60 +120,6 @@ setTimeout(() => {
 
 }, 1600);
 
-
-let device
-let rtpCapabilities
-let producerTransport
-let consumerTransport
-let producerVideo
-let producerAudio
-let consumer
-let originAssetId
-
-// https://mediasoup.org/documentation/v3/mediasoup-client/api/#ProducerOptions
-// https://mediasoup.org/documentation/v3/mediasoup-client/api/#transport-produce
-let videoParams = {
-  encodings: [
-    { scaleResolutionDownBy: 4, maxBitrate: 500000 },
-    { scaleResolutionDownBy: 2, maxBitrate: 1000000 },
-    { scaleResolutionDownBy: 1, maxBitrate: 5000000 },
-    { scalabilityMode: 'S3T3_KEY' }
-  ],
-  codecOptions: {
-    videoGoogleStartBitrate: 1000
-  }
-  // encodings: [
-  //   {
-  //     rid: 'r0',
-  //     maxBitrate: 100000,
-  //     scalabilityMode: 'S1T3',
-  //   },
-  //   {
-  //     rid: 'r1',
-  //     maxBitrate: 300000,
-  //     scalabilityMode: 'S1T3',
-  //   },
-  //   {
-  //     rid: 'r2',
-  //     maxBitrate: 900000,
-  //     scalabilityMode: 'S1T3',
-  //   },
-  // ],
-  // // https://mediasoup.org/documentation/v3/mediasoup-client/api/#ProducerCodecOptions
-  // codecOptions: {
-  //   videoGoogleStartBitrate: 1000
-  // }
-}
-
-let audioParams = {
-  codecOptions :
-  {
-    opusStereo : true,
-    opusDtx    : true
-  }
-}
-
-
 const streamSuccess = (stream) => {
   console.log('[streamSuccess] device', device);
   localVideo.srcObject = stream
@@ -153,8 +129,6 @@ const streamSuccess = (stream) => {
 
   videoParams = {
     track: videoTrack,
-    // codec : device.rtpCapabilities.codecs.find((codec) => codec.mimeType.toLowerCase() === 'video/vp9'),
-    // codec : 'video/vp9',
     ...videoParams
   }
 
@@ -165,14 +139,6 @@ const streamSuccess = (stream) => {
 
   console.log('[streamSuccess] videoParams', videoParams, ' | audioParams', audioParams);
   goConnect()
-  // console.log('[streamSuccess]');
-  // localVideo.srcObject = stream
-  // const track = stream.getVideoTracks()[0]
-  // videoParams = {
-  //   track,
-  //   ...videoParams
-  // }
-  // goConnect()
 }
 
 const getLocalStream = () => {
@@ -185,23 +151,18 @@ const getLocalStream = () => {
       hd   : { width: { ideal: 1280 }, height: { ideal: 720 } }
     }
   })
-  // navigator.mediaDevices.getUserMedia({
-  //   audio: false,
-  //   video: {
-  //     width: {
-  //       min: 640,
-  //       max: 1920,
-  //     },
-  //     height: {
-  //       min: 400,
-  //       max: 1080,
-  //     }
-  //   }
-  // })
   .then(streamSuccess)
   .catch(error => {
     console.log(error.message)
   })
+
+  navigator.permissions.query(
+    { name: 'microphone' }
+  ).then((permissionStatus) =>{
+    console.log('🟨 [PERMISSION] permissionStatus', permissionStatus); // granted, denied, prompt
+    // It will block the code from execution and display "Permission denied" if we don't have microphone permissions
+  })
+
 }
 
 const goConnect = () => {
@@ -218,7 +179,6 @@ const goCreateTransport = () => {
 // server side to send/recive media
 const createDevice = async () => {
   try {
-    console.log('[createDevice] 1 device', device);
     device = new mediasoupClient.Device()
 
     // https://mediasoup.org/documentation/v3/mediasoup-client/api/#device-load
@@ -229,7 +189,7 @@ const createDevice = async () => {
     })
 
     console.log('Device RTP Capabilities', device.rtpCapabilities)
-    console.log('[createDevice] 2 device', device);
+    console.log('[createDevice] device', device);
     
     // once the device loads, create transport
     goCreateTransport()
@@ -301,7 +261,7 @@ const createSendTransport = () => {
       console.log('[produce] parameters', parameters)
 
       try {
-        // tell the server to create a Producer
+        // Tell the server to create a Producer
         // with the following parameters and produce
         // and expect back a server side producer id
         // see server's socket.on('transport-produce', ...)
@@ -327,14 +287,16 @@ const connectSendTransport = async () => {
 
   console.log('[connectSendTransport] producerTransport');
 
-  // we now call produce() to instruct the producer transport
+  // We now call produce() to instruct the producer transport
   // to send media to the Router
   // https://mediasoup.org/documentation/v3/mediasoup-client/api/#transport-produce
   // this action will trigger the 'connect' and 'produce' events above
   
-  console.log('videoParams', videoParams);
+  // Produce video
   producerVideo = await producerTransport.produce(videoParams)
+  console.log('videoParams', videoParams);
   console.log('producerVideo', producerVideo);
+
   producerVideo.on('trackended', () => {
     console.log('track ended')
     // close video track
@@ -345,24 +307,20 @@ const connectSendTransport = async () => {
     // close video track
   })    
   
-    console.log('audioParams', audioParams);
-    producerAudio = await producerTransport.produce(audioParams)
-    console.log('producerAudio', producerAudio);
-    producerAudio.on('trackended', () => {
-      console.log('track ended')
-      // close video track
-    })
-  
-    producerAudio.on('transportclose', () => {
-      console.log('transport ended')
-  
-      // close video track
-    })
-  
-
-
+  // Produce audio
+  producerAudio = await producerTransport.produce(audioParams)
+  console.log('audioParams', audioParams);
+  console.log('producerAudio', producerAudio);
 
+  producerAudio.on('trackended', () => {
+    console.log('track ended')
+    // close audio track
+  })
 
+  producerAudio.on('transportclose', () => {
+    console.log('transport ended')
+    // close audio track
+  })
 
   const answer = {
     origin_asset_id: ASSET_ID,
@@ -372,7 +330,7 @@ const connectSendTransport = async () => {
     origin_asset_type_name: ASSET_TYPE,
     origin_asset_name: ASSET_NAME,
     video_call_id: callId,
-    answer: 'accepted', // answer: 'rejected'
+    answer: 'accepted', // answer: accepted/rejected
   };
   console.log('SEND answer', answer);
 
@@ -388,7 +346,7 @@ const connectSendTransport = async () => {
 
 const createRecvTransport = async () => {
   console.log('createRecvTransport');
-  // see server's socket.on('consume', sender?, ...)
+  // See server's socket.on('consume', sender?, ...)
   // this is a call from Consumer, so sender = false
   await socket.emit('createWebRtcTransport', { sender: false, callId }, ({ params }) => {
     // The server sends back params needed 
@@ -400,13 +358,13 @@ const createRecvTransport = async () => {
 
     console.log('[createRecvTransport] params', params)
 
-    // creates a new WebRTC Transport to receive media
+    // Creates a new WebRTC Transport to receive media
     // based on server's consumer transport params
     // https://mediasoup.org/documentation/v3/mediasoup-client/api/#device-createRecvTransport
     consumerTransport = device.createRecvTransport(params)
 
     // https://mediasoup.org/documentation/v3/communication-between-client-and-server/#producing-media
-    // this event is raised when a first call to transport.produce() is made
+    // This event is raised when a first call to transport.produce() is made
     // see connectRecvTransport() below
     consumerTransport.on('connect', async ({ dtlsParameters }, callback, errback) => {
       try {
@@ -440,7 +398,7 @@ const resetCallSettings = () => {
 
 const connectRecvTransport = async () => {
   console.log('connectRecvTransport');
-  // for consumer, we need to tell the server first
+  // For consumer, we need to tell the server first
   // to create a consumer based on the rtpCapabilities and consume
   // if the router can consume, it will send back a set of params as below
   await socket.emit('consume', {
@@ -452,7 +410,7 @@ const connectRecvTransport = async () => {
       return
     }
 
-    // then consume with the local consumer transport
+    // Then consume with the local consumer transport
     // which creates a consumer
     consumer = await consumerTransport.consume({
       id: params.id,
@@ -495,6 +453,7 @@ const closeCall = () => {
   resetCallSettings()
 }
 
+
 btnLocalVideo.addEventListener('click', getLocalStream)
 btnRecvSendTransport.addEventListener('click', goConnect)
 btnCloseCall.addEventListener('click', closeCall)