@browserless.io/browserless 2.12.0-beta-4 → 2.12.0

package/src/shared/json-protocol.http.ts

@@ -27,11 +27,7 @@ export default class ChromiumJSONProtocolGetRoute extends HTTPRoute {
   path = HTTPRoutes.jsonProtocol;
   tags = [APITags.browserAPI];
 
-  handler = async (
-    _req: Request,
-    res: Response,
-    logger: Logger,
-  ): Promise<void> => {
+  async handler(_req: Request, res: Response, logger: Logger): Promise<void> {
     const browserManager = this.browserManager();
 
     if (!this.cachedProtocol) {
@@ -39,5 +35,5 @@ export default class ChromiumJSONProtocolGetRoute extends HTTPRoute {
     }
 
     return jsonResponse(res, 200, this.cachedProtocol);
-  };
+  }
 }

package/src/shared/json-version.http.ts

@@ -31,11 +31,7 @@ export default class ChromiumJSONVersionGetRoute extends HTTPRoute {
   method = Methods.get;
   path = HTTPRoutes.jsonVersion;
   tags = [APITags.browserAPI];
-  handler = async (
-    req: Request,
-    res: Response,
-    logger: Logger,
-  ): Promise<void> => {
+  async handler(req: Request, res: Response, logger: Logger): Promise<void> {
     const baseUrl = req.parsed.host;
     const protocol = req.parsed.protocol.includes('s') ? 'wss' : 'ws';
 
@@ -56,5 +52,5 @@ export default class ChromiumJSONVersionGetRoute extends HTTPRoute {
         contentTypes.text,
       );
     }
-  };
+  }
 }

package/src/shared/page.ws.ts

@@ -29,11 +29,13 @@ export default class ChromiumPageWebSocketRoute extends BrowserWebsocketRoute {
   );
   path = WebsocketRoutes.page;
   tags = [APITags.browserWS];
-  handler = async (
+  async handler(
     req: Request,
     socket: Duplex,
     head: Buffer,
     _logger: Logger,
     browser: ChromiumCDP,
-  ): Promise<void> => browser.proxyPageWebSocket(req, socket, head);
+  ): Promise<void> {
+    return browser.proxyPageWebSocket(req, socket, head);
+  }
 }

package/src/shared/pdf.http.ts

