Close the AMQP client on CLI exit

[swansontec]

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

• Yes
• No

Dependencies

none

Description

none

---

• To see the specific tasks where the Asana app for GitHub is being used, see below:
  • https://app.asana.com/0/0/1209057644580861

[samholmes]

Is explicit client closing necessary for the main app (server) between restarts?

[samholmes]

Idea: You can create closeConnections(connections: DbConnections) function to close all connected resources. In there you can either close the connection to the ampqClient like you do here, or you can leverage the reference on the AMQPQueue interface:

function closeConnections(connections: DbConnections): Promise<void> {
  await connections.queue.channel.connection.close()
}

This way you can keep all resource cleanup in this single function and you can avoid leaking the abstraction provided by the queue interface.