Author

Kenton Varda <kenton@cloudflare.com> 2025-04-28 19:38:35 -0500

Parents

6141d69

Tree

b0a17d0

Bugfix: Pass correct ID type to Worker::Actor.

In a Durable Object, the type of `ctx.id` is supposed to be of type `DurableObjectId`, and in production, it is.

Way way back in #605, a bug was introduced such that `ctx.id` became a string for Durable Objects in workerd. But `ctx.id` is only supposed to be a string for "ephemeral objects" (aka colo-local actors).

Somehow, nobody ever noticed? Probably because the only thing most people would do with it anyway is stringify it, which JavaScript is really good at doing automatically.

Anyway, this fixes the bug, restoring the proper type.

(Also this tidies up ownership of the key strings in the actor map. The key is now owned by the ActorContainer.)