Workers says "is not a function" but it totally is!

jasonbyrne662 · June 1, 2022, 2:31am

I’m writing a class called Router with TypeScript. I have an instance of that class with a public method called handle, which is being called successfully to handle the incoming request. This public method has logic to work for both HTTP and Service Workers… it sees that the request is coming from Workers and calls a private method called this.handleEdge

Here’s the problem… Workers dies at that point and says “Uncaught (in promise) TypeError: this.handleEdge is not a function” I’m staring straight at the method. TypeScript is happy. The method is totally there right underneath the public method that it just called.

Does Cloudflare Workers not support private methods or something??

cherryjimbo · June 1, 2022, 3:54am

Can you share your code? Workers uses the most recent V8 version so should absolutely support private class methods.

jasonbyrne662 · June 2, 2022, 2:24am

Here is the router itself:

github.com

jasonbyrne/minikin/blob/master/packages/router/router.ts

import MinikinRequest from "./request";
import MinikinResponse from "./response";
import Handler from "./handler";
import Afterware from "./afterware";
import { AfterCallback, RouteCallback, RouterInit, Routes } from "./interfaces";
import { mapToObject } from "./utils/map-to-object";
import { syncForEach } from "./utils/sync-foreach";

export default class Router {
  public readonly passThroughOnException: boolean;
  public readonly base: string;
  private befores: Handler[] = [];
  private handlers: Handler[] = [];
  private afters: Afterware[] = [];

  public constructor(opts?: RouterInit) {
    if (opts?.routes) this.routes(opts?.routes);
    this.passThroughOnException = !!opts?.passThroughOnException;
    this.base = opts?.base || "";
  }

This file has been truncated. show original

And the code calling it:

import { Router } from "minikin-router";

const router = new Router();

router.route("GET *", () => "foobar");

export default {
  fetch: router.handle,
};

We get to the handle method and then it tries to call handleEdge and reports that the function doesn’t exist.