Skip to content

console: show real balancerd host in the OIDC connect modal#36878

Open
leedqin wants to merge 1 commit into
MaterializeInc:mainfrom
leedqin:console-oidc-connect-fixes
Open

console: show real balancerd host in the OIDC connect modal#36878
leedqin wants to merge 1 commit into
MaterializeInc:mainfrom
leedqin:console-oidc-connect-fixes

Conversation

@leedqin
Copy link
Copy Markdown
Contributor

@leedqin leedqin commented Jun 2, 2026

Read balancerd_dns_names from app-config and use
the first entry as the SQL host in the OIDC connect modal and the MCP server hostname. Falls back to the placeholder when the field is absent. Drop the "Replace ..." instruction from the intro when a real host is available.

Motivation

Fixes CNS-87

Verification

  • Reading the host name from app config
image
  • Host name in MCP connect modal
image
  • When no host name is available
image

Read balancerd_dns_names from app-config and use
the first entry as the SQL host in the OIDC connect modal and the MCP
server hostname. Falls back to the <host> placeholder when the field is
absent. Drop the "Replace <host>..." instruction from the intro
when a real host is available.
@leedqin leedqin requested a review from a team as a code owner June 2, 2026 21:20
@leedqin leedqin requested review from Alphadelta14 and removed request for a team June 2, 2026 21:20
@leedqin leedqin added the A-CONSOLE Area: Console label Jun 2, 2026
// Self-managed: Use a placeholder since the MCP endpoint may be behind a
// load balancer or custom domain that we can't determine from the console.
const balancerdHost =
appConfig.mode === "self-managed"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

do we not propagate the same for cloud though?

@leedqin leedqin requested a review from SangJunBak June 2, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CONSOLE Area: Console

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants