Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Message sequences time out for generic classes (v2) #230

Closed
videege opened this issue May 24, 2017 · 2 comments
Closed

Message sequences time out for generic classes (v2) #230

videege opened this issue May 24, 2017 · 2 comments
Assignees

Comments

@videege
Copy link
Contributor

videege commented May 24, 2017

In v2, the following works fine:

var response = await (_client.ExecuteSequence(p => p.PublishAsync(new HelloWorld()).Complete<HelloWorldResponse>())).Task;

However, if you add a generic wrapping the response class, the message on the queue doesn't get consumed and the sequence times out:

var response = await (_client.ExecuteSequence(p => p.PublishAsync(new HelloWorld()).Complete<Response<HelloWorldResponse>>())).Task;

If you don't use message sequences (instead, setting up a publisher and subscriber independently) it works fine, which leads me to believe there's a problem in the message sequence extension in how it sets up the subscriber for the response, although when I glanced at the logs it looked like it was subscribing to the correct queue.

@pardahlman
Copy link
Owner

Top of the morning, @videege 🌤

You are absolutely right. The reason for this is that I took a shortcut when setting the routing key in the message sequence. It should use naming convention, but it doesn't.

This will be fixed in the next release of 2.0!

@pardahlman
Copy link
Owner

Fixed in beta7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants