fix: Drop the stun.qq.com and use stun.l.google.com instead

Voxelum · Apr 20, 2024 · 6cfcf1a · 6cfcf1a
1 parent 7b2c563
commit 6cfcf1a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/xmcl-runtime/nat/NatService.ts b/xmcl-runtime/nat/NatService.ts
@@ -67,7 +67,8 @@ export class NatService extends StatefulService<NatState> implements INatService
   async refreshNatType(): Promise<void> {
     this.log('Start to sample the nat type')
 
-    const info = await getNatInfoUDP({ stun: 'stun.qq.com' })
+    const stun = 'stun.l.google.com'
+    const info = await getNatInfoUDP({ stun })
     if (info.type !== 'Blocked') {
       this.state.natInfoSet(info.externalIp, info.externalPort)
     }
@@ -77,7 +78,7 @@ export class NatService extends StatefulService<NatState> implements INatService
     const result = await sampleNatType({
       sampleCount: 3,
       retryInterval: 3_000,
-      stun: 'stun.qq.com',
+      stun,
     })
     if (result) {
       this.state.natTypeSet(result)

diff --git a/xmcl-runtime/peer/stun.ts b/xmcl-runtime/peer/stun.ts
@@ -1,5 +1,6 @@
 export const iceServers = [
-  'stun:stun.qq.com',
+  // 'stun:stun.qq.com',
+  'stun.l.google.com',
   // 'stun:stunserver.org',
   // 'stun:stun.voxgratia.org',
   // 'stun:stun.ideasip.com',