[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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.