[Enhancement]: Protocol specific GetBrokerAddress overload for ActiveMq

[YVbakker]

Problem

I'm using the Artemis container in a project using AmqpNetLite, and I'd like to be able to substitute the connection string 1:1 from the normal configuration. However, GetBrokerAddress() outputs it in the format tcp://{username}:{password}@{address}:{mappedPort}/. AmqpNetLite however, expects amqp:// or amqps://.

Solution

It would be useful to be able to optionally specify the desired protocol: perhaps instead of just a string we could use an enum to limit the input to what Artemis officially supports.

Benefit

Even more usable out-of-the-box (in more use cases than just when using Apache.NMS.ActiveMQ). Now I have to either build my own connection string or use string substitution to change tcp into amqp.

Alternatives

Alternatively we could build separate methods such as GetAmqpBrokerAddress() and GetMqttBrokerAddress() to make intent more clear.

Would you like to help contributing this enhancement?

Yes

[HofmeisterAn]

Yeah, this is a common issue with Testcontainers, and I haven't found a good solution for it yet. In general, it would be helpful to have a way to customize the connection string (see the referenced issue).