Skip to content

fix: General Service URI text in Database Summary page#13519

Open
hana-akamai wants to merge 4 commits intolinode:developfrom
hana-akamai:add-username-to-general-service-uri
Open

fix: General Service URI text in Database Summary page#13519
hana-akamai wants to merge 4 commits intolinode:developfrom
hana-akamai:add-username-to-general-service-uri

Conversation

@hana-akamai
Copy link
Contributor

@hana-akamai hana-akamai commented Mar 23, 2026

Description 📝

  • Add <username>: to the general Service URI in the Database Summary page
  • Display ssl-mode=REQUIRED for mysql clusters
  • Display placeholder text while hosts are still being retrieved

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

3/31

Preview 📷

Before After
image image

How to test 🧪

Prerequisites

(How to setup test environment)

  • Ensure you have the Database PgBouncer and HostnameEndpoints feature flag on
  • Have a Postgres and Mysql Database

Reproduction steps

(How to reproduce the issue, if applicable)

  • On remote devcloud, go to a Postgres' Database Cluster's summary page
  • Click reveal password for the service URI under Connection Details
  • The password does not include <username>:
  • Go to a Mysql cluster's summary page
  • The service URI ssl mode is sslmode=require instead of ssl-mode=REQUIRED
  • Add a VPC with public access
  • The Private Service URI hosts field displays undefined while hosts are being retrieved

Verification steps

(How to verify changes)

  • Checkout this PR and point to devcloud locally
  • Go to a Database Cluster's summary page and click reveal password for the service URI under Connection Details
  • The password includes <username>:
  • There should be no regressions with PgBouncer
  • Go to a mysql cluster's summary page
  • The service URI ssl mode is ssl-mode=REQUIRED
  • Add a VPC with public access
  • The Private Service URI hosts field displays a placeholder text while hosts are being retrieved
pnpm test ServiceURI
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@hana-akamai hana-akamai self-assigned this Mar 23, 2026
@hana-akamai hana-akamai added Bug Fixes for regressions or bugs DBaaS Relates to Database as a Service labels Mar 23, 2026
@hana-akamai hana-akamai marked this pull request as ready for review March 23, 2026 17:41
@hana-akamai hana-akamai changed the title fix: Add username to the general Service URI in Database Summary page fix: General Service URI text in Database Summary page Mar 23, 2026
Copy link
Contributor

@smans-akamai smans-akamai left a comment

Choose a reason for hiding this comment

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

Went through the verification steps and tested the changes locally. The updates look good!

@hana-akamai hana-akamai added Add'tl Approval Needed Waiting on another approval! 🚨 Urgent labels Mar 23, 2026
@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Mar 23, 2026
@bnussman-akamai bnussman-akamai removed the Add'tl Approval Needed Waiting on another approval! label Mar 23, 2026
@smans-akamai
Copy link
Contributor

Using the steps to test adding a VPC with public access, I was able reproduce the bug and tested the verification steps for the fix. I can see the placeholder appearing, so this fix looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fixes for regressions or bugs DBaaS Relates to Database as a Service 🚨 Urgent

Projects

Status: Approved

Development

Successfully merging this pull request may close these issues.

3 participants