Skip to content

DEVELOPMENT documentation adjustments to fix stale information about credentials management#159

Open
p-rog wants to merge 2 commits into
validatedpatterns:mainfrom
p-rog:docs-fix-development-md
Open

DEVELOPMENT documentation adjustments to fix stale information about credentials management#159
p-rog wants to merge 2 commits into
validatedpatterns:mainfrom
p-rog:docs-fix-development-md

Conversation

@p-rog

@p-rog p-rog commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Here's what changed:

  1. Removed the entire "Shortcut: Reading ESO-Synced Secrets" section — keycloak-users secret is always deleted after realm import, and central-htpasswd only has an htpasswd key (bcrypt hash), not password.
  2. Added "ACS Central Access" section — documents both login methods (Keycloak OIDC with acs-admin, basic auth with admin) and notes the shared password.
  3. Updated acs-central row in the Key Secret Paths table to link to the new ACS Central Access section

Fixes markdownlint MD004 (ul-style) — the rest of the file uses
asterisks for unordered lists.

Co-authored-by: Cursor <cursoragent@cursor.com>

@sabre1041 sabre1041 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the most part looks great. One question on ACS

Comment thread docs/DEVELOPMENT.md

Keycloak user passwords (synced to `keycloak-system`):
* **Keycloak OIDC** (default login page) — username `acs-admin`, password from Vault path `secret/hub/infra/acs/acs-central` → `admin-password`.
* **Basic auth** — select "Advanced" → "basic" on the login page — username `admin`, same password as above.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where are we selecting "Advanced" -> "Basic"? The only options on the login page are "OIDC" and "Login with username/password"

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.

2 participants