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

Fix socketio connections leak & deactivate this protocol by default #1483

Merged
merged 2 commits into from
Oct 7, 2019

Conversation

scottinet
Copy link
Contributor

Description

Same as #1482 , with the following additional change: the socketio protocol is now deactivated by default (files default.config.js and .kuzzlerc.sample)

@scottinet scottinet self-assigned this Oct 2, 2019
@scottinet scottinet changed the title Fix socketio connections leak Fix socketio connections leak & deactivate this protocol by default Oct 2, 2019
@codecov
Copy link

codecov bot commented Oct 2, 2019

Codecov Report

Merging #1483 into 2-dev will decrease coverage by <.01%.
The diff coverage is 94.73%.

Impacted file tree graph

@@            Coverage Diff             @@
##            2-dev    #1483      +/-   ##
==========================================
- Coverage   92.64%   92.63%   -0.01%     
==========================================
  Files          98       98              
  Lines        6637     6644       +7     
==========================================
+ Hits         6149     6155       +6     
- Misses        488      489       +1
Impacted Files Coverage Δ
lib/api/core/entrypoints/index.js 98.88% <100%> (+0.01%) ⬆️
lib/api/core/entrypoints/protocols/socketio.js 92.2% <92.85%> (-0.85%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7764d8...db13788. Read the comment docs.

@Aschen Aschen merged commit a11850b into 2-dev Oct 7, 2019
@Aschen Aschen deleted the deactivate-socketio branch October 7, 2019 05:34
@Aschen Aschen mentioned this pull request Nov 5, 2019
Aschen added a commit that referenced this pull request Nov 5, 2019
# [2.0.0-rc.2](https://github.com/kuzzleio/kuzzle/releases/tag/2.0.0-rc.2) (2019-11-05)


#### Breaking changes

- [ [#1500](#1500) ] Remove Dsl constructor from plugin context   ([Aschen](https://github.com/Aschen))
- [ [#1489](#1489) ] Drop support for socket.io   ([scottinet](https://github.com/scottinet))
- [ [#1491](#1491) ] Remove strategy constructors support   ([Aschen](https://github.com/Aschen))
- [ [#1476](#1476) ] Change Document controller m* routes returns & fix SDK functional tests   ([Aschen](https://github.com/Aschen))

#### Bug fixes

- [ [#1512](#1512) ] Fix subscribing with a dead connection   ([benoitvidis](https://github.com/benoitvidis))
- [ [#1507](#1507) ] Properly destroy a user after a credentials creation failure   ([scottinet](https://github.com/scottinet))
- [ [#1483](#1483) ] Fix socketio connections leak & deactivate this protocol by default   ([scottinet](https://github.com/scottinet))

#### New features

- [ [#1502](#1502) ] Add a new "document:exists" API route   ([scottinet](https://github.com/scottinet))
- [ [#1501](#1501) ] Add collection delete   ([Aschen](https://github.com/Aschen))
- [ [#1499](#1499) ] Expose an Elasticsearch client constructor in plugin context   ([Aschen](https://github.com/Aschen))
- [ [#1484](#1484) ] Upgrade script   ([scottinet](https://github.com/scottinet))

#### Enhancements

- [ [#1510](#1510) ] Mapping - enable dynamic templates   ([benoitvidis](https://github.com/benoitvidis))
- [ [#1495](#1495) ] Check for SDK compatibility against current Kuzzle version   ([Aschen](https://github.com/Aschen))
- [ [#1498](#1498) ] Add highlight to search results   ([Aschen](https://github.com/Aschen))
- [ [#1488](#1488) ] Add kuzzle version to the dumped informations   ([scottinet](https://github.com/scottinet))
- [ [#1481](#1481) ] Error codes normalization   ([scottinet](https://github.com/scottinet))
- [ [#1478](#1478) ] Error codes redux   ([scottinet](https://github.com/scottinet))
---
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants