flakey5
commented
1 year ago I think it looks good but I'd appreciate it if we could reach a consensus on how we'd like to do the rewrites, either in the worker as done in this pr or via Cloudflare rules like @jbergstroem suggested. Cloudflare rules might overlap with the Terraform changes happening now so I'd be in favor of merging this pr and then later on if we think there's benefits in using the Cloudflare rules (e.g. better perf, less worker cpu time, whatever) we can explore it more
Fixes https://github.com/nodejs/release-cloudflare-worker/issues/33