mediasoup-client 3.24.1 → 3.24.4

package/lib/handlers/Safari12.js

@@ -238,115 +238,131 @@ class Safari12 extends enhancedEvents_1.EnhancedEventEmitter {
         this.assertNotClosed();
         this.assertSendDirection();
         logger.debug('send() [kind:%s, track.id:%s, streamId:%s]', track.kind, track.id, streamId);
+        if (encodings && encodings.length > 1) {
+            // Set rid and verify scalabilityMode in each encoding.
+            // NOTE: Even if WebRTC allows different scalabilityMode (different number
+            // of temporal layers) per simulcast stream, we need that those are the
+            // same in all them, so let's pick up the highest value.
+            // NOTE: If scalabilityMode is not given, Safari will use L1T3.
+            let maxTemporalLayers = 1;
+            for (const encoding of encodings) {
+                const temporalLayers = encoding.scalabilityMode
+                    ? (0, scalabilityModes_1.parse)(encoding.scalabilityMode).temporalLayers
+                    : 3;
+                if (temporalLayers > maxTemporalLayers) {
+                    maxTemporalLayers = temporalLayers;
+                }
+            }
+            encodings.forEach((encoding, idx) => {
+                encoding.rid = `r${idx}`;
+                encoding.scalabilityMode = `L1T${maxTemporalLayers}`;
+            });
+        }
         const mediaSectionIdx = this._remoteSdp.getNextMediaSectionIdx();
         const transceiver = this._pc.addTransceiver(track, {
             direction: 'sendonly',
             streams: [this._sendStream],
             sendEncodings: encodings,
         });
-        if (this._forcedRtpExtensions) {
-            ortcUtils.applyForcedRtpExtensions(transceiver, this._forcedRtpExtensions);
-        }
-        if (onRtpSender) {
-            onRtpSender(transceiver.sender);
-        }
-        let offer = await this._pc.createOffer();
-        let localSdpObject = sdpTransform.parse(offer.sdp);
-        if (localSdpObject.extmapAllowMixed) {
-            this._remoteSdp.setSessionExtmapAllowMixed();
-        }
-        const nativeRtpCapabilities = Safari12.getLocalRtpCapabilities(localSdpObject);
-        const sendExtendedRtpCapabilities = this._getSendExtendedRtpCapabilities(nativeRtpCapabilities);
-        // Generic sending RTP parameters.
-        const sendingRtpParameters = ortc.getSendingRtpParameters(track.kind, sendExtendedRtpCapabilities);
-        // This may throw.
-        sendingRtpParameters.codecs = ortc.reduceCodecs(sendingRtpParameters.codecs, codec);
-        // Generic sending RTP parameters suitable for the SDP remote answer.
-        const sendingRemoteRtpParameters = ortc.getSendingRemoteRtpParameters(track.kind, sendExtendedRtpCapabilities);
-        // This may throw.
-        sendingRemoteRtpParameters.codecs = ortc.reduceCodecs(sendingRemoteRtpParameters.codecs, codec);
-        let offerMediaObject;
-        if (!this._transportReady) {
-            await this.setupTransport({
-                localDtlsRole: this._forcedLocalDtlsRole ?? 'client',
-                localSdpObject,
+        try {
+            if (this._forcedRtpExtensions) {
+                ortcUtils.applyForcedRtpExtensions(transceiver, this._forcedRtpExtensions);
+            }
+            if (onRtpSender) {
+                onRtpSender(transceiver.sender);
+            }
+            const offer = await this._pc.createOffer();
+            let localSdpObject = sdpTransform.parse(offer.sdp);
+            if (localSdpObject.extmapAllowMixed) {
+                this._remoteSdp.setSessionExtmapAllowMixed();
+            }
+            const nativeRtpCapabilities = Safari12.getLocalRtpCapabilities(localSdpObject);
+            const sendExtendedRtpCapabilities = this._getSendExtendedRtpCapabilities(nativeRtpCapabilities);
+            // Generic sending RTP parameters.
+            const sendingRtpParameters = ortc.getSendingRtpParameters(track.kind, sendExtendedRtpCapabilities);
+            // This may throw.
+            sendingRtpParameters.codecs = ortc.reduceCodecs(sendingRtpParameters.codecs, codec);
+            // Generic sending RTP parameters suitable for the SDP remote answer.
+            const sendingRemoteRtpParameters = ortc.getSendingRemoteRtpParameters(track.kind, sendExtendedRtpCapabilities);
+            // This may throw.
+            sendingRemoteRtpParameters.codecs = ortc.reduceCodecs(sendingRemoteRtpParameters.codecs, codec);
+            if (!this._transportReady) {
+                await this.setupTransport({
+                    localDtlsRole: this._forcedLocalDtlsRole ?? 'client',
+                    localSdpObject,
+                });
+            }
+            logger.debug('send() | calling pc.setLocalDescription() [offer:%o]', offer);
+            await this._pc.setLocalDescription(offer);
+            // We can now get the transceiver.mid.
+            const localId = transceiver.mid;
+            // Set MID.
+            sendingRtpParameters.mid = localId;
+            localSdpObject = sdpTransform.parse(this._pc.localDescription.sdp);
+            const offerMediaObject = localSdpObject.media[mediaSectionIdx.idx];
+            // Set RTCP CNAME.
+            sendingRtpParameters.rtcp.cname = sdpCommonUtils.getCname({
+                offerMediaObject,
             });
-        }
-        const layers = (0, scalabilityModes_1.parse)((encodings ?? [{}])[0].scalabilityMode);
-        if (encodings && encodings.length > 1) {
-            logger.debug('send() | enabling legacy simulcast');
-            localSdpObject = sdpTransform.parse(offer.sdp);
-            offerMediaObject = localSdpObject.media[mediaSectionIdx.idx];
-            sdpUnifiedPlanUtils.addLegacySimulcast({
+            // Set msid.
+            sendingRtpParameters.msid = `${streamId ?? this._sendStream.id} ${track.id}`;
+            // Set RTP encodings by parsing the SDP offer if no encodings are given.
+            if (!encodings) {
+                sendingRtpParameters.encodings = sdpUnifiedPlanUtils.getRtpEncodings({
+                    offerMediaObject,
+                    codecs: sendingRtpParameters.codecs,
+                });
+            }
+            // Set RTP encodings by parsing the SDP offer and complete them with given
+            // one if just a single encoding has been given.
+            else if (encodings.length === 1) {
+                const newEncodings = sdpUnifiedPlanUtils.getRtpEncodings({
+                    offerMediaObject,
+                    codecs: sendingRtpParameters.codecs,
+                });
+                Object.assign(newEncodings[0], encodings[0]);
+                sendingRtpParameters.encodings = newEncodings;
+            }
+            // Otherwise if more than 1 encoding are given use them verbatim.
+            else {
+                sendingRtpParameters.encodings = encodings;
+            }
+            this._remoteSdp.send({
                 offerMediaObject,
-                numStreams: encodings.length,
+                reuseMid: mediaSectionIdx.reuseMid,
+                offerRtpParameters: sendingRtpParameters,
+                answerRtpParameters: sendingRemoteRtpParameters,
+                codecOptions,
             });
-            offer = {
-                type: 'offer',
-                sdp: sdpTransform.write(localSdpObject),
+            const answer = {
+                type: 'answer',
+                sdp: this._remoteSdp.getSdp(),
+            };
+            logger.debug('send() | calling pc.setRemoteDescription() [answer:%o]', answer);
+            await this._pc.setRemoteDescription(answer);
+            // Store in the map.
+            this._mapMidTransceiver.set(localId, transceiver);
+            return {
+                localId,
+                rtpParameters: sendingRtpParameters,
+                rtpSender: transceiver.sender,
             };
         }
-        logger.debug('send() | calling pc.setLocalDescription() [offer:%o]', offer);
-        await this._pc.setLocalDescription(offer);
-        // We can now get the transceiver.mid.
-        const localId = transceiver.mid;
-        // Set MID.
-        sendingRtpParameters.mid = localId;
-        localSdpObject = sdpTransform.parse(this._pc.localDescription.sdp);
-        offerMediaObject = localSdpObject.media[mediaSectionIdx.idx];
-        // Set RTCP CNAME.
-        sendingRtpParameters.rtcp.cname = sdpCommonUtils.getCname({
-            offerMediaObject,
-        });
-        // Set msid.
-        sendingRtpParameters.msid = `${streamId ?? this._sendStream.id} ${track.id}`;
-        // Set RTP encodings.
-        sendingRtpParameters.encodings = sdpUnifiedPlanUtils.getRtpEncodings({
-            offerMediaObject,
-            codecs: sendingRtpParameters.codecs,
-        });
-        // Complete encodings with given values.
-        if (encodings) {
-            for (let idx = 0; idx < sendingRtpParameters.encodings.length; ++idx) {
-                if (encodings[idx]) {
-                    Object.assign(sendingRtpParameters.encodings[idx], encodings[idx]);
-                }
-            }
-        }
-        // If VP8 or H264 and there is effective simulcast, add scalabilityMode to
-        // each encoding.
-        if (sendingRtpParameters.encodings.length > 1 &&
-            (sendingRtpParameters.codecs[0].mimeType.toLowerCase() === 'video/vp8' ||
-                sendingRtpParameters.codecs[0].mimeType.toLowerCase() === 'video/h264')) {
-            for (const encoding of sendingRtpParameters.encodings) {
-                if (encoding.scalabilityMode) {
-                    encoding.scalabilityMode = `L1T${layers.temporalLayers}`;
-                }
-                else {
-                    encoding.scalabilityMode = 'L1T3';
+        catch (error) {
+            // Undo the changes done in the PeerConnection, otherwise it gets out of
+            // sync with the remote SDP and later calls to send() fail.
+            try {
+                if (this._pc.signalingState === 'have-local-offer') {
+                    try {
+                        await this._pc.setLocalDescription({ type: 'rollback' });
+                    }
+                    catch (error2) { }
                 }
+                transceiver.stop();
             }
+            catch (error3) { }
+            throw error;
         }
-        this._remoteSdp.send({
-            offerMediaObject,
-            reuseMid: mediaSectionIdx.reuseMid,
-            offerRtpParameters: sendingRtpParameters,
-            answerRtpParameters: sendingRemoteRtpParameters,
-            codecOptions,
-        });
-        const answer = {
-            type: 'answer',
-            sdp: this._remoteSdp.getSdp(),
-        };
-        logger.debug('send() | calling pc.setRemoteDescription() [answer:%o]', answer);
-        await this._pc.setRemoteDescription(answer);
-        // Store in the map.
-        this._mapMidTransceiver.set(localId, transceiver);
-        return {
-            localId,
-            rtpParameters: sendingRtpParameters,
-            rtpSender: transceiver.sender,
-        };
     }
     async stopSending(localId) {
         this.assertSendDirection();
@@ -514,25 +530,43 @@ class Safari12 extends enhancedEvents_1.EnhancedEventEmitter {
         // If this is the first DataChannel we need to create the SDP answer with
         // m=application section.
         if (!this._hasDataChannelMediaSection) {
-            const offer = await this._pc.createOffer();
-            const localSdpObject = sdpTransform.parse(offer.sdp);
-            const offerMediaObject = localSdpObject.media.find(m => m.type === 'application');
-            if (!this._transportReady) {
-                await this.setupTransport({
-                    localDtlsRole: this._forcedLocalDtlsRole ?? 'client',
-                    localSdpObject,
-                });
+            try {
+                const offer = await this._pc.createOffer();
+                const localSdpObject = sdpTransform.parse(offer.sdp);
+                const offerMediaObject = localSdpObject.media.find(m => m.type === 'application');
+                if (!this._transportReady) {
+                    await this.setupTransport({
+                        localDtlsRole: this._forcedLocalDtlsRole ?? 'client',
+                        localSdpObject,
+                    });
+                }
+                logger.debug('sendDataChannel() | calling pc.setLocalDescription() [offer:%o]', offer);
+                await this._pc.setLocalDescription(offer);
+                this._remoteSdp.sendSctpAssociation({ offerMediaObject });
+                const answer = {
+                    type: 'answer',
+                    sdp: this._remoteSdp.getSdp(),
+                };
+                logger.debug('sendDataChannel() | calling pc.setRemoteDescription() [answer:%o]', answer);
+                await this._pc.setRemoteDescription(answer);
+                this._hasDataChannelMediaSection = true;
+            }
+            catch (error) {
+                // Undo the changes done in the PeerConnection, otherwise it gets out of
+                // sync with the remote SDP and later calls to send() and
+                // sendDataChannel() fail.
+                try {
+                    if (this._pc.signalingState === 'have-local-offer') {
+                        try {
+                            await this._pc.setLocalDescription({ type: 'rollback' });
+                        }
+                        catch (error2) { }
+                    }
+                    dataChannel.close();
+                }
+                catch (error3) { }
+                throw error;
             }
-            logger.debug('sendDataChannel() | calling pc.setLocalDescription() [offer:%o]', offer);
-            await this._pc.setLocalDescription(offer);
-            this._remoteSdp.sendSctpAssociation({ offerMediaObject });
-            const answer = {
-                type: 'answer',
-                sdp: this._remoteSdp.getSdp(),
-            };
-            logger.debug('sendDataChannel() | calling pc.setRemoteDescription() [answer:%o]', answer);
-            await this._pc.setRemoteDescription(answer);
-            this._hasDataChannelMediaSection = true;
         }
         const newSctpStreamParameters = {
             streamId: options.id,

package/lib/index.d.ts

@@ -6,7 +6,7 @@ export * as types from './types';
 /**
  * Expose mediasoup-client version.
  */
-export declare const version = "3.24.1";
+export declare const version = "3.24.4";
 /**
  * Expose Device class and device detector helpers.
  */

package/lib/index.js

@@ -46,7 +46,7 @@ exports.types = __importStar(require("./types"));
 /**
  * Expose mediasoup-client version.
  */
-exports.version = '3.24.1';
+exports.version = '3.24.4';
 /**
  * Expose Device class and device detector helpers.
  */

package/lib/ortc.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ortc.d.ts","sourceRoot":"","sources":["../src/ortc.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACX,eAAe,EACf,SAAS,EACT,kBAAkB,EAElB,aAAa,EACb,kBAAkB,EAKlB,uBAAuB,EAGvB,MAAM,iBAAiB,CAAC;AACzB,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAC;AAO7D;;;;GAIG;AACH,wBAAgB,mCAAmC,CAClD,IAAI,EAAE,eAAe,GACnB,IAAI,CA0BN;AAED;;;;GAIG;AACH,wBAAgB,iCAAiC,CAAC,MAAM,EAAE,aAAa,GAAG,IAAI,CAiD7E;AAED;;;;GAIG;AACH,wBAAgB,wCAAwC,CACvD,MAAM,EAAE,oBAAoB,GAC1B,IAAI,CA8DN;AAED;;;;;GAKG;AACH,wBAAgB,0BAA0B,CACzC,SAAS,EAAE,eAAe,EAC1B,UAAU,EAAE,eAAe,EAC3B,sBAAsB,EAAE,OAAO,GAC7B,uBAAuB,CAiJzB;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,CACrC,uBAAuB,EAAE,uBAAuB,GAC9C,eAAe,CAEjB;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,CACrC,uBAAuB,EAAE,uBAAuB,GAC9C,eAAe,CAEjB;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CACtC,IAAI,EAAE,SAAS,EACf,uBAAuB,EAAE,uBAAuB,GAC9C,aAAa,CA8Df;AAED;;GAEG;AACH,wBAAgB,6BAA6B,CAC5C,IAAI,EAAE,SAAS,EACf,uBAAuB,EAAE,uBAAuB,GAC9C,aAAa,CA+Ff;AAED;;;;;;;;;GASG;AACH,wBAAgB,YAAY,CAC3B,MAAM,EAAE,kBAAkB,EAAE,EAC5B,QAAQ,CAAC,EAAE,kBAAkB,GAC3B,kBAAkB,EAAE,CA+BtB;AAED;;GAEG;AACH,wBAAgB,6BAA6B,CAC5C,kBAAkB,EAAE,aAAa,GAC/B,aAAa,CAoBf;AAED;;GAEG;AACH,wBAAgB,OAAO,CACtB,IAAI,EAAE,SAAS,EACf,eAAe,EAAE,eAAe,GAC9B,OAAO,CAET;AAED;;;GAGG;AACH,wBAAgB,UAAU,CACzB,aAAa,EAAE,aAAa,EAC5B,eAAe,EAAE,eAAe,GAC9B,OAAO,CAaT"}
+{"version":3,"file":"ortc.d.ts","sourceRoot":"","sources":["../src/ortc.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACX,eAAe,EACf,SAAS,EACT,kBAAkB,EAElB,aAAa,EACb,kBAAkB,EAKlB,uBAAuB,EAGvB,MAAM,iBAAiB,CAAC;AACzB,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAC;AAO7D;;;;GAIG;AACH,wBAAgB,mCAAmC,CAClD,IAAI,EAAE,eAAe,GACnB,IAAI,CA0BN;AAED;;;;GAIG;AACH,wBAAgB,iCAAiC,CAAC,MAAM,EAAE,aAAa,GAAG,IAAI,CAiD7E;AAED;;;;GAIG;AACH,wBAAgB,wCAAwC,CACvD,MAAM,EAAE,oBAAoB,GAC1B,IAAI,CAsEN;AAED;;;;;GAKG;AACH,wBAAgB,0BAA0B,CACzC,SAAS,EAAE,eAAe,EAC1B,UAAU,EAAE,eAAe,EAC3B,sBAAsB,EAAE,OAAO,GAC7B,uBAAuB,CAiJzB;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,CACrC,uBAAuB,EAAE,uBAAuB,GAC9C,eAAe,CAEjB;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,CACrC,uBAAuB,EAAE,uBAAuB,GAC9C,eAAe,CAEjB;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CACtC,IAAI,EAAE,SAAS,EACf,uBAAuB,EAAE,uBAAuB,GAC9C,aAAa,CA8Df;AAED;;GAEG;AACH,wBAAgB,6BAA6B,CAC5C,IAAI,EAAE,SAAS,EACf,uBAAuB,EAAE,uBAAuB,GAC9C,aAAa,CA+Ff;AAED;;;;;;;;;GASG;AACH,wBAAgB,YAAY,CAC3B,MAAM,EAAE,kBAAkB,EAAE,EAC5B,QAAQ,CAAC,EAAE,kBAAkB,GAC3B,kBAAkB,EAAE,CA+BtB;AAED;;GAEG;AACH,wBAAgB,6BAA6B,CAC5C,kBAAkB,EAAE,aAAa,GAC/B,aAAa,CAoBf;AAED;;GAEG;AACH,wBAAgB,OAAO,CACtB,IAAI,EAAE,SAAS,EACf,eAAe,EAAE,eAAe,GAC9B,OAAO,CAET;AAED;;;GAGG;AACH,wBAAgB,UAAU,CACzB,aAAa,EAAE,aAAa,EAC5B,eAAe,EAAE,eAAe,GAC9B,OAAO,CAaT"}

package/lib/ortc.js

@@ -151,24 +151,28 @@ function validateAndNormalizeSctpStreamParameters(params) {
         params.ordered = true;
     }
     // maxPacketLifeTime is optional.
-    if (params.maxPacketLifeTime &&
+    if (params.maxPacketLifeTime !== undefined &&
         typeof params.maxPacketLifeTime !== 'number') {
         throw new TypeError('invalid params.maxPacketLifeTime');
     }
     // maxRetransmits is optional.
-    if (params.maxRetransmits && typeof params.maxRetransmits !== 'number') {
+    if (params.maxRetransmits !== undefined &&
+        typeof params.maxRetransmits !== 'number') {
         throw new TypeError('invalid params.maxRetransmits');
     }
-    if (params.maxPacketLifeTime && params.maxRetransmits) {
+    if (params.maxPacketLifeTime !== undefined &&
+        params.maxRetransmits !== undefined) {
         throw new TypeError('cannot provide both maxPacketLifeTime and maxRetransmits');
     }
     if (orderedGiven &&
         params.ordered &&
-        (params.maxPacketLifeTime || params.maxRetransmits)) {
+        (params.maxPacketLifeTime !== undefined ||
+            params.maxRetransmits !== undefined)) {
         throw new TypeError('cannot be ordered with maxPacketLifeTime or maxRetransmits');
     }
     else if (!orderedGiven &&
-        (params.maxPacketLifeTime || params.maxRetransmits)) {
+        (params.maxPacketLifeTime !== undefined ||
+            params.maxRetransmits !== undefined)) {
         params.ordered = false;
     }
     // label is optional.

package/lib/test/test.js

@@ -761,6 +761,31 @@ test('transport.produceData() succeeds', async () => {
     expect(dataProducer.label).toBe('FOO');
     expect(dataProducer.protocol).toBe('BAR');
 }, 500);
+test('transport.produceData() with maxRetransmits 0 or maxPacketLifeTime 0 creates an unordered DataProducer', async () => {
+    const dataProducer1 = await ctx.connectedSendTransport.produceData({
+        maxRetransmits: 0,
+    });
+    expect(dataProducer1.sctpStreamParameters.ordered).toBe(false);
+    expect(dataProducer1.sctpStreamParameters.maxRetransmits).toBe(0);
+    const dataProducer2 = await ctx.connectedSendTransport.produceData({
+        maxPacketLifeTime: 0,
+    });
+    expect(dataProducer2.sctpStreamParameters.ordered).toBe(false);
+    expect(dataProducer2.sctpStreamParameters.maxPacketLifeTime).toBe(0);
+}, 500);
+test('transport.produceData() closes the DataChannel if the "producedata" listener rejects', async () => {
+    const handlerSpy = jest.spyOn(ctx.connectedSendTransport.handler, 'sendDataChannel');
+    const error = new Error('produceData() failed');
+    const listener = (parameters, callback, errback) => {
+        errback(error);
+    };
+    ctx.connectedSendTransport.prependListener('producedata', listener);
+    await expect(ctx.connectedSendTransport.produceData()).rejects.toThrow(error);
+    const { dataChannel } = await handlerSpy.mock.results[0].value;
+    expect(dataChannel.readyState).toBe('closed');
+    ctx.connectedSendTransport.off('producedata', listener);
+    handlerSpy.mockRestore();
+}, 500);
 test('transport.produceData() in a receiving Transport rejects with UnsupportedError', async () => {
     await expect(ctx.recvTransport.produceData({})).rejects.toThrow(errors_1.UnsupportedError);
 }, 500);
@@ -904,7 +929,7 @@ test('producer.replaceTrack() with a new track succeeds', async () => {
     const videoProducerPreviousTrack = ctx.videoProducer.track;
     const newVideoTrack = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'video' });
     await expect(ctx.videoProducer.replaceTrack({ track: newVideoTrack })).resolves.toBe(undefined);
-    // Previous track must be 'ended' due to stopTracks: true.
+    // Previous track must be 'ended' due to stopTracks: true (default value).
     expect(videoProducerPreviousTrack?.readyState).toBe('ended');
     expect(ctx.videoProducer.track).not.toBe(videoProducerPreviousTrack);
     expect(ctx.videoProducer.track).toBe(newVideoTrack);
@@ -949,6 +974,34 @@ test('producer.setMaxSpatialLayer() succeeds', async () => {
     await expect(ctx.videoProducer.setMaxSpatialLayer(0)).resolves.toBe(undefined);
     expect(ctx.videoProducer.maxSpatialLayer).toBe(0);
 }, 500);
+test('producer.replaceTrack() rejects with InvalidStateError and stops the new track if the Producer is closed while replacing', async () => {
+    const previousTrack = ctx.videoProducer.track;
+    const newTrack = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'video' });
+    const replacePromise = ctx.videoProducer.replaceTrack({ track: newTrack });
+    ctx.videoProducer.close();
+    await expect(replacePromise).rejects.toThrow(errors_1.InvalidStateError);
+    expect(previousTrack?.readyState).toBe('ended');
+    // The new track must be 'ended' due to stopTracks: true (default value).
+    expect(newTrack.readyState).toBe('ended');
+    expect(ctx.videoProducer.track).toBe(previousTrack);
+}, 500);
+test('producer.replaceTrack() stops the new track if the Transport is closed while replacing', async () => {
+    const newTrack = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'video' });
+    const replacePromise = ctx.videoProducer.replaceTrack({ track: newTrack });
+    ctx.connectedSendTransport.close();
+    await expect(replacePromise).rejects.toThrow();
+    // The new track must be 'ended' due to stopTracks: true (default value).
+    expect(newTrack.readyState).toBe('ended');
+}, 500);
+test('producer.setMaxSpatialLayer() rejects and keeps the previous value if the handler fails', async () => {
+    const previousMaxSpatialLayer = ctx.videoProducer.maxSpatialLayer;
+    const error = new Error('setParameters() failed');
+    const handlerSpy = jest.spyOn(ctx.connectedSendTransport.handler, 'setMaxSpatialLayer');
+    handlerSpy.mockRejectedValueOnce(error);
+    await expect(ctx.videoProducer.setMaxSpatialLayer(1)).rejects.toThrow(error);
+    expect(ctx.videoProducer.maxSpatialLayer).toBe(previousMaxSpatialLayer);
+    handlerSpy.mockRestore();
+}, 500);
 test('producer.setMaxSpatialLayer() in an audio Producer rejects with UnsupportedError', async () => {
     await expect(ctx.audioProducer.setMaxSpatialLayer(1)).rejects.toThrow(errors_1.UnsupportedError);
     expect(ctx.audioProducer.maxSpatialLayer).toBe(undefined);
@@ -1083,6 +1136,12 @@ test('transport.produce() rejects with InvalidStateError if closed', async () =>
     // The track must be 'live' due to stopTracks: false.
     expect(track.readyState).toBe('live');
 }, 500);
+test('transport.produce() stops the track if the Transport is closed', async () => {
+    const track = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'audio' });
+    ctx.connectedSendTransport.close();
+    await expect(ctx.connectedSendTransport.produce({ track })).rejects.toThrow(errors_1.InvalidStateError);
+    expect(track.readyState).toBe('ended');
+}, 500);
 test('transport.consume() rejects with InvalidStateError if closed', async () => {
     ctx.connectedRecvTransport.close();
     // @ts-expect-error --- On purpose.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "mediasoup-client",
-	"version": "3.24.1",
+	"version": "3.24.4",
 	"description": "mediasoup client side TypeScript library",
 	"contributors": [
 		"Iñaki Baz Castillo <ibc@aliax.net> (https://inakibaz.me)",
@@ -91,13 +91,13 @@
 		"eslint-config-prettier": "^10.1.8",
 		"eslint-plugin-jest": "^29.16.6",
 		"eslint-plugin-prettier": "^5.5.6",
-		"globals": "^17.12.0",
+		"globals": "^17.13.0",
 		"jest": "^30.5.2",
-		"knip": "^6.38.0",
+		"knip": "^6.39.0",
 		"open-cli": "^9.0.0",
 		"prettier": "^3.9.9",
-		"ts-jest": "^29.4.13",
+		"ts-jest": "^29.4.14",
 		"typescript": "^6.0.3",
-		"typescript-eslint": "^8.70.1"
+		"typescript-eslint": "^8.71.0"
 	}
 }