@@ -80,12 +80,12 @@ export default class ChromiumPDFPostRoute extends BrowserHTTPRoute {
   method = Methods.post;
   path = [HTTPRoutes.pdf, HTTPRoutes.chromiumPdf];
   tags = [APITags.browserAPI];
-  handler = async (
+  async handler(
     req: Request,
     res: ServerResponse,
     logger: Logger,
     browser: BrowserInstance,
-  ): Promise<void> => {
+  ): Promise<void> {
     logger.info('PDF API invoked with body:', req.body);
     const contentType =
       !req.headers.accept || req.headers.accept?.includes('*')
@@ -246,5 +246,5 @@ export default class ChromiumPDFPostRoute extends BrowserHTTPRoute {
     page.close().catch(noop);
 
     logger.info('PDF API request completed');
-  };
+  }
 }

package/src/shared/performance.http.ts

@@ -45,12 +45,12 @@ export default class PerformancePost extends BrowserHTTPRoute {
   method = Methods.post;
   path = [HTTPRoutes.performance, HTTPRoutes.chromiumPerformance];
   tags = [APITags.browserAPI];
-  handler = async (
+  async handler(
     req: Request,
     res: ServerResponse,
     _logger: Logger,
     browser: BrowserInstance,
-  ): Promise<void> => {
+  ): Promise<void> {
     const config = this.config();
     const response = await main({
       browser,
@@ -60,5 +60,5 @@ export default class PerformancePost extends BrowserHTTPRoute {
     });
 
     return jsonResponse(res, 200, response);
-  };
+  }
 }

package/src/shared/scrape.http.ts

@@ -216,12 +216,12 @@ export default class ChromiumScrapePostRoute extends BrowserHTTPRoute {
   method = Methods.post;
   path = [HTTPRoutes.scrape, HTTPRoutes.chromiumScrape];
   tags = [APITags.browserAPI];
-  handler = async (
+  async handler(
     req: Request,
     res: ServerResponse,
     logger: Logger,
     browser: BrowserInstance,
-  ) => {
+  ) {
     logger.info('Scrape API invoked with body:', req.body);
     const contentType =
       !req.headers.accept || req.headers.accept?.includes('*')
@@ -440,5 +440,5 @@ export default class ChromiumScrapePostRoute extends BrowserHTTPRoute {
     logger.info('Scrape API request completed');
 
     return jsonResponse(res, 200, response, false);
-  };
+  }
 }

package/src/shared/screenshot.http.ts

@@ -83,12 +83,12 @@ export default class ScreenshotPost extends BrowserHTTPRoute {
   method = Methods.post;
   path = [HTTPRoutes.screenshot, HTTPRoutes.chromiumScreenshot];
   tags = [APITags.browserAPI];
-  handler = async (
+  async handler(
     req: Request,
     res: ServerResponse,
     logger: Logger,
     browser: BrowserInstance,
-  ): Promise<void> => {
+  ): Promise<void> {
     logger.info('Screenshot API invoked with body:', req.body);
     const contentType =
       !req.headers.accept || req.headers.accept?.includes('*')
@@ -267,5 +267,5 @@ export default class ScreenshotPost extends BrowserHTTPRoute {
 
     page.close().catch(noop);
     logger.info('Screenshot API request completed');
-  };
+  }
 }

package/src/shared/utils/function/client.ts

@@ -11,7 +11,9 @@ export class FunctionRunner {
   protected browser?: Browser;
   protected page?: Page;
 
-  public log = () => console.log.bind(console);
+  public log() {
+    return console.log.bind(console);
+  }
 
   public async start(data: {
     browserWSEndpoint: string;

package/src/shim.ts

@@ -14,7 +14,7 @@ const shimParam = ['headless', 'stealth', 'ignoreDefaultArgs', 'slowMo'];
  *
  * @param req A parsed user requests
  */
-export const shimLegacyRequests = (url: URL): URL => {
+export function shimLegacyRequests(url: URL): URL {
   const { searchParams } = url;
   const params = [...searchParams];
   const names = params.map(([k]) => k);
@@ -76,4 +76,4 @@ export const shimLegacyRequests = (url: URL): URL => {
   }
 
   return url;
-};
+}

package/src/token.ts

@@ -14,14 +14,14 @@ export class Token extends EventEmitter {
     super();
   }
 
-  public isAuthorized = async (
+  public async isAuthorized(
     req: Request,
     route:
       | BrowserHTTPRoute
       | BrowserWebsocketRoute
       | HTTPRoute
       | WebSocketRoute,
-  ): Promise<boolean> => {
+  ): Promise<boolean> {
     const token = this.config.getToken();
 
     if (token === null) {
@@ -39,18 +39,18 @@ export class Token extends EventEmitter {
     }
 
     return (Array.isArray(token) ? token : [token]).includes(requestToken);
-  };
+  }
 
   /**
    * Implement any browserless-core-specific shutdown logic here.
    * Calls the empty-SDK stop method for downstream implementations.
    */
-  public shutdown = async () => {
-    await this.stop();
-  };
+  public async shutdown() {
+    return await this.stop();
+  }
 
   /**
    * Left blank for downstream SDK modules to optionally implement.
    */
-  public stop = () => {};
+  public stop() {}
 }

package/src/types.ts

@@ -251,11 +251,11 @@ export abstract class HTTPRoute extends BasicHTTPRoute {
   /**
    * Handles an inbound HTTP request, and supplies the Request and Response objects from node's HTTP request event
    */
-  abstract handler: (
+  abstract handler(
     req: Request,
     res: http.ServerResponse,
     logger: Logger,
-  ) => Promise<unknown>;
+  ): Promise<unknown>;
 }
 
 /**
@@ -272,12 +272,12 @@ export abstract class BrowserHTTPRoute extends BasicHTTPRoute {
    * Handles an inbound HTTP request with a 3rd param of the predefined
    * browser used for the route -- only Chrome CDP is support currently.
    */
-  abstract handler: (
+  abstract handler(
     req: Request,
     res: http.ServerResponse,
     logger: Logger,
     browser: BrowserInstance,
-  ) => Promise<unknown>;
+  ): Promise<unknown>;
 
   /**
    * An optional function to automatically set up or handle new page
@@ -296,12 +296,12 @@ export abstract class WebSocketRoute extends Route {
   /**
    * Handles an inbound Websocket request, and handles the connection
    */
-  abstract handler: (
+  abstract handler(
     req: Request,
     socket: stream.Duplex,
     head: Buffer,
     logger: Logger,
-  ) => Promise<unknown>;
+  ): Promise<unknown>;
 }
 
 /**
@@ -382,7 +382,7 @@ export interface BrowserlessSession {
   isTempDataDir: boolean;
   launchOptions: CDPLaunchOptions | BrowserServerOptions;
   numbConnected: number;
-  resolver: (val: unknown) => void;
+  resolver(val: unknown): void;
   routePath: string | string[];
   startedOn: number;
   ttl: number;

package/src/webhooks.ts

@@ -55,12 +55,12 @@ export class WebHooks extends EventEmitter {
    * Implement any browserless-core-specific shutdown logic here.
    * Calls the empty-SDK stop method for downstream implementations.
    */
-  public shutdown = async () => {
-    await this.stop();
-  };
+  public async shutdown() {
+    return await this.stop();
+  }
 
   /**
    * Left blank for downstream SDK modules to optionally implement.
    */
-  public stop = () => {};
+  public stop() {}
 }