fastify / fastify-reply-from

fastify plugin to forward the current http request to another server
MIT License
148 stars 76 forks source link

Update for v5 #359

Closed gurgunday closed 4 months ago

gurgunday commented 5 months ago

There are multiple errors when tap is updated to v18, and it doesn't seem trivial to fix them...

If people really need the release, we can follow-up later to upgrade it

gurgunday commented 4 months ago

Done

elmarcusec commented 4 months ago

@gurgunday @mcollina I have a question. When and in which version do you plan to release changes to upgrade the undici library?

gurgunday commented 4 months ago

This will be released as a major

gurgunday commented 4 months ago

Ok I made a mistake, should've merged to next first 😄

Will fix in a sec

gurgunday commented 4 months ago

This PR is ready, only note I'll leave is that for some reason, in certain cases t.teardown is not functioning and I have to resolve the promises myself, not sure if this is a tap issue or what