Conversation
Exact match preferred over one-label wildcard. Used by the certificate mismatch warning; extracted so the warning stands alone. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Non-blocking warning in the Proxy, Redirection and 404 host modals, shown under both the domain names field and the SSL certificate select (the two live on different tabs). Reuses the auto-pick coverage logic via a new uncoveredDomains() export; silent for None, "new" and while certificates load. Fixes NginxProxyManager#3. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
As the AI mentioned i just wanted these features myself, i do not know coding at all, but it works as i wanted on my own system. |
|
Docker Image for build 2 is available on DockerHub: Note Ensure you backup your NPM instance before testing this image! Especially if there are database changes. Warning Changes and additions to DNS Providers require verification by at least 2 members of the community! |
|
Thanks for taking the time on this.
|
Why
It's currently easy to save a proxy host with an SSL certificate that doesn't actually cover the typed domain names — nothing in the UI checks that the selected certificate matches. The mistake only surfaces later as browser certificate errors on the live host, which is confusing to debug because the UI showed no problem at save time.
This adds a non-blocking warning to the Proxy Host, Redirection Host, and 404 Host modals when one or more typed domains are not covered by the selected certificate (exact or one-label wildcard match). The warning appears under both the domain names field and the SSL certificate selector, since the two live on different tabs and the user may be looking at either one.
Deliberately non-blocking: there are legitimate reasons to save a mismatched pair (cert renewal in flight, DNS not cut over yet), so this informs rather than prevents. The warning stays silent when no certificate is selected ("None"), when requesting a new certificate inline, and while the certificate list is still loading — no false positives during normal flow.
Frontend-only; no API or backend changes. The domain/cert matching logic lives in a new
CertificateMatchmodule with unit tests. Note: #5716 adds the same module for certificate auto-selection — whichever lands second rebases trivially.Full disclosure: this was AI-written (boxes checked below) — I made it because I wanted the feature myself. It's been running on my own NPM instance and works for my day-to-day use, but that's the extent of the real-world testing, so review accordingly.
Type of Change
AI Usage