Iterate

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
matrix-org · May 14, 2024 · 837ddf2 · 837ddf2
1 parent fc4866f
commit 837ddf2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/SlidingSyncManager.ts b/src/SlidingSyncManager.ts
@@ -401,7 +401,7 @@ export class SlidingSyncManager {
 
         const proxyUrl = await this.getProxyFromWellKnown(client);
         if (proxyUrl != undefined) {
-            const response = await fetch(proxyUrl + "/client/server.json", {
+            const response = await fetch(new URL("/client/server.json", proxyUrl), {
                 method: Method.Get,
                 signal: timeoutSignal(10 * 1000), // 10s
             });

diff --git a/test/SlidingSyncManager-test.ts b/test/SlidingSyncManager-test.ts
@@ -40,6 +40,8 @@ describe("SlidingSyncManager", () => {
         mocked(client.getRoom).mockReturnValue(null);
         manager.configure(client, "invalid");
         manager.slidingSync = slidingSync;
+        fetchMockJest.reset();
+        fetchMockJest.get("https://proxy/client/server.json", {});
     });
 
     describe("setRoomVisible", () => {
@@ -237,7 +239,7 @@ describe("SlidingSyncManager", () => {
     describe("checkSupport", () => {
         beforeEach(() => {
             SlidingSyncController.serverSupportsSlidingSync = false;
-            jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("proxy");
+            jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("https://proxy/");
         });
         it("shorts out if the server has 'native' sliding sync support", async () => {
             jest.spyOn(manager, "nativeSlidingSyncSupport").mockResolvedValue(true);
@@ -260,7 +262,7 @@ describe("SlidingSyncManager", () => {
                     server: "matrix.org",
                 },
                 "org.matrix.msc3575.proxy": {
-                    url: "proxy",
+                    url: "https://proxy/",
                 },
             });
             fetchMockJest.get("https://matrix-client.matrix.org/_matrix/client/versions", { versions: ["v1.4"] });
@@ -286,7 +288,7 @@ describe("SlidingSyncManager", () => {
                     expect(feature).toBe("org.matrix.msc3575");
                     return true;
                 });
-            const proxySpy = jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("proxy");
+            const proxySpy = jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("https://proxy/");
 
             expect(SlidingSyncController.serverSupportsSlidingSync).toBeFalsy();
             await manager.checkSupport(client); // first thing it does is call nativeSlidingSyncSupport
@@ -307,14 +309,14 @@ describe("SlidingSyncManager", () => {
             expect(manager.startSpidering).toHaveBeenCalled();
         });
         it("uses the proxy declared in the client well-known", async () => {
-            jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("proxy");
+            jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("https://proxy/");
             await manager.setup(client);
             expect(manager.configure).toHaveBeenCalled();
-            expect(manager.configure).toHaveBeenCalledWith(client, "proxy");
+            expect(manager.configure).toHaveBeenCalledWith(client, "https://proxy/");
             expect(manager.startSpidering).toHaveBeenCalled();
         });
         it("uses the legacy `feature_sliding_sync_proxy_url` if it was set", async () => {
-            jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("proxy");
+            jest.spyOn(manager, "getProxyFromWellKnown").mockResolvedValue("https://proxy/");
             jest.spyOn(SettingsStore, "getValue").mockImplementation((name: string) => {
                 if (name === "feature_sliding_sync_proxy_url") return "legacy-proxy";
             });