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

fixes qosBasic and rejectBasic #17

Conversation

bradjorgensen
Copy link

Adds the needed qosBasic things to the appropriate state arrays so it will work.
Moves rejectBasic to the correct channel state array.
fixes #16

@sanjay-saxena
Copy link

Thanks for looking into these issues! If you have sample code snippet that was used to test these changes, we can add them to the integration test suite.

@robinzimmermann
Copy link
Contributor

Hi Brad. In the meantime, please look in CONTRIBUTING.md for instructions on electronically signing the Contributor License Agreement (CLA). This is necessary before any pull request can be merged.

@sanjay-saxena
Copy link

Hi Brad -- Can you please include the sample code snippet that you had used to test your changes? We can add it to our test suite. In order for this pull request to make into AMQP 4.0.7, we need to add a test.

@sbadugu sbadugu closed this May 27, 2015
@sbadugu
Copy link

sbadugu commented May 27, 2015

There is a issue filed for this and it will be addressed here

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

Successfully merging this pull request may close these issues.

rejectBasic and qosBasic don't work
4 participants