fix: sni is missing in surge trojan config

surgioproject · Apr 5, 2020 · be0c340 · be0c340
1 parent 2d01be4
commit be0c340
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/lib/utils/index.ts b/lib/utils/index.ts
@@ -414,12 +414,7 @@ export const getSurgeNodes = function(
             nodeConfig.hostname,
             `${nodeConfig.port}`,
             `password=${nodeConfig.password}`,
-            ...(typeof nodeConfig.tfo === 'boolean' ? [
-              `tfo=${nodeConfig.tfo}`,
-            ] : []),
-            ...(typeof nodeConfig.mptcp === 'boolean' ? [
-              `mptcp=${nodeConfig.mptcp}`,
-            ] : []),
+            ...pickAndFormatStringList(nodeConfig, ['tfo', 'mptcp', 'sni']),
             ...(typeof nodeConfig.skipCertVerify === 'boolean' ? [
               `skip-cert-verify=${nodeConfig.skipCertVerify}`,
             ] : []),

diff --git a/test/utils/index.test.ts b/test/utils/index.test.ts
@@ -207,11 +207,12 @@ test('getSurgeNodes', async t => {
       hostname: 'example.com',
       port: 443,
       password: 'password1',
+      sni: 'sni.com',
       tfo: true,
       mptcp: true,
       skipCertVerify: true,
     }]),
-    'trojan node 2 = trojan, example.com, 443, password=password1, tfo=true, mptcp=true, skip-cert-verify=true',
+    'trojan node 2 = trojan, example.com, 443, password=password1, tfo=true, mptcp=true, sni=sni.com, skip-cert-verify=true',
   );
 
   t.is(