set_error_handler doesn't handle string rejections about hyper-express HOT 6 CLOSED

Interesting, I thought that

from hyper-express.

Good catch, seems rejecting with a string was preventing it from being caught as it was not an Error type. The Response.throw() method expects an Error type, so I would assume we would have to wrap non Error type rejections but we would lose the stack trace of the rejection.

from hyper-express.

This issue has been fixed in the recently released v6.2.4 of HyperExpress. Let me know If you encounter any other issues.

from hyper-express.

Thank you so much, this works perfectly. 🥳

from hyper-express.

I'll give that a go today and let you know.

from hyper-express.

Oh found the issue.

// Never responds
webserver.get('/not-working', async () => {
    await new Promise((_resolve, reject) => {
        return reject('Hello World');
    });
});

// Responds
webserver.get('/working', async () => {
    await new Promise((_resolve, reject) => {
        return reject(new Error('Hello World')); // The new Error() is required otherwise it never finishes the request
    });
});

from hyper-express.