Safemobile/mediasoup
21
0
Fork 0
Code Issues Pull Requests 1 Projects Releases Wiki Activity

WIP: router-close-connections #2

Closed
sergiu wants to merge 13 commits from router-close-connections into master
pull from: router-close-connections
merge into: Safemobile:master
Conversation 0 Commits 13 Files Changed 3 +99 -79
3 changed files with 99 additions and 79 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
app.js
View File

@ -149,7 +149,7 @@ peers.on('connection', async socket => {
})
// see client's socket.emit('transport-produce', ...)
socket.on('transport-produce', async ({ kind, rtpParameters, appData }) => {
socket.on('transport-produce', async ({ kind, rtpParameters, appData }, callback) => {
// call produce based on the prameters from the client
producer = await producerTransport.produce({
kind,
@ -157,6 +157,10 @@ peers.on('connection', async socket => {
})
console.log(`[transport-produce] Producer ID: ${producer.id} | kind: ${producer.kind}`)
// If we have multiple producers/consumers on the same room
// We need to notify all the consumers of the room
// --- code ---
producer.on('transportclose', () => {
console.log('transport for this producer closed', callId)
@ -167,6 +171,15 @@ peers.on('connection', async socket => {
router[callId].close()
delete router[callId]
})
try {
// Send back to the client the Producer's id
callback({
id: producer.id
})
} catch (error) {
cristi commented 2022-09-04 13:40:38 +00:00
Review
Copy Link

Isn't the producer a global variable?

What should the client do with the producer id? Looks below that is sent back to server, why?

Isn't the `producer` a global variable? What should the client do with the producer id? Looks below that is sent back to server, why?
console.log(`ERROR | transport-produce | ${error}`)
}
})
// see client's socket.emit('transport-recv-connect', ...)
@ -196,8 +209,8 @@ peers.on('connection', async socket => {
// https://mediasoup.org/documentation/v3/mediasoup/api/#router-close
router[callId].close()
delete router[callId]
producer.close()
consumer.close()
// producer.close()
// consumer.close()
})
consumer.on('producerclose', () => {
@ -206,8 +219,8 @@ peers.on('connection', async socket => {
// https://mediasoup.org/documentation/v3/mediasoup/api/#router-close
router[callId].close()
delete router[callId]
producer.close()
consumer.close()
// producer.close()
// consumer.close()
})
// from the consumer extract the following params

27
public/bundle.js
View File

@ -20653,8 +20653,8 @@ module.exports = yeast;
},{}],95:[function(require,module,exports){
module.exports = {
hubAddress: 'https://hub.dev.linx.safemobile.com/',
// mediasoupAddress: 'https://video.safemobile.org/mediasoup',
mediasoupAddress: 'http://localhost:3000/mediasoup',
mediasoupAddress: 'https://video.safemobile.org/mediasoup',
// mediasoupAddress: 'http://localhost:3000/mediasoup',
}
},{}],96:[function(require,module,exports){
const io = require('socket.io-client')
@ -20866,6 +20866,7 @@ const getRtpCapabilities = () => {
}
const createSendTransport = () => {
// see server's socket.on('createWebRtcTransport', sender?, ...)
// this is a call from Producer, so sender = true
socket.emit('createWebRtcTransport', { sender: true, callId }, ({ params }) => {
@ -20876,7 +20877,7 @@ const createSendTransport = () => {
return
}
console.log(params)
console.log('params', params)
// creates a new WebRTC Transport to send media
// based on the server's producer transport params
@ -20903,7 +20904,7 @@ const createSendTransport = () => {
})
producerTransport.on('produce', async (parameters, callback, errback) => {
console.log(parameters)
console.log('produce', parameters)
try {
// tell the server to create a Producer
@ -20915,26 +20916,29 @@ const createSendTransport = () => {
rtpParameters: parameters.rtpParameters,
appData: parameters.appData,
}, ({ id }) => {
console.log('[transport-produce] callback | id', id);
// Tell the transport that parameters were transmitted and provide it with the
// server side producer's id.
callback({ id })
})
} catch (error) {
console.log('ERROR | produce |', error);
errback(error)
}
})
connectSendTransport()
})
}
const connectSendTransport = async () => {
console.log('[connectSendTransport]');
// 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
producer = await producerTransport.produce(params)
setTimeout(() => {
producer.on('trackended', () => {
console.log('track ended')
// close video track
@ -20945,6 +20949,10 @@ const connectSendTransport = async () => {
// close video track
})
// Enable Close call button
const closeCallBtn = document.getElementById('btnCloseCall');
closeCallBtn.removeAttribute('disabled');
const answer = {
origin_asset_id: ASSET_ID,
dest_asset_id: originAssetId || parseInt(urlParams.get('dest_asset_id')),
@ -20956,15 +20964,11 @@ const connectSendTransport = async () => {
answer: 'accepted', // answer: 'rejected'
};
console.log('SEND answer', answer);
hub.emit(
'video',
JSON.stringify(answer)
);
// Enable Close call button
const closeCallBtn = document.getElementById('btnCloseCall');
closeCallBtn.removeAttribute('disabled');
}, 7000);
}
const createRecvTransport = async () => {
@ -21009,7 +21013,6 @@ const createRecvTransport = async () => {
}
const resetCallSettings = () => {
socket.emit('transportclose', { callId })
localVideo.srcObject = null
remoteVideo.srcObject = null
consumer = null
@ -21072,7 +21075,7 @@ const closeCall = () => {
const closeCallBtn = document.getElementById('btnCloseCall')
closeCallBtn.setAttribute('disabled', '')
// Reset settings and send closeTransport to video server
// Reset settings
resetCallSettings()
}

20
public/index.js
View File

@ -207,6 +207,7 @@ const getRtpCapabilities = () => {
}
const createSendTransport = () => {
// see server's socket.on('createWebRtcTransport', sender?, ...)
// this is a call from Producer, so sender = true
socket.emit('createWebRtcTransport', { sender: true, callId }, ({ params }) => {
@ -217,7 +218,7 @@ const createSendTransport = () => {
return
}
console.log(params)
console.log('params', params)
// creates a new WebRTC Transport to send media
// based on the server's producer transport params
@ -244,7 +245,7 @@ const createSendTransport = () => {
})
producerTransport.on('produce', async (parameters, callback, errback) => {
console.log(parameters)
console.log('produce', parameters)
try {
// tell the server to create a Producer
@ -256,26 +257,29 @@ const createSendTransport = () => {
rtpParameters: parameters.rtpParameters,
appData: parameters.appData,
}, ({ id }) => {
console.log('[transport-produce] callback | id', id);
// Tell the transport that parameters were transmitted and provide it with the
// server side producer's id.
callback({ id })
})
} catch (error) {
console.log('ERROR | produce |', error);
errback(error)
}
})
connectSendTransport()
})
}
const connectSendTransport = async () => {
console.log('[connectSendTransport]');
// 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
producer = await producerTransport.produce(params)
setTimeout(() => {
producer.on('trackended', () => {
console.log('track ended')
// close video track
@ -286,6 +290,10 @@ const connectSendTransport = async () => {
// close video track
})
// Enable Close call button
const closeCallBtn = document.getElementById('btnCloseCall');
closeCallBtn.removeAttribute('disabled');
const answer = {
origin_asset_id: ASSET_ID,
dest_asset_id: originAssetId || parseInt(urlParams.get('dest_asset_id')),
@ -297,15 +305,11 @@ const connectSendTransport = async () => {
answer: 'accepted', // answer: 'rejected'
};
console.log('SEND answer', answer);
hub.emit(
'video',
JSON.stringify(answer)
);
// Enable Close call button
const closeCallBtn = document.getElementById('btnCloseCall');
closeCallBtn.removeAttribute('disabled');
}, 7000);
}
const createRecvTransport = async () => {