Be ready for numeric codec parameters exposed as strings in Edge (rel…

…ated to #15)
versatica · Jan 22, 2018 · 094601f · 094601f
1 parent e4e8606
commit 094601f
Show file tree

Hide file tree

Showing 4 changed files with 572 additions and 457 deletions.
diff --git a/dist/mediasoup-client.js b/dist/mediasoup-client.js
diff --git a/lib/handlers/ortc/edgeUtils.js b/lib/handlers/ortc/edgeUtils.js
@@ -28,9 +28,17 @@ export function getCapabilities()
 		// Add mimeType.
 		codec.mimeType = `${codec.kind}/${codec.name}`;
 
-		// NOTE: Edge sets parameters.apt as String rather than Number. Fix it.
-		if (codec.name === 'rtx')
-			codec.parameters.apt = Number(codec.parameters.apt);
+		// NOTE: Edge sets some numeric parameters as String rather than Number. Fix them.
+		if (codec.parameters)
+		{
+			const parameters = codec.parameters;
+
+			if (parameters.apt)
+				parameters.apt = Number(parameters.apt);
+
+			if (parameters['packetization-mode'])
+				parameters['packetization-mode'] = Number(parameters['packetization-mode']);
+		}
 
 		// Delete emty parameter String in rtcpFeedback.
 		for (const feedback of codec.rtcpFeedback || [])