bug(ui): settings/integration source cleanup #724

New issue

Closed

opened 2026-03-28 04:27:51 +00:00 by mfreeman451 · 1 comment

mfreeman451 commented

2026-03-28 04:27:51 +00:00

Owner

Imported from GitHub.

Original GitHub issue: #2259
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/issues/2259
Original created: 2026-01-12T01:23:28Z

Describe the bug

The Create Integration Source UI modal/form should only show Armis specific fields when the source type is set to Armis, same with Netbox, syslog, etc.
armis integration should have form fields for API credentials not freetext insert JSON
netbox integration source should allow users to configure credentials for netbox API as well
network blacklist should only show up for integrations that support blacklisting
we might want to create an ash resource just for integrations with a well defined interface, that would be a better place to define if an integration supports blacklisting and then be able to leverage that in the UI easier
"nmap" is not an integration source, we could replace this with our own network sweep/scan functionalit in the agent and we could let users scan through cidr blocks/network ranges to find hosts and add hosts that way, would rather put this into a "Discovery" feature later though

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Imported from GitHub. Original GitHub issue: #2259 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/issues/2259 Original created: 2026-01-12T01:23:28Z --- **Describe the bug** - [ ] The `Create Integration Source` UI modal/form should only show Armis specific fields when the source type is set to Armis, same with Netbox, syslog, etc. - [ ] armis integration should have form fields for API credentials not freetext insert JSON - [ ] netbox integration source should allow users to configure credentials for netbox API as well - [ ] network blacklist should only show up for integrations that support blacklisting - [ ] we might want to create an ash resource just for integrations with a well defined interface, that would be a better place to define if an integration supports blacklisting and then be able to leverage that in the UI easier - [ ] "nmap" is not an integration source, we could replace this with our own network sweep/scan functionalit in the agent and we could let users scan through cidr blocks/network ranges to find hosts and add hosts that way, would rather put this into a "Discovery" feature later though - [ ] **To Reproduce** Steps to reproduce the behavior: 1. Go to '...' 2. Click on '....' 3. Scroll down to '....' 4. See error **Expected behavior** A clear and concise description of what you expected to happen. **Screenshots** If applicable, add screenshots to help explain your problem. **Desktop (please complete the following information):** - OS: [e.g. iOS] - Browser [e.g. chrome, safari] - Version [e.g. 22] **Smartphone (please complete the following information):** - Device: [e.g. iPhone6] - OS: [e.g. iOS8.1] - Browser [e.g. stock browser, safari] - Version [e.g. 22] **Additional context** Add any other context about the problem here.

mfreeman451 commented

2026-03-28 04:27:51 +00:00

Author

Owner

Imported GitHub comment.

Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/issues/2259#issuecomment-3752882442
Original created: 2026-01-15T04:43:13Z

Fixed in UI cleanup work. Added dynamic credential fields for integration sources - Armis (api_key, api_secret), SNMP (version, community), with JSON fallback for other types. See #2310.

Imported GitHub comment. Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/issues/2259#issuecomment-3752882442 Original created: 2026-01-15T04:43:13Z --- Fixed in UI cleanup work. Added dynamic credential fields for integration sources - Armis (api_key, api_secret), SNMP (version, community), with JSON fallback for other types. See #2310.