-
Notifications
You must be signed in to change notification settings - Fork 29
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
feat: support rabbitmq headers #997
Conversation
Hi @DaAlbrecht , Build failure: The existing tests seems to have failed? Did you try building this locally? |
Hi @aashikam The tests didn't pass because I haven't added support for consuming messages and the corresponding headers. Fixed this now and added a new test. |
Some compiler plugin tests seem to be failing still |
I inspected the error. This comes for the following files but they have correctly used the And the PR does not include any compiler plugin related changes. FixAdded the fix here DaAlbrecht#1 |
Fix compiler plugin test failure
Thank you for the fix! @aashikam merged your pr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Arshika Mohottige <[email protected]>
Purpose
This pull request adds the ability to include custom headers in RabbitMQ message properties. It introduces support for the headers field in BasicProperties and updates the Java code to map these headers during message publishing
Examples
Checklist