[OpenNMS/opennms] 9e5eed: NMS-20100: copy-on-write destination paths, fresh ...
joseanesONMS via opennms-cvs <[email protected]>
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/jira/NMS-20100/[email protected]> |
Branch: refs/heads/jira/NMS-20100
Home: https://github.com/OpenNMS/opennms
Commit: 9e5eedf9e17dad0b502903a4022e20a2a8bf6049
https://github.com/OpenNMS/opennms/commit/9e5eedf9e17dad0b502903a4022e20a2a8bf6049
Author: Jose Anes <[email protected]>
Date: 2026-09-02 (Wed, 02 Sep 2026)
Changed paths:
M opennms-config/src/main/java/org/opennms/netmgt/config/DestinationPathManager.java
M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NotificationConfigRestService.java
M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/NotificationConfigRestServiceIT.java
M ui/src/components/AdminNotifications/DestinationPathEditorDialog.vue
M ui/src/components/AdminNotifications/EventNotificationEditorDialog.vue
M ui/src/lib/adminValidation.ts
Log Message:
-----------
NMS-20100: copy-on-write destination paths, fresh role names, addressable names !smoke
DestinationPathManager mutated its TreeMap in place under the mutator's
lock while getPath/getPaths read it unlocked. Every change now applies to
a copy, persists that copy, and publishes the reference only on success,
which also removes the snapshot-and-restore code: a published map is never
mutated again, so unlocked readers always see a consistent snapshot.
Target validation read role names from the cached group config; user and
group names refresh on read but getRoleNames does not, so a role added by
hand or through the on-call roles API was rejected until an unrelated read
reloaded the file. Notification and destination-path names may no longer
contain '/', '\', '%' or control characters, which the HTTP firewall
rejects as a path segment, so such an entry could be created but never
edited or deleted; both editor dialogs flag the same rule inline.
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