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

Off by one when handling basic.return frames? #40

Open
Keruspe opened this issue Nov 5, 2016 · 2 comments
Open

Off by one when handling basic.return frames? #40

Keruspe opened this issue Nov 5, 2016 · 2 comments

Comments

@Keruspe
Copy link

Keruspe commented Nov 5, 2016

If I try to publish a message to a queue that no longer exists with the mandatory field set to true, the publish "succeeds", but then if I try to start a consumer, I get a "basic.return" instead of a "basic.consume-ok" thus the creation of the consumer fails.

If mandatory is set to false everything works properly.
If I ignore the error the consumer works properly.

@Antti
Copy link
Owner

Antti commented Dec 13, 2016

Is this happening in the same program and the same channel?
Unfortunately asynchronous methods like basic.publish, basic.return are not handled properly yet. There should be a generic handler for such methods, which will handle them in the right way.
I implemented a better way of handling them in the tokio branch, but it's not ready yet.

@Keruspe
Copy link
Author

Keruspe commented Dec 13, 2016

Same channel, yup.
I can provide a simple reproducer if needed

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

No branches or pull requests

2 participants