Skip to content

feat(config): make local dashboard authentication optional - #1707

Merged
Teingi merged 1 commit into
oceanbase:masterfrom
PsiACE:codex/local-dashboard-optional-auth
Sep 30, 2026
Merged

Teingi merged 1 commit into
oceanbase:masterfrom
PsiACE:codex/local-dashboard-optional-auth

Conversation

@PsiACE

@PsiACE PsiACE commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Which issue or RFC does this PR close?

Related to #1695.

Rationale for this change

Local users should be able to open Dashboard without managing a Server token.

What changes are included in this PR?

Decouple Dashboard enablement from token generation in the local configuration wizard, allow Dashboard with ACCESS_MODE=disabled, and update documentation and behavior tests.

Are there any user-facing changes?

Enabling Dashboard in a new local setup defaults to no authentication, with an explicit option to enable it for Dashboard, HTTP API, and MCP. Existing authenticated and remote setups retain authentication.

How was this change tested?

  • Focused wizard, Dashboard, transport, and acceptance tests.
  • make check and make docs-test.
  • Isolated wheel installation: configuration wizard, real HTTP/MCP requests, and Chromium navigation, search, and login in both authentication modes.

AI usage statement

Codex (gpt-6-astra, medium).

@PsiACE
PsiACE force-pushed the codex/local-dashboard-optional-auth branch from 1e3b78b to 86394bf Compare September 24, 2026 07:36
@PsiACE
PsiACE marked this pull request as ready for review September 24, 2026 08:08
@PsiACE
PsiACE force-pushed the codex/local-dashboard-optional-auth branch from 86394bf to 45f5c65 Compare September 24, 2026 08:11
@PsiACE
PsiACE requested a review from Teingi September 30, 2026 07:41

@Teingi Teingi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Teingi
Teingi merged commit f2c8530 into oceanbase:master Sep 30, 2026
24 checks passed
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