Email targets do not handle trailing semicolons

[matt-richardson]

Prerequisites

• I have verified the problem exists in the latest version
• I have searched open and closed issues to make sure it isn't already reported
• I have written a descriptive issue title
• I have linked the original source of this report
• I have tagged the issue appropriately (area/*, kind/bug, tag/regression?)

The bug

A user added a trailing semicolon to the bcc field for an email step, which caused the email step to fail. Our logic does not handle this case.

What I expected to happen

Trailing semicolons, commas, spaces etc should be ignored.

Steps to reproduce

1. Create a project
2. Add a "send email step"
3. set the "to" address to "test@example.com;"
4. Deploy the release
   --> error An invalid character was found in the mail header: ,`

Affected versions

Octopus Server: ~3.0.19

Workarounds

Remove the trailing semicolon

Links

source: customer ticket (private link)

[octoreleasebot]

Release Note: Handled trailing semicolons in the address field of the send email step

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. If you think you've found a related issue, please contact our support team so we can triage your issue, and make sure it's handled appropriately.