mdevilliers / SignalR.RabbitMq

MessageBus implementation using RabbitMq as the backing store.
MIT License
89 stars 40 forks source link

Complete the Task returned from Send once the message has been queued #14

Closed baralong closed 11 years ago

baralong commented 11 years ago

Added the TaskCompletionSource to the RabbitMqMessageWrapper so that it can be completed once the message is queued. This is required for the Task returned from Groups.Add(...) to complete and to rigger the ".done(..)" or ".fail(...)" on the client.

mdevilliers commented 11 years ago

This looks good - will just need to test locally.

Thanks

mdevilliers commented 11 years ago

I've ran this all day and seems good to go! Thanks again,

Mark

mdevilliers commented 11 years ago

Hey, Just to say I've updated the nuget package with both of the issues you have helped with - the virtual host bug and the Task completion code fix.

Thanks for you help diagnosing both of these issues.

Mark

baralong commented 11 years ago

Thanks, I'll update things today.

Glad to help, and thanks for getting the fix out so quickly.

Doug