fix: Add workaround for authn library issue

CommunitySolidServer · Oct 6, 2023 · 180d5f1 · 180d5f1
1 parent cccca96
commit 180d5f1
Show file tree

Hide file tree

Showing 3 changed files with 48 additions and 62 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -161,8 +161,8 @@
   "devDependencies": {
     "@commitlint/cli": "^17.6.1",
     "@commitlint/config-conventional": "^17.6.1",
-    "@inrupt/solid-client-authn-core": "^1.16.0",
-    "@inrupt/solid-client-authn-node": "^1.16.0",
+    "@inrupt/solid-client-authn-core": "^1.17.2",
+    "@inrupt/solid-client-authn-node": "^1.17.2",
     "@microsoft/tsdoc-config": "^0.16.2",
     "@tsconfig/node18": "^2.0.1",
     "@types/cheerio": "^0.22.31",

diff --git a/test/integration/IdentityTestState.ts b/test/integration/IdentityTestState.ts
@@ -132,7 +132,10 @@ export class IdentityTestState {
     const mockUrl = res.headers.get('location')!;
     expect(mockUrl.startsWith(this.redirectUrl)).toBeTruthy();
 
-    const info = await this.session.handleIncomingRedirect(mockUrl);
+    // Workaround for https://github.com/inrupt/solid-client-authn-js/issues/2985
+    const strippedUrl = new URL(mockUrl);
+    strippedUrl.searchParams.delete('iss');
+    const info = await this.session.handleIncomingRedirect(strippedUrl.href);
     expect(info?.isLoggedIn).toBe(true);
   }