Fix EADDRINUSE impress issue

Refs: metarhia/impress#1890 PR-URL: #455 Co-authored-by: Timur Shemsedinov <[email protected]>
metarhia · Oct 3, 2023 · 14eeffd · 14eeffd
1 parent d94caee
commit 14eeffd
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lib/server.js b/lib/server.js
@@ -10,6 +10,7 @@ const { MetaReadable, MetaWritable, chunkDecode } = require('./streams.js');
 
 const SHORT_TIMEOUT = 500;
 const EMPTY_PACKET = Buffer.from('{}');
+const DEFAULT_LISTEN_RETRY = 3;
 
 const createProxy = (data, save) =>
   new Proxy(data, {
@@ -153,8 +154,10 @@ const addHeaders = ({ origin }) => {
   if (origin) HEADERS['Access-Control-Allow-Origin'] = origin;
 };
 
-class Server {
+class Server extends EventEmitter {
   constructor(application, options) {
+    super();
+    this.retry = options.retry ?? DEFAULT_LISTEN_RETRY;
     this.application = application;
     this.options = options;
     this.balancer = options.kind === 'balancer';
@@ -210,8 +213,10 @@ class Server {
       });
     });
 
-    this.httpServer.on('error', (err) => {
+    this.wsServer.on('error', (err) => {
       if (err.code !== 'EADDRINUSE') return;
+      this.retry--;
+      if (this.retry === 0) return void this.emit('error', err);
       console.warn(`Address in use: ${host}:${port}, retry...`);
       setTimeout(() => {
         this.bind();