Skip to content
This repository was archived by the owner on Oct 23, 2024. It is now read-only.

Commit 88dfc7f

Browse files
authored
Merge pull request #850 from jianjunz/revert-794-install-quic
Revert "Enable QUIC agent by default."
2 parents 131f7c4 + 741eb24 commit 88dfc7f

File tree

7 files changed

+27
-36
lines changed

7 files changed

+27
-36
lines changed

scripts/build.json

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@
1212
"webrtc",
1313
"avstream",
1414
"gst-pipeline",
15-
"sip",
16-
"quic"
15+
"sip"
1716
]
1817
},
1918
"mcu-msdk" : {
@@ -27,8 +26,7 @@
2726
"audio-ranker",
2827
"webrtc",
2928
"avstream",
30-
"sip",
31-
"quic"
29+
"sip"
3230
]
3331
},
3432
"mcu-all" : {
@@ -51,6 +49,12 @@
5149
"webrtc-frame"
5250
]
5351
},
52+
"mcu-quic": {
53+
"include":[
54+
"mcu",
55+
"quic"
56+
]
57+
},
5458
"video-mixer-sw" : {
5559
"path" : "source/agent/video/videoMixer/videoMixer_sw",
5660
"gyp" : "binding.sw.gyp"
@@ -116,6 +120,6 @@
116120
},
117121
"quic" : {
118122
"path" : "source/agent/addons/quic",
119-
"description": "QUIC agent supports WebTransport over QUIC connections."
123+
"description": "QUIC agent supports WebTransport - QuicTransport connections. Experimental feature."
120124
}
121125
}

scripts/installCommonDeps.sh

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -299,30 +299,13 @@ install_licode(){
299299
}
300300

301301
install_quic(){
302-
# QUIC IO
303302
rm $ROOT/third_party/quic-lib -rf
304303
mkdir $ROOT/third_party/quic-lib
305304

306305
pushd ${ROOT}/third_party/quic-lib
307306
wget https://github.com/open-webrtc-toolkit/owt-deps-quic/releases/download/v0.1/dist.tgz
308307
tar xzf dist.tgz
309308
popd
310-
311-
# QUIC transport
312-
local QUIC_SDK_VERSION=`cat $ROOT/source/agent/addons/quic/quic_sdk_version`
313-
local QUIC_TRANSPORT_PATH=$ROOT/third_party/quic-transport
314-
rm -r $QUIC_TRANSPORT_PATH
315-
mkdir $QUIC_TRANSPORT_PATH
316-
pushd $QUIC_TRANSPORT_PATH
317-
wget $QUIC_TRANSPORT_PACKAGE_URL_PREFIX/linux/$QUIC_SDK_VERSION.zip
318-
if [ $? -eq 0 ]; then
319-
unzip $QUIC_SDK_VERSION.zip
320-
rm $QUIC_SDK_VERSION.zip
321-
cp bin/release/libowt_quic_transport.so $ROOT/build/libdeps/build/lib
322-
else
323-
read -p "Failed to download prebuild QUIC SDK. Please download and compile QUIC SDK version $QUIC_SDK_VERSION from https://github.com/open-webrtc-toolkit/owt-deps-quic."
324-
fi
325-
popd
326309
}
327310

328311
install_nicer(){

scripts/pack.js

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,8 @@ const originCwd = cwd();
4040
const osScript = path.join(rootDir, 'scripts/detectOS.sh');
4141
const osType = execSync(`bash ${osScript}`).toString().toLowerCase();
4242

43+
const experimentalTargets = ['quic-agent'];
44+
4345
var allTargets = [];
4446

4547
if (options.full) {
@@ -128,7 +130,8 @@ function getPackList(targets) {
128130
}
129131

130132
var packList = targets.filter((element) => {
131-
if (options.target.includes('all')) return true;
133+
// Don't include QUIC agent by default until CI is added for QUIC SDK.
134+
if (options.target.includes('all') && !experimentalTargets.includes(element.rules.name)) return true;
132135
return options.target.includes(element.rules.name);
133136
});
134137
if (packList.length === 0) {
@@ -415,7 +418,7 @@ function isLibAllowed(libSrc) {
415418
if (!libSrc)
416419
return false;
417420

418-
const allowList = [
421+
const whiteList = [
419422
'rtcadapter',
420423
'libssl.so.1.1',
421424
'libcrypto',
@@ -425,19 +428,18 @@ function isLibAllowed(libSrc) {
425428
'libopenh264',
426429
'libre',
427430
'sipLib',
428-
'librawquic',
429-
'libowt_quic_transport',
431+
'librawquic'
430432
];
431433
if (!options['archive'] || options['with-ffmpeg']) {
432-
allowList.push('libav');
433-
allowList.push('libsw');
434+
whiteList.push('libav');
435+
whiteList.push('libsw');
434436
}
435437

436438
const libName = path.basename(libSrc);
437439

438440
var found = false;
439-
for (let i in allowList) {
440-
if (libName.indexOf(allowList[i]) === 0) {
441+
for (let i in whiteList) {
442+
if (libName.indexOf(whiteList[i]) === 0) {
441443
found = true;
442444
break;
443445
}
@@ -590,6 +592,9 @@ function packScripts() {
590592
}
591593
scriptItems.push('app');
592594
scriptItems.forEach((m) => {
595+
if (experimentalTargets.includes(m)) {
596+
return;
597+
}
593598
startCommands += '${bin}/daemon.sh start ' + m + ' $1\n';
594599
stopCommands += '${bin}/daemon.sh stop ' + m + '\n';
595600
});

source/agent/addons/quic/binding.gyp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,16 @@
3838
'../../../core/common',
3939
'../../../core/owt_base',
4040
'../../../../build/libdeps/build/include',
41-
'../../../../third_party/quic-transport/include',
41+
# TODO: Fix it when CI for QUIC SDK is ready.
42+
'/home/jianjunz/code/quic/src/owt/quic_transport/api',
4243
'<!@(pkg-config glib-2.0 --cflags-only-I | sed s/-I//g)',
4344
],
4445
'ldflags': [
4546
'-Wl,--no-as-needed',
47+
'-Wl,-rpath,/home/jianjunz/code/quic/src/out/debug',
4648
'-L$(CORE_HOME)/../../build/libdeps/build/lib',
4749
'-fPIC',
48-
'-L<(module_root_dir)/../../../../third_party/quic-transport/bin/release',
50+
'-L/home/jianjunz/code/quic/src/out/debug',
4951
],
5052
'cflags_cc!': [
5153
'-std=gnu++0x',

source/agent/addons/quic/quic_sdk_version

Lines changed: 0 additions & 1 deletion
This file was deleted.

source/agent/quic/dist.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@
1818
"../../common/clusterWorker.js",
1919
"../../common/loadCollector.js",
2020
"../../common/logger.js",
21-
"../../common/makeRPC.js",
22-
"../../common/rpcChannel.js",
2321
"../../../scripts/release/initcert.js",
2422
"../../../scripts/release/initauth.js",
2523
"../../../scripts/detectOS.sh"

source/management_api/management_api.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ superserviceID = "_auto_generated_ID_" #default: ""
44
ssl = true #if use HTTPS
55
port = 3000 #default: 3000, port of server
66
numberOfProcess = 4 #default: 1
7-
enableWebTransport = true #default: true.
7+
enableWebTransport = false #default: false.
88

99
[cluster]
1010
name = "owt-cluster"

0 commit comments

Comments
 (0)