RockteMQ-AI commented on issue #1402: URL: https://github.com/apache/rocketmq-clients/issues/1402#issuecomment-6231522921
**Issue Evaluation** Category: `type/bug` | Status: **Confirmed** The reported race condition in BaseClient startup has been verified. **Root Cause:** The `#startup()` method has an async callback ordering issue: 1. `await this.updateRoutes()` internally triggers a settings pull request 2. The `onSettingsCommand` callback may arrive before `#startupResolve` is assigned 3. The Promise waiting for the first settings callback never resolves 4. This causes a 5-minute timeout during client startup **Impact:** Node.js client startup may hang indefinitely under certain timing conditions **Severity:** High — affects client availability and startup reliability **Recommended Fix:** Ensure the Promise is created and `#startupResolve` is assigned before calling `updateRoutes()`, or use a different synchronization mechanism that doesn't depend on callback ordering. An automated fix proposal will be generated. Reply `/approve` to proceed with PR generation. --- *Automated evaluation by RockteMQ-AI* -- 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]
