GetSendEndpoint should raise exception on non-existing queue

Is this a bug report?

Yes

Can you also reproduce the problem with the latest version?

Yes

Environment

1. Operating system: Windows 10 Enterprise 64-bit
2. Visual Studio version: VS 2017
3. Dotnet version: .NET Framework 4.6.2

Steps to Reproduce

1. Have a RabbitMQ running locally
2. Create a new Console application
3. Paste following code:

        static void Main()
        {
            var mtBus = Bus.Factory.CreateUsingRabbitMq(busFactoryConfig =>
            {
                busFactoryConfig.Host(new Uri("rabbitmq://localhost"), hostConfig =>
                {
                    hostConfig.Username("admin");
                    hostConfig.Password("admin");
                });
            });

            mtBus.Start();

            //should raise exception
            var result = mtBus.GetSendEndpoint(new Uri("rabbitmq://localhost/testQueue?queue=testQueue&bind=false")).GetAwaiter().GetResult();
        }

1. the last line, GetSendEndpoint should throw an EndpointNotFoundException because testQueue does not exist on RabbitMQ.

No vhost is configured.

Expected Behavior

GetSendEndpoint should throw an EndpointNotFoundException because testQueue does not exist on RabbitMQ.

Actual Behavior

It just continues and creates an exchange with name testQueue with no bindings, messages are discarded.

Reproducible Demo

Run a local RabbitMQ server and create a console app with code below:

        static void Main()
        {
            var mtBus = Bus.Factory.CreateUsingRabbitMq(busFactoryConfig =>
            {
                busFactoryConfig.Host(new Uri("rabbitmq://localhost"), hostConfig =>
                {
                    hostConfig.Username("admin");
                    hostConfig.Password("admin");
                });
            });

            mtBus.Start();

            //should raise exception
            var result = mtBus.GetSendEndpoint(new Uri("rabbitmq://localhost/testQueue?queue=testQueue&bind=false")).GetAwaiter().GetResult();
        }

[ozdotnet]

GetSendEndpoint(new Uri("rabbitmq://localhost/testQueue?queue=testQueue&bind=false"));

Will create the exchange queue. The proper syntax for this address is:

var result = mtBus.GetSendEndpoint(new Uri("rabbitmq://localhost/testQueue"));

Exchange is still created in the latter way. Anyway, my question was different…

Expected Behavior GetSendEndpoint should throw an EndpointNotFoundException because testQueue does not exist on RabbitMQ.

In response to our chat on Gitter; Indeed the behavior was always the same, GetSendEnpoint never threw an exception in previous versions of MassTransit. I’m just questioning the design, A Send should always put the given message in the given queue right? If queue/exchange doesn’t exist, then an exception should be thrown in my opinion. Just like the mandatory parameter when doing a publish

[youqingz]

If you’re using RabbitMQ, and pass queue:name as the destination address, it will create the queue using the default queue parameters (durable=true, autoDelete=false).

Oh cool, just discovered short addresses, coming from an older version of MT. For others like me: var result = mtBus.GetSendEndpoint(new Uri("queue:testQueue"));