Merge pull request #35 from JacobLinCool/fix-error-handling

fix: remove redundant error event listeners in Tunnel class

Author: JacobLinCool

.changeset/pink-spoons-smash.md

@@ -0,0 +1,5 @@
+---
+"cloudflared": patch
+---
+
+Remove redundant error event listeners in Tunnel class

examples/tunnel.js

@@ -1,9 +1,15 @@
-const { Tunnel } = require("cloudflared");
+const fs = require("node:fs");
+const { Tunnel, bin, install } = require("cloudflared");
 
 console.log("Cloudflared Tunnel Example.");
 main();
 
 async function main() {
+    if (!fs.existsSync(bin)) {
+        // install cloudflared binary
+        await install(bin);
+    }
+
     // run: cloudflared tunnel --hello-world
     const tunnel = Tunnel.quick();
 

examples/tunnel.mjs

@@ -1,9 +1,15 @@
-import { Tunnel } from "cloudflared";
+import fs from "node:fs";
+import { Tunnel, bin, install } from "cloudflared";
 
 console.log("Cloudflared Tunnel Example.");
 main();
 
 async function main() {
+    if (!fs.existsSync(bin)) {
+        // install cloudflared binary
+        await install(bin)
+    }
+
     // run: cloudflared tunnel --hello-world
     const tunnel = Tunnel.quick();
 

src/tunnel.ts

@@ -76,14 +76,10 @@ export class Tunnel extends EventEmitter {
         // cloudflared outputs to stderr, but I think its better to listen to stdout too
         this.on("stdout", (output) => {
             this.processOutput(output);
-        }).on("error", (err) => {
-            this.emit("error", err);
         });
 
         this.on("stderr", (output) => {
             this.processOutput(output);
-        }).on("error", (err) => {
-            this.emit("error", err);
         });
     }