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

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   Verified against `web/src/pages/cluster/index.tsx`:
   
   - `handleConfigPreview` (line 641) awaits `previewClusterConfig(request)` 
then unconditionally calls `setConfigPreview(preview)` (line 654) — no 
request-generation guard.
   - Other handlers in the same file (`nsConfigDiffRequestRef`, 
`brokerConfigDiffRequestRef`, `connectionTestRequestRef`) already use ref-based 
generation counters to discard stale responses.
   - `handleConfigPreview` is the only async handler missing this pattern.
   
   **Root Cause:** Missing request-generation guard in `handleConfigPreview` — 
a slow first response can overwrite a newer preview.
   **Severity:** medium — the preview panel is a safety check before applying 
config changes; showing stale diffs invites incorrect confirmations.
   
   Noted that a fix PR is incoming from the reporter.
   
   ---
   *Automated evaluation by github-manager*


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