Antti / rust-amqp

AMQP client in pure rust. Corresponds to rabbitmq spec.
MIT License
247 stars 45 forks source link

Fails to connect unless URL has trailing // #33

Closed jonnor closed 6 years ago

jonnor commented 8 years ago

For instance both amqp://localhost and amqp://localhost fails, where as amqp://localhost// succeeds.

jonnor commented 8 years ago

It also seems to panic: panicked at 'outport setup failed: Protocol("Unexpected method frame: channel.close, expected: exchange.declare-ok"

emk commented 8 years ago

This appears to be a problem with how the default vhost is handled, judging from my RabbitMQ logs.

andrenth commented 7 years ago

I believe the default vhost should be "/".to_string(). This also causes a panic when Session::new() is used without passing the vhost option explicitly.

whitfin commented 6 years ago

I filed a PR to fix this in #68 - appreciate any feedback you guys might have (and if you want to test it out, go ahead).

emk commented 6 years ago

Thank you for this PR!