DefGuard / DefGuard/defguard

Change UI's Location "Allowed ips" from a single line text entry field to a managed table

Open
#2,422 6 comments 0 reactions 1 assignee Claimed by @Va1ter View on GitHub
design feature help wanted st3 t3
Dominant language
Rust
Stars
2.8k
Forks
115
Avg merge
1d 4h
Merged PRs (30d)
51

Description

### Problem description

We have many IPs that we need to configure for split tunnel - this is quite cumbersome with the current text field in a Location's "Allowed ips" that's present in the 1.6.*. release. It's hard to see what is present (and whether it's valid) and edit what's already there.

### Proposed solution

An improvement might be a managed table, with a single (validated) IP input box as the means to add new entries. The table itself could provide a way to remove entries, and if the IPs could be sorted that would be useful too. I'm thinking along the lines of how some networking apps manage the entry of MAC addresses for wifi endpoint allow lists, that kind of thing.

### Alternatives considered

_No response_

### Impact

Nice to have

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.