Implement client.getDomain() null check

matrix-org · May 20, 2024 · 43373fd · 43373fd
1 parent 9c383c4
commit 43373fd
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/SlidingSyncManager.ts b/src/SlidingSyncManager.ts
@@ -359,10 +359,14 @@ export class SlidingSyncManager {
         let proxyUrl: string | undefined;
 
         try {
-            const clientWellKnown = await AutoDiscovery.findClientConfig(client.getDomain());
+            const clientDomain = await client.getDomain();
+            if (clientDomain === null) {
+                throw new RangeError("Homeserver domain is null");
+            }
+            const clientWellKnown = await AutoDiscovery.findClientConfig(clientDomain);
             proxyUrl = clientWellKnown?.["org.matrix.msc3575.proxy"]?.url;
         } catch (e) {
-            // client.getDomain() is invalid, `AutoDiscovery.findClientConfig` has thrown
+            // client.getDomain() is null, we've thrown
         }
 
         if (proxyUrl != undefined) {