Skip to content

[Studio] fix: validate instance updates - #615

Merged
lizhimins merged 1 commit into
apache:rocketmq-studiofrom
yx9o:dev-613
Jul 29, 2026
Merged

[Studio] fix: validate instance updates#615
lizhimins merged 1 commit into
apache:rocketmq-studiofrom
yx9o:dev-613

Conversation

@yx9o

@yx9o yx9o commented Jul 26, 2026

Copy link
Copy Markdown

What is the purpose of the change

Fixes #613 .

Prevent /api/instances/update from accepting blank name or endpoint values and persisting invalid instance data.

Brief changelog

  • Reject non-null blank instance names and endpoints with HTTP 400.
  • Validate all supplied required fields before modifying the existing instance.
  • Preserve partial update semantics for omitted fields.
  • Add regression tests for validation failures and mutation prevention.

@lizhimins
lizhimins force-pushed the rocketmq-studio branch 5 times, most recently from b1f151b to 434c2ea Compare July 28, 2026 11:40
@lizhimins
lizhimins merged commit faa48b7 into apache:rocketmq-studio Jul 29, 2026
3 checks passed
@yx9o
yx9o deleted the dev-613 branch July 29, 2026 02:14
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