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

   ### Summary
   
   Refs #5373 and the closed, unmerged #4219. Correct the alerting design’s 
claim that notification configuration is encrypted at rest and clarify that 
Base64 storage is encoding. Describe existing API write-only secret fields, 
administrator webhook access, non-admin redaction, and the separate 
database/backup protection responsibility.
   
   This is a documentation-only correction. It does not implement encryption or 
close the key-management design issue.
   
   ### Verification
   
   - Read against MybatisPlusSettingsRepository.saveGeneralSettings, 
GeneralSettingsVO JSON annotations, 
SettingsService.getGeneralSettings/redactNotificationWebhooks, and 
CredentialUtils
   - git diff --check passes
   - Independent read-only review completed
   - No code, configuration or tests changed; no runtime suite rerun for this 
prose-only patch
   
   AI assistance was used for source cross-checking and review.


-- 
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