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

Allows multiple reply-to addresses #17321

Merged
merged 1 commit into from
May 19, 2021

Conversation

cescoffier
Copy link
Member

This change to not break the API. The only difference is if you pass multiple reply-to address, the getReplyTo will get a comma-separated list (as indicated by the RFC).

Fix #17308

@cescoffier cescoffier requested a review from mkouba May 18, 2021 13:49
@quarkus-bot
Copy link

quarkus-bot bot commented May 18, 2021

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should not end up with dot

This message is automatically generated by a bot.

@cescoffier cescoffier changed the title Allows multiple reply-to addresses. Allows multiple reply-to addresses May 18, 2021
This change to not break the API. The only difference is if you pass multiple reply-to address, the getReplyTo will get a comma-separated list (as indicated by the RFC).

Fix quarkusio#17308
@cescoffier cescoffier force-pushed the fix-17308-multiple-reply-to branch from baba4d5 to b28f88c Compare May 19, 2021 14:41
@gastaldi gastaldi dismissed gsmet’s stale review May 19, 2021 15:26

Changes addressed

@gastaldi gastaldi merged commit 9506d8c into quarkusio:main May 19, 2021
@quarkus-bot quarkus-bot bot added this to the 2.0 - main milestone May 19, 2021
@cescoffier cescoffier deleted the fix-17308-multiple-reply-to branch August 1, 2021 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quarkus Mailer - support multiple replyTo Addresses
4 participants