Unstable behaviour due to missing await

[robblovell]

there is a missing await in index.js in the connect:

This:

async function connect(opts) {
  opts = defaults(opts);
  validate(opts);
  if (opts.authtoken) {
    await setAuthtoken(opts);
  }

  processUrl = await getProcess(opts);
  ngrokClient = new NgrokClient(processUrl);
  return connectRetry(opts);
}

async function connectRetry(opts, retryCount = 0) {
  opts.name = String(opts.name || uuid.v4());
  try {
    const response = await ngrokClient.startTunnel(opts);
    return response.public_url;
  } catch (err) {
    if (!isRetriable(err) || retryCount >= 100) {
      throw err;
    }
    await new Promise((resolve) => setTimeout(resolve, 200));
    return connectRetry(opts, ++retryCount);
  }
}

Should be:

async function connect(opts) {
...
  ngrokClient = new NgrokClient(processUrl);
  return await connectRetry(opts);
}
async function connectRetry(opts, retryCount = 0) {
...
    return await connectRetry(opts, ++retryCount);
  }
}

What I have found is that sometimes it works, but if you are connecting and disconnecting a lot, it becomes undependable as I think there is a race condition without the await.