Cors headers dropped in miniflare@v2

[maraisr]

With a worktop project, cors headers seem to be dropped in miniflare v2 as opposed to v1.

import { Router } from "worktop";
import { preflight } from "worktop/cors";
import * as Module from "worktop/module";
import { send } from "worktop/response";

const API = new Router();

API.prepare = preflight({
	methods: ["POST", "GET"],
	origin: false,
});

API.add("POST", "/test", async () => {
    return send(200);
});

export default Module.reply(API.run);

curl -v -X OPTIONS localhost:8787/test

Notice no;

< Access-Control-Allow-Origin: *
< Access-Control-Allow-Methods: POST,GET

Despite miniflare v1 has these.

[mrbbot]

Hey! 👋 Which version of worktop are you using? I can't seem to import send from worktop/response?

I've tried the example here (https://github.com/lukeed/worktop/tree/master/examples/workers/cors) and it appears to be working in Miniflare 2. 😕

There's a chance this will be fixed by an issue with enumerable properties on Responses I've just fixed though. I'm hoping to get another pre-release you could try out soon. 👍

[maraisr]

ah! 🤪😅 silly me should have told you which versions 😔

worktop@next ☝🏻

[mrbbot]

Hey! 👋 miniflare@2.0.0-next.2 has just been released, including a fix for this. You can find the changelog here. Please let me know if you have any other issues, and feel free to ask questions in the #miniflare channel of the Cloudflare Workers Discord server.