Skip to content

feat(console): NGINX connection fields in ⚙ Setup - #58

Merged
henleda merged 1 commit into
mainfrom
feat-nginx-setup-fields
Aug 19, 2026
Merged

feat(console): NGINX connection fields in ⚙ Setup#58
henleda merged 1 commit into
mainfrom
feat-nginx-setup-fields

Conversation

@henleda

@henleda henleda commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Follow-up to #57 (merged). The ④ Mitigate "Apply on your own NGINX" panel and its help text tell the operator to set NGINX_SSH_* "in ⚙ Setup" — but Setup had no NGINX fields, so the only way to point the copilot at the box was hand-editing .env. (The BIG-IP has its Setup fields; NGINX was missed when the panel was added.)

  • Added the NGINX_SSH_* keys to MANAGED_KEYS so the generic Setup config editor renders them as editable fields; NGINX_SSH_PASSWORD is in SECRET_KEYS (redacted), the key path is not secret (echoed so the page shows what's set).
  • Added a "NGINX + App Protect box" card to Setup → Advanced integrations with a live status readout (loadNginx/api/nginx-lab), mirroring the BIG-IP card.

+2 tests (config exposes the keys / password redacted; the Setup card + loader are wired). Verified live: Setup manages all 9 NGINX fields and the card reads "nginx-plus-r37.0.4 · App Protect: loaded".

🤖 Generated with Claude Code

The ④ Mitigate "Apply on your own NGINX" panel + its help told the operator to set
NGINX_SSH_* "in ⚙ Setup", but Setup had no NGINX fields — the only way to point it
at the box was hand-editing .env. Added the NGINX_SSH_* keys to MANAGED_KEYS (so the
generic Setup config editor renders them as fields) with NGINX_SSH_PASSWORD in
SECRET_KEYS (the key PATH is not secret), plus a NGINX + App Protect card in Setup's
Advanced integrations with a live status readout (loadNginx → /api/nginx-lab),
mirroring the BIG-IP card. +2 tests.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@henleda
henleda merged commit b5b3c69 into main Aug 19, 2026
4 checks passed
@henleda
henleda deleted the feat-nginx-setup-fields branch August 19, 2026 02:07
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