[OpenNMS/opennms] dc30e2: NMS-20102: Address Copilot review feedback
Marshall Massengill via opennms-cvs <[email protected]> Thu, 30 Jul 2026 11:09:30 -0700
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/mm/NMS-20102-smoke/[email protected]> |
Branch: refs/heads/mm/NMS-20102-smoke
Home: https://github.com/OpenNMS/opennms
Commit: dc30e2210c7828b72c8d9ea1970030b5d65367dd
https://github.com/OpenNMS/opennms/commit/dc30e2210c7828b72c8d9ea1970030b5d65367dd
Author: Marshall Massengill <[email protected]>
Date: 2026-07-30 (Thu, 30 Jul 2026)
Changed paths:
M opennms-javamail/opennms-javamail-api/src/main/java/org/opennms/javamail/JavaSendMailer.java
M opennms-javamail/opennms-javamail-api/src/test/java/org/opennms/javamail/JavaMailerWireTest.java
Log Message:
-----------
NMS-20102: Address Copilot review feedback
JavaSendMailer.buildMimeMessage() parses the to address with
InternetAddress.parse(to, false), matching JavaMailer, so
comma-separated recipient lists in sendmail-message work; the previous
single-address constructor matched the old MimeMessageHelper behavior
but diverged from the rest of the module. Wire test now sends to two
recipients. Also drops the duplicated mail.smtp.quitwait defaulting
block (pre-existing) and restores the opennms.home system property
after each JavaMailerWireTest.
To unsubscribe from these emails, change your notification settings at https://github.com/OpenNMS/opennms/settings/notifications
_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/wiki/index.php?page=MailingListFaq
opennms-cvs mailing list
To *unsubscribe* or change your subscription options, see the bottom of this page:
https://lists.sourceforge.net/lists/listinfo/opennms-cvs