Skip to content

Harden team and read-only region output - #1389

Merged
no-itsbackpack merged 1 commit into
mainfrom
egomez/fix-team-json-region-guard
Aug 26, 2026
Merged

Harden team and read-only region output#1389
no-itsbackpack merged 1 commit into
mainfrom
egomez/fix-team-json-region-guard

Conversation

@no-itsbackpack

Copy link
Copy Markdown
Contributor

Summary

  • serialize team memberships through a safe JSON projection that omits password metadata
  • check the database engine before listing Vitess read-only regions
  • return a clear error for PostgreSQL without calling the read-only-region endpoint

Test plan

  • go test ./internal/cmd/org ./internal/cmd/database -count=1

Omit password metadata from team-member JSON and reject the Vitess-only read-only-region command before calling it for PostgreSQL databases.

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack requested a review from a team as a code owner August 26, 2026 01:08

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk: low. Approved based on a successful Cursor Bugbot check with no findings requiring human review (Cursor Security Agent was not running). No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: cli-reviewer

@no-itsbackpack
no-itsbackpack merged commit 9014db1 into main Aug 26, 2026
5 checks passed
@no-itsbackpack
no-itsbackpack deleted the egomez/fix-team-json-region-guard branch August 26, 2026 01:48
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