Change event emitter handling. - Githubissues

cloudant / nodejs-cloudant

Cloudant Node.js client library

Apache License 2.0

255 stars 90 forks source link

Change event emitter handling. #443

Closed vmatyus closed 3 years ago

vmatyus commented 3 years ago

Checklist

[ ] Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1
[ ] Added tests for code changes or test/build only changes
[ ] Updated the change log file (CHANGES.md|CHANGELOG.md) or test/build only changes
[ ] Completed the PR template below:

Description

Directs the memory leak problem.

Approach

In all of the snapshot the global event emitter also keeps the TokenManager object in the memory. This change clears its content and its events.

Screenshot 2021-05-07 at 12 38 09

Schema & API Changes

Security and Privacy

Testing

Monitoring and Logging

vmatyus commented 3 years ago

This change is not enough.