fix: RabbitMQ connection not closing

[Icidis]

This pull request fixes issue #157

[ghost]

All CLA requirements met.

[Icidis]

Good day, any chance we can get this approved? Let me know if you need more information.

[Socolin]

Hello,

If this extension is to follow some day the best practice on how to use RabbitMQ.Client (Still waiting for an answer #96), we may have multiple IRabbitMQModel (at least one for consuming and one for publishing, and maybe one for monitor) so closing connection inside RabbittMQModel is not ideal I think.

IMHO: the connection should be reused accross various functions using the same connection string and it should be initialized once and closed when the function stop and we should not need this.

[grimmersnee]

Any idea when this is being merged?

[JatinSanghvi]

Added comment to associate issue #157. Awaiting response.

[ghost]

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 7 days. It will be closed if no further activity occurs within 7 days of this comment.

[Icidis]

Please see comment in #157. I still think this is a valid pull request

[grimmersnee]

Any word on this issue?

[JatinSanghvi]

Reopening the pull request so it stays visible. I had planned to get the issue reproduced, but haven't gotten much time out of other work.

[JatinSanghvi]

Requesting to not spend time in editing the PR to work with the restructured files unless I or someone verifies the issue and confirms here.

[steveb85]

bump on closing this?