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

Support 'Cc' in the web mail/reply composer #250

Merged
merged 2 commits into from
Apr 30, 2024
Merged

Conversation

raboof
Copy link
Member

@raboof raboof commented Aug 30, 2023

Fixes #249

Tested with https://github.com/mailhog/MailHog that:

  • The Cc header is added to the payload
  • The Bcc header is not added to the payload
  • The To, Cc and Bcc addresses are all added to the recipients of the email

@raboof raboof marked this pull request as ready for review September 1, 2023 09:52
@sebbASF sebbASF merged commit 7aade30 into apache:master Apr 30, 2024
@Humbedooh
Copy link
Member

cc and bcc needs to be verified against the list of valid recipient domains, as with the 'to' field. This could be done in a single iteration over all gathered values.

raboof added a commit to raboof/incubator-ponymail-foal that referenced this pull request May 1, 2024
raboof added a commit to raboof/incubator-ponymail-foal that referenced this pull request May 1, 2024
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.

Support 'Cc' when using the integrated mail/reply composer
3 participants