RabbitMqPortal needs to be revisited because it's not functional in the current state

[StefanOssendorf]

The RabbitMqPortal implementation has two constructors.

csla/Source/Csla.Channels.RabbitMq/RabbitMqPortal.cs

Line 33 in a60f4da

public RabbitMqPortal(ApplicationContext applicationContext, IDataPortalServer dataPortal)

csla/Source/Csla.Channels.RabbitMq/RabbitMqPortal.cs

Line 58 in a60f4da

public RabbitMqPortal(string dataPortalUrl)

As you can see we can either set the necessary services or the data portal uri but NOT both. Hence I dare to say this implementation doesn't work as inteded.
So we should revisit this implementation to make it right.

Version and Platform
CSLA version: Current

[rockfordlhotka]

Agreed. It should work with an options class like HttpProxy.

[rockfordlhotka]

I wonder if GrpcPortal has the same issue?

[StefanOssendorf]

I wonder if GrpcPortal has the same issue?

I don't think so. Or at least I did not create an issue during adding the nullable stuff :D

[Freelancingonupwork]

Hello, i am not able to find any code related to this. May I know how I test it?

[rockfordlhotka]

Hello, i am not able to find any code related to this. May I know how I test it?

It was originally something I created as a proof of concept, and so isn't well documented. I have now created a readme describing the way it should work.

The RabbitMqPortal type should really be getting its configuration from a RabbitMqPortalOptions instance (that type doesn't exist). And there should be a static RabbitMqPortalServerExtensions class that implements a UseRabbitMqPortal extension method that can be used on the host to configure the data portal host.