unbridled-41 commented on PR #2835: URL: https://github.com/apache/rocketmq-dashboard/pull/2835#issuecomment-5535571913
> Merged, thanks! One follow-up spotted during review: `RocketMQDLQProvider` still has an unguarded `viewMessage` call on the DLQ lookup path (~line 261), similar to what #2833 fixed for the message query path. Would you be interested in a follow-up PR adding the same broker-topology guard there? Yes, thanks for spotting this. I'll send a follow-up PR applying the same broker-topology guard from #2833 to the DLQ msgId lookup path. -- 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]
