-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Snyk] Upgrade ws from 8.13.0 to 8.16.0 #16
base: master
Are you sure you want to change the base?
Conversation
Snyk has created this PR to upgrade ws from 8.13.0 to 8.16.0. See this package in npm: https://www.npmjs.com/package/ws See this project in Snyk: https://app.snyk.io/org/sammytezzy/project/801c3797-5f02-4e73-9d0b-aff26283dffb?utm_source=github&utm_medium=referral&page=upgrade-pr
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Run & review this pull request in StackBlitz Codeflow. |
|
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.
PR Type: Enhancement
PR Summary: This pull request updates the 'ws' package from version 8.13.0 to 8.16.0. It aims to keep the project's dependencies up-to-date, addressing potential vulnerabilities and ensuring compatibility with the latest features and improvements offered by the 'ws' package. The upgrade includes several notable changes, such as the introduction of the 'autoPong' option and adjustments to event handling, which could enhance the project's WebSocket functionality.
Decision: Comment
📝 Type: 'Enhancement' - not supported yet.
- Sourcery currently only approves 'Typo fix' PRs.
✅ Issue addressed: this change correctly addresses the issue or implements the desired feature.
No details provided.
📝 Complexity: the changes are too large or complex for Sourcery to approve.
- Unsupported files: the diff contains files that Sourcery does not currently support during reviews.
General suggestions:
- Ensure thorough testing of WebSocket-related functionality to verify that the upgrade does not introduce any regressions.
- Review the release notes and commit messages associated with the 'ws' package upgrade to understand the impact of the new features and changes on your project.
- Consider the potential breaking change introduced in version 8.15.1 regarding the renaming of an option. Verify that this change is accounted for in your project's usage of the 'ws' package.
Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨
This PR was automatically created by Snyk using the credentials of a real user.
Snyk has created this PR to upgrade ws from 8.13.0 to 8.16.0.
ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.
Release notes
Package name: ws
Features
autoPong
option (01ba54e).Notable changes
allowMultipleEventsPerMicrotask
option has been renamed toallowSynchronousEvents
(4ed7fe5).This is a breaking change in a patch release that could have been avoided with
an alias, but the renamed option was added only 3 days ago, so hopefully it
hasn't already been widely used.
Features
allowMultipleEventsPerMicrotask
option (93e3552).Bug fixes
swallowed when running tests (7f4e1a7).
Bug fixes
Features
WebSocket
constructor now accepts HTTP(S) URLs (#2162).socket
argument ofserver.handleUpgrade()
can now be a genericDuplex
stream (#2165).Other notable changes
Features
finishRequest
option to support late addition of headers (#2123).Commit messages
Package name: ws
[HMR] Waiting for update signal from WDS...
is not affected byclientLogLevel: 'silent'
webpack/webpack-dev-server#2166)Compare
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs