Skip to content

Commit

Permalink
Merge pull request #110 from the-ress/max-listener-limit
Browse files Browse the repository at this point in the history
Set the max listener limit to 1000 for `RheaConnection`
  • Loading branch information
jeremymeng authored May 2, 2024
2 parents e37362a + c24eb37 commit cd5c42c
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions lib/connection.ts
Original file line number Diff line number Diff line change
Expand Up @@ -202,6 +202,8 @@ export declare interface Connection {
on(event: ConnectionEvents, listener: OnAmqpEvent): this;
}

const maxListenerLimit = 1000;

/**
* Describes the AMQP Connection.
* @class Connection
Expand Down Expand Up @@ -256,6 +258,10 @@ export class Connection extends Entity {
this.options.operationTimeoutInSeconds = options?.operationTimeoutInSeconds ?? defaultOperationTimeoutInSeconds;

this._initializeEventListeners();

// Set max listeners on the connection to 1000 because Session and Link add their own listeners
// and the default value of 10 in NodeJS is too low.
this._connection.setMaxListeners(maxListenerLimit);
}

/**
Expand Down

0 comments on commit cd5c42c

Please sign in to comment.