elysiajs / elysia

Ergonomic Framework for Humans
https://elysiajs.com
MIT License
9.74k stars 207 forks source link

Renaming handler parameters causes hang on start in v1.1.3 #736

Closed nxht closed 1 month ago

nxht commented 1 month ago

What version of Elysia.JS is running?

1.1.3

What platform is your computer?

Linux 5.15.153.1-microsoft-standard-WSL2 x86_64 x86_64

What steps can reproduce the bug?

import { Elysia } from 'elysia';

const app = new Elysia().onRequest(({ set: a }) => {});
console.log('start');
app.listen(3000);

What is the expected behavior?

start printed and app runs

What do you see instead?

No output and the web server doesn't run

Additional information

This cause plugins that uses bun build / bundler like tobias-kaerst-software/elysia-helmet to hang. Also, the issue is solved on downgrade to 1.1.2

cholasimmons commented 1 month ago

I can confirm, Elysia v1.1.3 does not initialize if you're using .use(helmet())

nxht commented 1 month ago

The issue seems to be fixed in 1.1.4