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

Bump engine.io to version 2.0.2 #2864

Merged
merged 1 commit into from
Feb 16, 2017

Conversation

sgress454
Copy link
Contributor

@sgress454 sgress454 commented Feb 16, 2017

Includes the following (from engine.io changelog):

  • [fix] Initialize the WebSocket server in the Server constructor (#476)
  • [chore] Bump ws to version 1.1.2 (vulnerability fix) (#480)

The kind of change this PR does introduce

  • a bug fix
  • a new feature
  • an update to the documentation
  • a code change that improves performance
  • other

Current behaviour

Socket.io installs version 2.0.0 of the engine.io dependency.

New behaviour

Socket.io installs version 2.0.2 of the engine.io dependency.

Other information (e.g. related issues)

IMHO it's worth considering changing this dependency to 'engine-io': '~2.0.0' to avoid having to make new releases of socket.io every time engine.io is patched, since both modules are maintained by the same group. But I can appreciate the desire to lock things down hard.

Includes the following (from engine.io changelog):

* [fix] Initialize the WebSocket server in the Server constructor (socketio#476)
* [chore] Bump ws to version 1.1.2 (vulnerability fix) (socketio#480)
@darrachequesne darrachequesne merged commit 4d8f68c into socketio:master Feb 16, 2017
@darrachequesne darrachequesne added this to the 2.0.0 milestone May 13, 2017
dzad pushed a commit to dzad/socket.io that referenced this pull request May 29, 2023
Includes the following (from engine.io changelog):

* [fix] Initialize the WebSocket server in the Server constructor (socketio#476)
* [chore] Bump ws to version 1.1.2 (vulnerability fix) (socketio#480)
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