RockteMQ-AI commented on issue #2553:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/2553#issuecomment-5404947671

   ## Issue Evaluation
   
   **Type:** Enhancement
   **Priority:** Medium
   **Status:** Acknowledged — depends on #2533
   
   This is a well-structured feature request to add reusable notification 
channels and policies for the native alerting system. The scope covers:
   
   - Separate persisted notification channels from alert rules
   - Support DingTalk, email, SMS/Webhook-compatible channel types
   - Notification policy CRUD APIs with channel selection
   - Alert rules reference a notification policy (backwards-compatible with 
existing channel-list)
   - SSRF and redirect protections for Webhook delivery
   
   **Dependency:** This builds on #2533 (native cluster alert collection), 
which is currently open. Implementation should wait until #2533 is merged.
   
   **Assessment:** Good separation of concerns — decoupling channels from rules 
makes the system more maintainable and allows channel reuse across multiple 
rules. The SSRF protection requirement is important for webhook security.
   
   > This issue will be revisited once #2533 progresses.


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