debugger: allow longer time to connect

Make five attempts with a timeout of 1 second each rather than 10 attempts with a timeout of 500ms each. This is to allow for slower-connecting devices like Raspberry Pi. PR-URL: #38161 Refs: #36481 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Jan Krems <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Stephen Belanger <[email protected]> Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
nodejs · Apr 29, 2021 · 6fff9ff · 6fff9ff
1 parent def85da
commit 6fff9ff
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/internal/inspector/_inspect.js b/lib/internal/inspector/_inspect.js
@@ -234,13 +234,13 @@ class NodeInspector {
             this.stdout.write(' ok\n');
           }, (error) => {
             debuglog('connect failed', error);
-            // If it's failed to connect 10 times then print failed message
-            if (connectionAttempts >= 10) {
+            // If it's failed to connect 5 times then print failed message
+            if (connectionAttempts >= 5) {
               this.stdout.write(' failed to connect, please retry\n');
               process.exit(1);
             }
 
-            return new Promise((resolve) => setTimeout(resolve, 500))
+            return new Promise((resolve) => setTimeout(resolve, 1000))
               .then(attemptConnect);
           });
       };