webrtc: fix mtu issue on cell networks

This commit is contained in:
Koushik Dutta
2023-01-09 14:44:51 -08:00
parent 51b6a04d24
commit 85953f4ea5
4 changed files with 9 additions and 6 deletions

View File

@@ -1,4 +1,4 @@
{
"scrypted.debugHost": "127.0.0.1",
"scrypted.debugHost": "koushik-ubuntu",
}

View File

@@ -1,12 +1,12 @@
{
"name": "@scrypted/webrtc",
"version": "0.0.125",
"version": "0.1.1",
"lockfileVersion": 2,
"requires": true,
"packages": {
"": {
"name": "@scrypted/webrtc",
"version": "0.0.125",
"version": "0.1.1",
"dependencies": {
"@scrypted/common": "file:../../common",
"@scrypted/sdk": "file:../../sdk",

View File

@@ -1,6 +1,6 @@
{
"name": "@scrypted/webrtc",
"version": "0.0.125",
"version": "0.1.1",
"scripts": {
"scrypted-setup-project": "scrypted-setup-project",
"prescrypted-setup-project": "scrypted-package-json",

View File

@@ -190,8 +190,11 @@ export async function createTrackForwarder(options: {
// ipv4 mtu is 1500
// so max usable packet size is 1500 - rtp header - tcp header - ip header
// 1500 - 12 - 20 - 20 = 1448.
// but set to 1440 cause that's what cameras seem to use for some reason.
const videoPacketSize = options.ipv4 ? 1440 : 1300;
// but set to 1424 just to be safe.
// 1/9/2023: bug report from eweber discovered that usable MTU on tmobile is 1424.
// additional consideration should be given whether to always enforce ipv6 mtu on
// non-local destination?
const videoPacketSize = options.ipv4 ? 1424 : 1300;
let h264Repacketizer: H264Repacketizer;
let spsPps: ReturnType<typeof getSpsPps>;