Consider making ConnectionFactory.HostName a property

[nkumars]

Hi,

The properties in RabbitMQ.Client.ConnectionFactory such as UserName and Password are actually set as properties (means it has {get;set;}) whereas for HostName does not exist, which actually forces us to use GetField method while the other two can be used with GetProperty.

I dont know the purpose behind this, but I would suggest to have a consistent declaration, i.e., with {get;set;} or without.

Below is the screenshot that describes my explanation.

Regards,
Naveen

[kjnilsson]

Field -> Property is a breaking change as applications using .e.g HostName would need to be recompiled. I propose moving this to the 3.7 (v4.0.0) milestone and address it there.

[michaelklishin]

@kjnilsson fair enough.