[OpenNMS/opennms] 6f238f: NMS-20056: Fix discovery Rest to restrict endpoint...

Chandra Gorantla via opennms-cvs <[email protected]>
Newsgroups gmane.network.opennms.cvs
Message-ID <OpenNMS/opennms/push/refs/heads/cg/jira/NMS-20030/[email protected]>
  Branch: refs/heads/cg/jira/NMS-20030
  Home:   https://github.com/OpenNMS/opennms
  Commit: 6f238fef7ab87a5d2299a4f35eab3008ed36a438
      https://github.com/OpenNMS/opennms/commit/6f238fef7ab87a5d2299a4f35eab3008ed36a438
  Author: Chandra Gorantla <[email protected]>
  Date:   2026-07-27 (Mon, 27 Jul 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/DiscoveryRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/DiscoveryRestServiceIT.java
    M opennms-webapp/src/main/webapp/WEB-INF/applicationContext-spring-security.xml

  Log Message:
  -----------
  NMS-20056: Fix discovery Rest to restrict endpoints to Admins (#8658)


  Commit: 848693946b76a0e244773d90e5214f23d5e8ac7d
      https://github.com/OpenNMS/opennms/commit/848693946b76a0e244773d90e5214f23d5e8ac7d
  Author: Chandra Gorantla <[email protected]>
  Date:   2026-07-29 (Wed, 29 Jul 2026)

  Changed paths:
    M opennms-web-api/src/main/java/org/opennms/web/api/RestUtils.java
    M opennms-web-api/src/main/java/org/opennms/web/svclayer/support/DefaultRequisitionAccessService.java
    A opennms-web-api/src/test/java/org/opennms/web/api/RestUtilsTest.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/AssetRecordResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/CategoryRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/NodeRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/OnmsIpInterfaceResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/OnmsMonitoredServiceResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/OnmsSnmpInterfaceResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/AbstractDaoRestServiceWithDTO.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeCategoriesRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeIpInterfacesRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeSnmpInterfacesRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/CategoryRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/NodeRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/AlarmRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/MonitoringLocationRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/NodeRestServiceIT.java

  Log Message:
  -----------
  NMS-20055: Fix rest endpoints that expose protected fields (#8659)

* NMS-20055: Fix rest endpoints that expose protected fields

* NMS-20055: Address review comments

* NMS-20055: Avoid nested properties update for blocked fields

guard some primary key fields

* NMS-20055: Add some more tests

* NMS-20055: More coverage for some of missed Rest API


  Commit: 1f1ca9541e2e3c3e814dd445462160895645950a
      https://github.com/OpenNMS/opennms/commit/1f1ca9541e2e3c3e814dd445462160895645950a
  Author: Chandra Gorantla <[email protected]>
  Date:   2026-08-06 (Thu, 06 Aug 2026)

  Changed paths:
    M opennms-web-api/src/main/java/org/opennms/web/api/RestUtils.java
    M opennms-web-api/src/main/java/org/opennms/web/svclayer/support/DefaultRequisitionAccessService.java
    A opennms-web-api/src/test/java/org/opennms/web/api/RestUtilsTest.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/AssetRecordResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/CategoryRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/NodeRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/OnmsIpInterfaceResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/OnmsMonitoredServiceResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/OnmsSnmpInterfaceResource.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/AbstractDaoRestServiceWithDTO.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/DiscoveryRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeCategoriesRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeIpInterfacesRestService.java
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/NodeSnmpInterfacesRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/CategoryRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/NodeRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/AlarmRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/DiscoveryRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/MonitoringLocationRestServiceIT.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/NodeRestServiceIT.java
    M opennms-webapp/src/main/webapp/WEB-INF/applicationContext-spring-security.xml

  Log Message:
  -----------
  Merge remote-tracking branch 'origin/foundation-2023' into cg/jira/NMS-20030


Compare: https://github.com/OpenNMS/opennms/compare/344e9fb6b063...1f1ca9541e2e

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.