[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