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
Allow more than one Reply-To contact in order to be compliant with the Mail RFC #265
Allow more than one Reply-To contact in order to be compliant with the Mail RFC #265
Conversation
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.
what about postmark and mailjet?
email-amazon-ses/src/test/groovy/io/micronaut/email/ses/SesEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-amazon-ses/src/test/groovy/io/micronaut/email/ses/SesEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-amazon-ses/src/test/groovy/io/micronaut/email/ses/SesEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-amazon-ses/src/test/groovy/io/micronaut/email/ses/SesEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-amazon-ses/src/test/groovy/io/micronaut/email/ses/SesEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-sendgrid/src/test/groovy/io/micronaut/email/sendgrid/SendgridEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-sendgrid/src/test/groovy/io/micronaut/email/sendgrid/SendgridEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-sendgrid/src/test/groovy/io/micronaut/email/sendgrid/SendgridEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
email-amazon-ses/src/test/groovy/io/micronaut/email/ses/SesEmailComposerSpec.groovy
Outdated
Show resolved
Hide resolved
Co-authored-by: Sergio del Amo <sergio.delamo@softamo.com>
…ridEmailComposer.java Co-authored-by: Sergio del Amo <sergio.delamo@softamo.com>
Those two weren't even implementing single "replyTo" address support. I just pushed some changes to make them take "replyTo" into account. |
0e63993
to
1d8a179
Compare
@guillermocalvo can you merge master into this PR. thanks. I will merge it after you have resolved the conflicts. |
…ore-than-one-reply-to-contact-in-order-to-be-compliant-with-the-mail-rfc
Kudos, SonarCloud Quality Gate passed! |
Added support for multiple
replyTo
perEmail
in a backward-compatible way.Sendgrid does not support multiple 'replyTo' addresses, so we will log a warning.
Closes #217