coder999o opened a new pull request, #2645:
URL: https://github.com/apache/rocketmq-dashboard/pull/2645

   ## What is the purpose of the change
   
   Complete audit coverage and presentation for RocketMQ Studio control-plane 
operations that change runtime metadata or proxy routing state.
   
   Apache Topic and Consumer Group operations already record provider-level 
audit entries, but cloud provider metadata mutations only passed through the 
service boundary. Proxy address management and config reloads also changed 
Studio runtime state without audit records. This change records those 
operations in the existing `rmq_operation_audit` flow and makes the Audit page 
display those operation/resource/result codes with readable labels and parsed 
detail values.
   
   ## Brief changelog
   
   - Add shared audit operation constants for Studio control-plane audit 
vocabulary.
   - Add service-boundary audit records for non-Apache Topic and Consumer Group 
create/update/delete/reset operations without duplicating Apache provider audit 
entries.
   - Add audit records for Proxy address add/remove and Proxy config reload 
operations.
   - Keep audit write failures non-blocking so successful control-plane 
operations are not rolled back by audit storage issues.
   - Improve the Audit page operation/resource/result presentation and 
key-value detail rendering.
   - Extend mock audit records with Proxy and cloud metadata control-plane 
examples.
   - Add backend and frontend tests for audit recording, Apache de-duplication, 
UI presentation, detail parsing, and audit-failure isolation.
   
   ## Verifying this change
   
   - `git diff --check`
   - `cd server && JAVA_HOME=/Users/yx9o/home/env/jdk-22.0.2.jdk/Contents/Home 
mvn 
-Dtest=MetadataServiceTest,ProxyAddressServiceTest,ProxyControllerTest,TopicControllerTest,ConsumerGroupControllerTest
 test`
   - `cd web && npm run test -- AuditPage.test.tsx auditPresentation.test.ts 
opsService.test.ts`
   - `cd web && npm exec eslint -- src/pages/ops/audit.tsx 
src/pages/ops/auditPresentation.ts src/pages/ops/__tests__/AuditPage.test.tsx 
src/pages/ops/__tests__/auditPresentation.test.ts src/mock/audit.ts 
--max-warnings=0`
   - `cd web && npm exec prettier -- --check src/pages/ops/audit.tsx 
src/pages/ops/auditPresentation.ts src/pages/ops/__tests__/AuditPage.test.tsx 
src/pages/ops/__tests__/auditPresentation.test.ts src/mock/audit.ts`
   - `cd web && npm run build`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to