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

Include RabbitMQ UserId Header in TransportMessage #45

Merged
merged 1 commit into from
Feb 19, 2019

Conversation

michalsteyn
Copy link
Contributor

@michalsteyn michalsteyn commented Feb 14, 2019

Fixes #44

Preserve the UserId RabbitMq Property when creating the TransportMessage.


Rebus is MIT-licensed. The code submitted in this pull request needs to carry the MIT license too. By leaving this text in, I hereby acknowledge that the code submitted in the pull request has the MIT license and can be merged with the Rebus codebase.

@CLAassistant
Copy link

CLAassistant commented Feb 14, 2019

CLA assistant check
All committers have signed the CLA.

@mookid8000
Copy link
Member

Cool, thanks 👍 the CLA thing does not detect that you've signed it.... it's probably because your GitHub profile is not associated with the email address used in the commit.

If you go to your GitHub profile and add the email address to your profile, then I'll merge your PR right away 😄

@michalsteyn
Copy link
Contributor Author

CLA email profile updated.
Thanks for the quick response :-)

@mookid8000 mookid8000 merged commit 89af174 into rebus-org:master Feb 19, 2019
@mookid8000
Copy link
Member

perfect 👍 it's out as Rebus.RabbitMq 5.0.2 in a few minutes 😄

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.

3 participants