-
Type: Task
-
Resolution: Done
-
Priority: Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
Use Case
As a Node.js driver engineer
I want to simplify signal handling code
So that it is easy to avoid memory leaks
Currently, it is necessary to overwrite the controller saved to the Connection class because the events.on async generator helper does not remove the abort listener it attaches to its given signal, and there is no way to remove it from the caller.
User Impact
- None (maybe a small performance improvement)
Dependencies
- Connection layer
Unknowns
- The precise solution needed to avoid this hack
Acceptance Criteria
Implementation Requirements
- Remove the creation of a new AbortController on every sendCommand call
Testing Requirements
- The AbortSignal passed to read/write methods must not have an unbounded growth of abort listeners
Documentation Requirements
- None
Follow Up Requirements
- None
- has to be done before
-
NODE-5743 use modern promise connection.command API
- Closed