Skip to content

Add base-domain filter chips to Proxy Hosts page#5625

Open
Anilkhanna wants to merge 1 commit into
NginxProxyManager:developfrom
Anilkhanna:develop
Open

Add base-domain filter chips to Proxy Hosts page#5625
Anilkhanna wants to merge 1 commit into
NginxProxyManager:developfrom
Anilkhanna:develop

Conversation

@Anilkhanna
Copy link
Copy Markdown

Adds a single-select chip row above the Proxy Hosts table that filters hosts by their registrable (base) domain, with a per-domain host count and an "All" chip to clear. Base-domain extraction uses the public suffix list via tldts so multi-part suffixes (.co.uk, .com.au) resolve correctly. The chip filter composes with the existing text search.

Why

Type of Change

  • [ X] Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • API changes
  • Performance improvement
  • Test addition or update

AI Usage

  • AI was used to write this
  • AI was used to review this

Adds a single-select chip row above the Proxy Hosts table that filters
hosts by their base domain, with a per-domain host count and an "All"
chip to clear. Base-domain extraction uses a dependency-free
last-two-labels rule. The chip filter composes with the existing text
search.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nginxproxymanagerci
Copy link
Copy Markdown

Docker Image for build 3 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-5625

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant