Merge pull request #18 from steel-dev/nasr/fix-wrong-target-type

fix(api): logging listeners not being used
steel-dev · Dec 3, 2024 · bd8accb · bd8accb
2 parents 5f7c4f3 + b886f18
commit bd8accb
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/api/src/services/cdp.service.ts b/api/src/services/cdp.service.ts
@@ -125,7 +125,7 @@ export class CDPService extends EventEmitter {
   }
 
   private async handleNewTarget(target: Target) {
-    if (target.type() !== "page") {
+    if (target.type() === "page") {
       const page = await target.page().catch((e) => {
         this.logger.error(`Error handling new target in CDPService: ${e}`);
         return null;
@@ -181,7 +181,6 @@ export class CDPService extends EventEmitter {
         });
 
         page.on("console", (message) => {
-          console.log("console - ", message.text());
           this.logger.info(`Console message: ${message.type()}: ${message.text()}`);
           this.customEmit(EmitEvent.Log, {
             type: BrowserEventType.Console,