diff --git a/src/state/Connection.ts b/src/state/Connection.ts index bb18daaa..2f294057 100644 --- a/src/state/Connection.ts +++ b/src/state/Connection.ts @@ -120,7 +120,6 @@ export class Connection { state: "FetchingConfig", focus: this.localTransport, }); - // TODO could this be loaded earlier to save time? const { url, jwt } = await this.getSFUConfigWithOpenID(); // If we were stopped while fetching the config, don't proceed to connect if (this.stopped) return; diff --git a/src/state/PublishConnection.ts b/src/state/PublishConnection.ts index 52db6968..1bb79211 100644 --- a/src/state/PublishConnection.ts +++ b/src/state/PublishConnection.ts @@ -105,6 +105,10 @@ export class PublishConnection extends Connection { // Observe mute state changes and update LiveKit microphone/camera states accordingly this.observeMuteStates(this.scope); + // TODO: This will fetch the JWT token. Perhaps we could keep it preloaded + // instead? This optimization would only be safe for a publish connection, + // because we don't want to leak the user's intent to perhaps join a call to + // remote servers before they actually commit to it. await super.start(); if (this.stopped) return;