[OpenNMS/opennms] 863763: NMS-20154: use Onms-XXX wrappers and a REST downlo...
joseanesONMS via opennms-cvs <[email protected]>
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/jira/NMS-20154-system-report/[email protected]> |
Branch: refs/heads/jira/NMS-20154-system-report
Home: https://github.com/OpenNMS/opennms
Commit: 863763bc44c20f1adde7240970433f02394b1d36
https://github.com/OpenNMS/opennms/commit/863763bc44c20f1adde7240970433f02394b1d36
Author: Jose Anes <[email protected]>
Date: 2026-08-04 (Tue, 04 Aug 2026)
Changed paths:
M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v2/SystemReportRestService.java
M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v2/SystemReportRestServiceTest.java
M opennms-webapp/src/main/webapp/WEB-INF/applicationContext-spring-security.xml
M ui/src/containers/SystemReport.vue
M ui/src/services/index.ts
M ui/src/services/systemReportService.ts
M ui/src/types/systemReport.ts
M ui/tests/systemReport.test.ts
Log Message:
-----------
NMS-20154: use Onms-XXX wrappers and a REST download for system report
Address review on the Generate System Report page:
- Swap direct PrimeVue (Card, Checkbox, Select, InputText) for the
@opennms/onms-ui wrappers (OnmsCard/OnmsCheckbox/OnmsSelect/
OnmsInputText); the binary OnmsCheckbox drives per-plugin selection
through an explicit add/remove toggle.
- Replace the hidden-iframe form POST to the legacy systemReport.htm
with a POST /api/v2/system-report/generate that streams the report as
an attachment, consumed via composables/useDownload. The endpoint
ports FormatterView (resolve formatter, run selected plugins, sanitize
filename) and rejects unknown/non-streaming formatters and empty
plugin sets with 400s. Generation is admin-only in spring-security.
- Cover the endpoint (stream order, attachment header, filename
sanitising, the 400 paths) and rewrite the page test for the new flow.
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