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

feat: expose AmqpConnectionManagerClass #181

Conversation

luddd3
Copy link
Collaborator

@luddd3 luddd3 commented Aug 30, 2021

Exposes the underlying class and extracts the connect method
its constructor. The reason for this is that we might want to
initiate the connection later or await its result. With this code
change we can also more easily reject on operational errors and
set a timeout for the initial connect.

The naming is strange because I don't want to introduce any
breaking change.

Note that jest complains about leaking resources due to:
amqp-node/amqplib#584

luddd3 and others added 6 commits August 27, 2021 10:38
Provided that they consumed with the channel wrapper function
`consume`
# [3.6.0](jwalton/node-amqp-connection-manager@v3.5.2...v3.6.0) (2021-08-27)

### Features

* reconnect and cancelAll consumers ([fb0c00b](jwalton@fb0c00b))
The naming is strange because I don't want to introduce any
breaking change.

Note that jest complains about leaking resources due to:
amqp-node/amqplib#584
@luddd3 luddd3 force-pushed the feat/expose-amqpconnectionmanager branch from e86ed3c to 835a81f Compare August 30, 2021 06:49
@jwalton jwalton changed the base branch from master to expose-amqpconnectionmanager September 21, 2021 13:16
@jwalton jwalton merged commit 449f5db into jwalton:expose-amqpconnectionmanager Sep 21, 2021
@jwalton
Copy link
Owner

jwalton commented Sep 21, 2021

Merged to master via d209cb5.

@jwalton
Copy link
Owner

jwalton commented Sep 21, 2021

🎉 This PR is included in version 3.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@luddd3 luddd3 deleted the feat/expose-amqpconnectionmanager branch September 21, 2021 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants