fix: spawn ENAMETOOLONG by reduceTrackerString

agalwood · Jun 15, 2021 · 1de2acc · 1de2acc
1 parent 333f01a
commit 1de2acc
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 4 deletions.
diff --git a/src/main/Application.js b/src/main/Application.js
@@ -13,7 +13,8 @@ import {
 import { checkIsNeedRun } from '@shared/utils'
 import {
   convertTrackerDataToComma,
-  fetchBtTrackerFromSource
+  fetchBtTrackerFromSource,
+  reduceTrackerString
 } from '@shared/utils/tracker'
 import logger from './core/Logger'
 import ConfigManager from './core/ConfigManager'
@@ -316,7 +317,8 @@ export default class Application extends EventEmitter {
           return
         }
 
-        const tracker = convertTrackerDataToComma(data)
+        let tracker = convertTrackerDataToComma(data)
+        tracker = reduceTrackerString(tracker)
         this.savePreference({
           system: {
             'bt-tracker': tracker

diff --git a/src/main/core/ConfigManager.js b/src/main/core/ConfigManager.js
@@ -19,6 +19,7 @@ import {
   NGOSANG_TRACKERS_BEST_URL_CDN
 } from '@shared/constants'
 import { separateConfig } from '@shared/utils'
+import { reduceTrackerString } from '@shared/utils/tracker'
 
 export default class ConfigManager {
   constructor () {
@@ -140,6 +141,10 @@ export default class ConfigManager {
     Object.keys(others).forEach(key => {
       this.systemConfig.delete(key)
     })
+
+    // Fix spawn ENAMETOOLONG on Windows
+    const tracker = reduceTrackerString(this.systemConfig.get('bt-tracker'))
+    this.setSystemConfig('bt-tracker', tracker)
   }
 
   fixUserConfig () {

diff --git a/src/renderer/components/Preference/Advanced.vue b/src/renderer/components/Preference/Advanced.vue
@@ -387,7 +387,7 @@
     diffConfig,
     getRandomInt
   } from '@shared/utils'
-  import { convertTrackerDataToLine } from '@shared/utils/tracker'
+  import { convertTrackerDataToLine, reduceTrackerString } from '@shared/utils/tracker'
   import '@/components/Icons/dice'
   import '@/components/Icons/sync'
   import '@/components/Icons/refresh'
@@ -613,7 +613,7 @@
 
           const { btTracker, noProxy } = changed
           if (btTracker) {
-            data.btTracker = convertLineToComma(btTracker)
+            data.btTracker = reduceTrackerString(convertLineToComma(btTracker))
           }
 
           if (noProxy) {

diff --git a/src/shared/constants.js b/src/shared/constants.js
@@ -32,6 +32,8 @@ export const ENGINE_RPC_PORT = 16800
 export const UNKNOWN_PEERID = '%00%00%00%00%00%00%00%00%00%00%00%00%00%00%00%00%00%00%00%00'
 export const GRAPHIC = '░▒▓█'
 
+export const MAX_BT_TRACKER_LENGTH = 6144
+
 /**
  * @see https://github.com/ngosang/trackerslist
  */

diff --git a/src/shared/utils/tracker.js b/src/shared/utils/tracker.js
@@ -1,5 +1,6 @@
 import { isEmpty } from 'lodash'
 import axios from 'axios'
+import { MAX_BT_TRACKER_LENGTH } from '@shared/constants'
 
 export const fetchBtTrackerFromSource = async (source) => {
   if (isEmpty(source)) {
@@ -26,3 +27,14 @@ export function convertTrackerDataToComma (arr = []) {
   const result = convertTrackerDataToLine(arr).replace(/(?:\r\n|\r|\n)/g, ',').trim()
   return result
 }
+
+export function reduceTrackerString (str = '') {
+  if (str.length <= MAX_BT_TRACKER_LENGTH) {
+    return str
+  }
+
+  const subStr = str.substring(0, MAX_BT_TRACKER_LENGTH)
+  const index = subStr.lastIndexOf(',')
+  const result = subStr.substring(0, index)
+  return result
+}