LAPI-675: Comment code<DEBUG>

Reviewed-on: #23
Reviewed-by: Cristi Ene <cristi.ene@safemobile.com>

app.js

@@ -28,8 +28,8 @@ let worker;
  *  initiatorConsumerVideo: Consumer,
  *  initiatorConsumerAudio: Consumer,
  *  initiatorConsumerTransport: Consumer Transport
- *  initiatorSockerId
- *  receiverSocketId
+ *  initiatorSocket
+ *  receiverSocket
  * }
  * 
  **/
@@ -193,11 +193,11 @@ peers.on('connection', async socket => {
         console.log(`[createRoom] socket.id ${socket.id} callId ${callId}`);
         if (!videoCalls[callId]) {
           console.log('[createRoom] callId', callId);
-          videoCalls[callId] = { router: await worker.createRouter({ mediaCodecs }) }
+          videoCalls[callId] = { router: await worker.createRouter({ mediaCodecs }) };
           console.log(`[createRoom] Router ID: ${videoCalls[callId].router.id}`);
-          videoCalls[callId].receiverSocketId = socket.id
+          videoCalls[callId].receiverSocket = socket;
         } else {
-          videoCalls[callId].initiatorSockerId = socket.id
+          videoCalls[callId].initiatorSocket = socket;
         }
         socketDetails[socket.id] = callId;
         // rtpCapabilities is set for callback
@@ -280,6 +280,72 @@ peers.on('connection', async socket => {
       if (typeof rtpParameters === 'string') rtpParameters = JSON.parse(rtpParameters);
       
       console.log(`[transport-produce] kind: ${kind} | socket: ${socket.id} | callId: ${callId}`);
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+      // const router = videoCalls[callId].router;
+
+      // router.observer.on("newtransport", (transport) => {
+      //   transport.observer.on("newproducer", (producer) => {
+
+      //     console.log(
+      //       "new producer created [worker.pid:%d, router.id:%s, transport.id:%s, producer.id:%s]",
+      //       worker.pid, router.id, transport.id, producer.id
+      //     );
+
+      //     if (!isInitiator(callId, socket.id)) {
+      //       console.log('🟥 Emit to initiatorSocket');
+      //       initiatorSocket.emit('new-producer', {
+      //         callId
+      //       });
+
+      //     } else {
+      //       console.log('🟥 Emit to receiverSocket');
+      //       receiverSocket.emit('new-producer', {
+      //         callId
+      //       });
+      //     }
+      //   });
+      
+      // });
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+      
       if (kind === 'video') {
         if (!isInitiator(callId, socket.id)) {
           videoCalls[callId].receiverVideoProducer = await videoCalls[callId].receiverProducerTransport.produce({
@@ -369,6 +435,7 @@ peers.on('connection', async socket => {
     try {
       const callId = socketDetails[socket.id];
       console.log(`[transport-recv-connect] socket ${socket.id} | callId ${callId}`);
+      if (typeof dtlsParameters === 'string') dtlsParameters = JSON.parse(dtlsParameters);
       // await videoCalls[callId].consumerTransport.connect({ dtlsParameters });
       if(!isInitiator(callId, socket.id)) {
         await videoCalls[callId].receiverConsumerTransport.connect({ dtlsParameters });
@@ -392,6 +459,8 @@ peers.on('connection', async socket => {
       const callId = socketDetails[socket.id];
       console.log(`[consume] socket ${socket.id} | callId ${callId} | rtpCapabilities: ${JSON.stringify(rtpCapabilities)}`);
 
+      if (typeof rtpCapabilities === 'string') rtpCapabilities = JSON.parse(rtpCapabilities);
+
       console.log('[consume] callId', callId);
       let canConsumeVideo, canConsumeAudio;
       if (isInitiator(callId, socket.id)) {
@@ -442,20 +511,18 @@ peers.on('connection', async socket => {
   /*
     - Event sent by the consumer after consuming to resume the pause
     - When consuming on consumerTransport, it is initially done with paused: true, here we will resume
+    - For the initiator we resume the initiatorConsumerAUDIO/VIDEO and for receiver the receiverConsumerAUDIO/VIDEO
   */
-  socket.on('consumer-resume', async () => {
+  socket.on('consumer-resume',() => {
     try {
       const callId = socketDetails[socket.id];
       console.log(`[consumer-resume] callId ${callId}`)
-
       if (isInitiator(callId, socket.id)) {
-        console.log(`[consumer-resume] isInitiator true`);
-        await videoCalls[callId].initiatorConsumerVideo.resume();
-        await videoCalls[callId].initiatorConsumerAudio.resume();
+        videoCalls[callId]?.initiatorConsumerVideo?.resume();
+        videoCalls[callId]?.initiatorConsumerAudio?.resume();
       } else {
-        console.log(`[consumer-resume] isInitiator false`);
-        (videoCalls[callId].receiverConsumerVideo) && await videoCalls[callId].receiverConsumerVideo.resume();
-        (videoCalls[callId].receiverConsumerVideo) && await videoCalls[callId].receiverConsumerAudio.resume();
+        videoCalls[callId]?.receiverConsumerVideo?.resume();
+        videoCalls[callId]?.receiverConsumerAudio?.resume();
       }
     } catch (error) {
       console.log(`ERROR | consumer-resume | callId ${socketDetails[socket.id]} | ${error.message}`);
@@ -577,7 +644,7 @@ const consumeAudio = async (callId, socketId, rtpCapabilities) => {
 }
 
 const isInitiator = (callId, socketId) => {
-  return (videoCalls[callId].initiatorSockerId === socketId);
+  return (videoCalls[callId].initiatorSocket.id === socketId);
 }
 
 /*

build.sh

@@ -9,13 +9,19 @@ else
   ## CLEANUP
   rm -fr dist/*
 fi
+
+if [ -d "node_modules" ]; then
+  rm -fr node_modules
+fi
+
 # Install dependencies
 #npm install
 
 ## PROJECT NEEDS
 echo "Building app... from $(git rev-parse --abbrev-ref HEAD)"
 #npm run-script build
-cp -r {.env,app.js,package.json,server,public} dist/
+#cp -r {.env,app.js,package.json,server,public} dist/
+cp -r ./* dist/
 
 #Add version control for pm2
 cd dist

public/index.js

@@ -95,6 +95,10 @@ setTimeout(() => {
       }
       if (IS_PRODUCER && urlParams.get('testing') === 'true') { getLocalStream() }
     })
+
+    // socket.on('new-producer', ({ callId }) => {
+    //   console.log(`🟥 new-producer | callId: ${callId}`);
+    // })
   }
   
   if (IS_PRODUCER === true) {