Give up on reconnections if token is invalid

[lukasIO]

Reconnect attempts will now fail immediately in case the reason for reconnection failing was the user being unauthorized to join

[changeset-bot]

🦋 Changeset detected

Latest commit: 0030ff1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
livekit-client	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[davidzhao]

I think this change itself looks good. Though double checking when we'd consider it NotAllowed, it seems the current logic has a bug:

            const resp = await fetch(`http${url.substring(2)}/validate${params}`);
            if (!resp.ok) {
              const msg = await resp.text();
              reject(new ConnectionError(msg, ConnectionErrorReason.NotAllowed, resp.status));

it assumes any validate errors to be NotAllowed, including if it's a 500. I think we'd need to tighten that and compare if response code is 4xx (i.e. user error)

[lukasIO]

very good point! I had missed that.

[davidzhao]

lgtm!