Port config parameter not used while creating rabbit connection factory

[tomas0svk]

DynamicRabbitMQDefinitionRegistrar::postProcessBeanDefinitionRegistry does not configure rabbit broker port on connectionFactory.

It is misleading, because RabbitConnectionConfiguration as well as examples express ability to configure port (however property is unused).
Hence it is was not possible to mock rabbit feed using TestContainers.

[tomas0svk]

Similar topic ...
In trade360-samples you have misconfigured consumer:

rabbitmq.prematch.network_recovery_interval:30
It is value in millis, not in seconds, so it tries to reconnect every 30ms

[shirbenishay]

Thank you for raising this.

After reviewing the implementation, we have decided not to change this behavior at this time. The current configuration flow is aligned with the existing SDK design, and modifying it could introduce unnecessary breaking changes for consumers relying on the established setup.

Therefore, we will not be applying a fix for the port configuration handling in the near term.

We appreciate the feedback.

[tomas0svk]

Fixing the bug "could introduce unnecessary breaking changes for consumers", really?