DefGuard/defguard

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

Open

#2,422 opened on Mar 21, 2026

View on GitHub
 (3 comments) (0 reactions) (1 assignee)Rust (107 forks)auto 404
designhelp wantedst3t3

Repository metrics

Stars
 (2,779 stars)
PR merge metrics
 (PR metrics pending)

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

Research direction
Examine the current IP input field in the Location settings UI. Find how other tables are implemented in the project (e.g., in the admin panel). Then implement a managed table with add, remove, sort, and IP validation.
Tech stack
None
Domain
frontend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Git
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65