[OpenNMS/opennms] b15b15: NMS-20118: harden event-notification deletes and u...

joseanesONMS via opennms-cvs <[email protected]> Fri, 31 Jul 2026 10:28:44 -0700
Newsgroups gmane.network.opennms.cvs
Message-ID <OpenNMS/opennms/push/refs/heads/jira/NMS-20118-event-notifications/[email protected]>
  Branch: refs/heads/jira/NMS-20118-event-notifications
  Home:   https://github.com/OpenNMS/opennms
  Commit: b15b1505b16a928963d5e568719ecfb3165b1049
      https://github.com/OpenNMS/opennms/commit/b15b1505b16a928963d5e568719ecfb3165b1049
  Author: Jose Anes <[email protected]>
  Date:   2026-07-31 (Fri, 31 Jul 2026)

  Changed paths:
    A opennms-webapp-rest/127.0.0.1.tm0.epoch
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/NotificationConfigRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/NotificationConfigRestServiceIT.java
    A opennms-webapp-rest/tmlog0.log

  Log Message:
  -----------
  NMS-20118: harden event-notification deletes and updates

Deleting the last event notification is rejected: notifications.xsd
requires at least one, so the schema-validated save throws after memory
was already emptied and the divergence survives until a restart.
Validation now covers status and parameter names/values, the remaining
fields whose asserting setters could kill the factory's in-place update
chain after part of the live entry was overwritten. The validation
comment describes the actual factory behavior for both paths.



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