Improve the re-announce timings

Author: xploSEoF

lib/registry.js

@@ -3,6 +3,7 @@ import Service from './service.js';
 
 const REANNOUNCE_MAX_MS = 60 * 60 * 1000;
 const REANNOUNCE_FACTOR = 3;
+const REANNOUNCE_TIMES = 3;
 
 export default class Registry {
 	_server;
@@ -135,38 +136,31 @@ export default class Registry {
 	 * Broadcasts right away, then after 3 seconds, 9 seconds, 27 seconds,
 	 * and so on, up to a maximum interval of one hour.
 	 */
-	#announce(service) {
-		let delay = 1000;
+	#announce(service, count = 1) {
 		let packet = service._records();
 
 		this._server.register(packet);
 
-		let broadcast = () => {
-			// abort if the service have or is being stopped in the meantime
-			if ( ! service._activated || service._destroyed) {
-				return;
-			}
-
-			service.emit('anouncing', JSON.parse(JSON.stringify(packet)));
-
-			this._server.mdns.respond(packet, () => {
-				// This function will optionally be called with an error object. We'll
-				// just silently ignore it and retry as we normally would
-				if ( ! service.published) {
-					service._activated = true;
-					service.published = true;
-					service.emit('up');
-				}
+		// abort if the service have or is being stopped in the meantime
+		if ( ! service._activated || service._destroyed || count > REANNOUNCE_TIMES) {
+			return;
+		}
 
-				delay = Math.min(delay * REANNOUNCE_FACTOR, REANNOUNCE_MAX_MS);
+		service.emit('anouncing', JSON.parse(JSON.stringify(packet)));
 
-				if ( ! service._destroyed) {
-					// setTimeout(broadcast, delay).unref();
-				}
-			});
-		};
+		this._server.mdns.respond(packet, () => {
+			// This function will optionally be called with an error object. We'll
+			// just silently ignore it and retry as we normally would
+			if ( ! service.published) {
+				service._activated = true;
+				service.published = true;
+				service.emit('up');
+			}
 
-		broadcast();
+			if ( ! service._destroyed) {
+				setTimeout(this.#announce.bind(this, service, count + 1), Math.min(count * REANNOUNCE_FACTOR * 1000, REANNOUNCE_MAX_MS));
+			}
+		});
 	}
 
 	/**