Skip to content
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

Support Fastify 5 in @connectrpc/connect-fastify #1250

Closed
boan-anbo opened this issue Sep 25, 2024 · 2 comments · Fixed by #1296
Closed

Support Fastify 5 in @connectrpc/connect-fastify #1250

boan-anbo opened this issue Sep 25, 2024 · 2 comments · Fixed by #1296
Assignees
Labels
enhancement New feature or request

Comments

@boan-anbo
Copy link

boan-anbo commented Sep 25, 2024

Is your feature request related to a problem? Please describe.
Support the Fastify v5: https://github.com/fastify/fastify/releases/tag/v5.0.0

Describe the solution you'd like
Can support the Fastify v5 and install the npm plugin package without conflicts. Currently it shows a conflict between 4.22.1 which [email protected] uses.

@boan-anbo boan-anbo added the enhancement New feature or request label Sep 25, 2024
@srikrsna-buf
Copy link
Member

srikrsna-buf commented Sep 25, 2024

Thank you for opening the issue, we definitely want to support v5. Can you send a PR for this?

@timostamm
Copy link
Member

From a quick look at the migration guide, this should be simple.

We can relax the peer dependency constraint to accept v5, and update conformance-fastify.yaml to only run in Node.js v20 and later.

@timostamm timostamm changed the title connect-fastify plugin upgrade to support v5? Support Fastify 5 in @connectrpc/connect-fastify Nov 4, 2024
@timostamm timostamm self-assigned this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants