Fix timeout

[zhiyuanliang-ms]

Why this PR?

Promise.race will not cancel other pending promises. The setTimeout will keep the node process running.

This bug caused this call: const settings = await load(connectionString); will always let the node process run for 30 seconds.

An example to illustrate the issue:

async function foo() {
  await new Promise(resolve => setTimeout(resolve, 1000));
  return "done";
}

async function main() {
    let timeoutId;
    const ms = 5000;
    const timeoutPromise = new Promise((_, reject) => {
        timeoutId = setTimeout(() => {
          reject(new Error(`Operation timed out after ${ms}ms`));
        }, ms);
      });

    try {
        let result = await Promise.race([foo(), timeoutPromise]);   
    }  catch (error) {
        throw new Error(`Failed to build fallback clients, ${error.message}`);
    } 
    ////// Solution
    // finally {
    //     clearTimeout(timeoutId);
    // }
    //////
}

main();

The node process will run for 5 secs instead of ending after foo is done.

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.