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

Take message properties into account on sending #28

Merged
merged 4 commits into from
Jun 15, 2017

Conversation

acogoluegnes
Copy link
Contributor

Priority, delivery mode, and expiration aren't taken into
account on sending when set in the JMS message. This is compliant with
the specifiation though. This commit provide a flag in
RMQConnectionFactory to make the RMQMessageProducer use the
message properties if set. This is non-compliant with the specification
but more natural. The existing and spec-compliant behavior is still
the default.

Fixes #26

@michaelklishin
Copy link
Member

Please rebase against master.

Priority, delivery mode, and expiration aren't taken into
account on sending when set in the JMS message. This is compliant with
the specifiation though. This commit provide a flag in
RMQConnectionFactory to make the RMQMessageProducer use the
message properties if set. This is non-compliant with the specification
but more natural. The existing and spec-compliant behavior is still
the default.

Fixes #26
@acogoluegnes
Copy link
Contributor Author

Done.

@michaelklishin michaelklishin merged commit d91af11 into master Jun 15, 2017
@acogoluegnes acogoluegnes deleted the rabbitmq-jms-26 branch March 21, 2019 08:43
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.

None yet

2 participants