feat(trpc-server): use c.body to create response to avoid header loss

[BlankParticle]

in trpc-server middleware, we currently return the response directly which causes the respose to loose information about context. So if we set a header/cookie inside a trpc route, we don't get that in the respose.

I have fixed it by creating a new response object with c.body and returning that.

A reproduction of the issue can be found at https://github.com/BlankParticle/hono-header-reproduction/tree/main

[changeset-bot]

🦋 Changeset detected

Latest commit: 69a6e7a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@hono/trpc-server	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[yusukebe]

Hi @BlankParticle

Thank you for the PR. Can you see the error on the CI?

[BlankParticle]

Hi @BlankParticle

Thank you for the PR. Can you see the error on the CI?

I didn't get this error while developing. Turns out I was using node 20, I was able to reproduce the errors in node 18. The fix for it is to update hono dep in the package to 4.x.x. Should I update the Hono dependency?

[yusukebe]

LGTM!

[yusukebe]

@BlankParticle

Thanks! Looks good. Updating Hono to v4.* is good. I'll merge this now.

[codeflows]

Thank you so much, was just struggling with this issue!

[codeflows]

It seems setting multiple cookies at once does not work, I've created a PR in the main project to try and fix this.