AudioRedWorker guaranteed to leak after terminating a call

[issacgerges]

What happened and what did you expect to happen?

When terminating a call when audioRed was enabled, the this.audioRedWorker reference is cleaned up, but the worker isn't explicitly terminated. This means the worker will live on for the length of the session, retaining some memory

The following should call this.audoRedWorker.terminate() before removing the reference via this.audioRedWorker = null;

amazon-chime-sdk-js/src/transceivercontroller/DefaultTransceiverController.ts

Lines 630 to 645 in b4e0280

	private destroyAudioRedWorkerAndStates(): void {
	if (this.audioRedWorker) {
	URL.revokeObjectURL(this.audioRedWorkerURL);
	this.audioRedWorkerURL = null;
	this.audioRedWorker = null;
	this.currentNumRedundantEncodings = 0;
	this.lastRedHolddownTimerStartTimestampMs = 0;
	this.lastAudioRedTurnOffTimestampMs = 0;
	this.lastHighPacketLossEventTimestampMs = 0;
	this.audioRedEnabled = true;
	/* istanbul ignore next */
	this.meetingSessionContext?.audioVideoController.removeObserver(this);
	/* istanbul ignore next */
	this.removeRedundantAudioRecoveryMetricsObserver(this.meetingSessionContext?.statsCollector);
	}
	}

Have you reviewed our existing documentation?

• GitHub issues
• FAQs page
• README page
• Feature-specific guides

Reproduction steps

Create a call with audioRed enabled and end it and see the worker global scopes retained

Amazon Chime SDK for JavaScript version

3.23.0

What browsers are you seeing the problem on?

Chromium

Browser version

128.0.6613.120

Meeting and Attendee ID Information.

No response

Browser console logs

n/a

[hensmi-amazon]

Made change recommended in #2960, thanks!

[ltrung]

Fix has been merged. Closing for now.