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

chore(dev-server): upgrade fastify & related dependencies in #465

Merged
merged 3 commits into from
Nov 17, 2023

Conversation

jbroma
Copy link
Member

@jbroma jbroma commented Nov 17, 2023

Summary

Upgrade fastify and related packages to v4 and v4 compatible.

Migration included changing from deprecated instance.listen() variadic function to currently supported one with configuration object.

This PR also fixes an issue where dev-server was only listening on only IPv4 or IPv6. Now, when running on localhost, both interfaces are being used for listening for connections.

Test plan

  • - tested locally in TesterApp

Copy link

changeset-bot bot commented Nov 17, 2023

⚠️ No Changeset found

Latest commit: bbb41b2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

Copy link
Member

@thymikee thymikee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Can you link the issues that this upgrade resolves?

@jbroma
Copy link
Member Author

jbroma commented Nov 17, 2023

Cool! Can you link the issues that this upgrade resolves?

not sure there are any open issues about that, I've discovered that some time ago - many issue simply say that the Android can't find the bundle so that will indirectly help limit those kind of questions :)

@thymikee thymikee merged commit e6308b0 into v4 Nov 17, 2023
3 checks passed
@thymikee thymikee deleted the chore/upgrade-fastify branch November 17, 2023 19:02
jbroma added a commit that referenced this pull request Feb 1, 2024
* chore: upgrade fastify to v4

* refactor(dev-server): use non-deprecated version of listen

* fix(dev-server): use proper name for @fastify/sensible as a plugin dependency
@jbroma jbroma mentioned this pull request Feb 29, 2024
jbroma added a commit that referenced this pull request Feb 29, 2024
* chore: upgrade fastify to v4

* refactor(dev-server): use non-deprecated version of listen

* fix(dev-server): use proper name for @fastify/sensible as a plugin dependency
jbroma added a commit that referenced this pull request Apr 5, 2024
* chore: upgrade fastify to v4

* refactor(dev-server): use non-deprecated version of listen

* fix(dev-server): use proper name for @fastify/sensible as a plugin dependency
jbroma added a commit that referenced this pull request Apr 5, 2024
* chore: upgrade fastify to v4

* refactor(dev-server): use non-deprecated version of listen

* fix(dev-server): use proper name for @fastify/sensible as a plugin dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants