New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: properly handle Express.js send payload #221
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, wow!
Oh, sweet! I opened up #222 for the type error - would be great to land it then rebase this PR to verify CI is happy (the unit tests passed, so I assume so 🙂) |
@L2jLiga could you rebase? |
8b3ecf5
to
156812f
Compare
Yep, just pushed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
See fastify/fastify-express#47 (comment) for a reproduction |
Took test from #155
Also reverted changes in
response.js
made in #154fixed in #222tsd
tests seems to fail on master tooChecklist
npm run test
andnpm run benchmark
and the Code of conduct