[OpenNMS/opennms] 43d1b0: NMS-20200: Address review findings on the browser ...
Marshall Massengill via opennms-cvs <[email protected]>
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/mm/NMS-20200-smoke/[email protected]> |
Branch: refs/heads/mm/NMS-20200-smoke
Home: https://github.com/OpenNMS/opennms
Commit: 43d1b00d70a3dbc95c0eba906e7a1cf6d6eb11dd
https://github.com/OpenNMS/opennms/commit/43d1b00d70a3dbc95c0eba906e7a1cf6d6eb11dd
Author: Marshall Massengill <[email protected]>
Date: 2026-08-12 (Wed, 12 Aug 2026)
Changed paths:
M core/web-assets/src/main/assets/js/apps/notifications/index.js
Log Message:
-----------
NMS-20200: Address review findings on the browser notification client
Guard connect() on an existing socket and check in each handler that the
socket is still the current one. A socket dropped by disconnect() still
delivers its close, which would otherwise clear the reference to a newer
socket and schedule a further connect, leaving one open and unreachable.
Fall back to an "OpenNMS" title when head is absent, which is what a
command with no -subject produces once org.json drops the null key.
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