Skip to content

[Studio] Add proxy address removal flow - #577

Merged
lizhimins merged 1 commit into
apache:rocketmq-studiofrom
Aias00:fix/studio-proxy-remove-api
Jul 28, 2026
Merged

[Studio] Add proxy address removal flow#577
lizhimins merged 1 commit into
apache:rocketmq-studiofrom
Aias00:fix/studio-proxy-remove-api

Conversation

@Aias00

@Aias00 Aias00 commented Jul 25, 2026

Copy link
Copy Markdown

Summary

  • add a form-encoded /api/proxy/removeProxyAddr.do compatibility endpoint and service logic
  • wire the Studio Proxy page remove action to the backend and refresh the node list after success
  • cover backend removal semantics and frontend API request encoding

Validation

  • JAVA_HOME=/Users/aias/Library/Java/JavaVirtualMachines/openjdk-21.0.2/Contents/Home mvn -q -Dtest=ProxyCompatControllerTest,ProxyAddressServiceTest test
  • JAVA_HOME=/Users/aias/Library/Java/JavaVirtualMachines/openjdk-21.0.2/Contents/Home mvn -q test (50 surefire files, 298 tests, 0 failures, 0 errors)
  • npm test -- --run src/api/proxy.test.ts
  • npm test (36 files, 164 tests)
  • npm run lint (0 errors, 4 existing fast-refresh warnings)
  • npm run build
  • git diff --check

Copilot AI review requested due to automatic review settings July 25, 2026 05:33

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@lizhimins
lizhimins force-pushed the fix/studio-proxy-remove-api branch from f07f081 to ad72afb Compare July 28, 2026 12:33
@lizhimins
lizhimins merged commit ab85dd9 into apache:rocketmq-studio Jul 28, 2026
3 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.

3 participants