[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