updated libs, changed e2e and URL.canParse

- Updated @inrupt/solid-client-authn-browser and @inrupt/solid-client-authn-node to last version, with the bug fix for the parsing of clientAppId.
- Added a workaround for URL.canParse as lint was complaining about it.
- Updated index e2e test as it was before to check if it still needs the change.

e2e/index.playwright.ts

@@ -54,13 +54,14 @@ test("user can login into the application", async ({
   await setup();
   await login();
 
-  // As we now distinguish between idp and app logouts, this logout won't
-  // cause a redirection through the idp logout page anymore, as, locally,
-  // the AMC does not have a Solid-OIDc client ID, and hence the logout will
-  // be treated as an app logout.
+  const endpointRequest = page.waitForRequest((request) =>
+    request.url().startsWith(endSessionEndpoint)
+  );
   await page.getByTestId("logout-button").click();
+  // Redirect through the idp logout page
+  await endpointRequest;
 
-  await page.waitForURL(new URL("login", AMI_URL).href);
+  await page.waitForURL(new URL("login?returnTo=%2F", AMI_URL).href);
 
   // Land back at the applications login page
   await login();

package.json

@@ -30,8 +30,8 @@
     "@comunica/core": "^2.8.1",
     "@inrupt/solid-client": "^1.23.3",
     "@inrupt/solid-client-access-grants": "^3.0.4",
-    "@inrupt/solid-client-authn-browser": "^2.0.0",
-    "@inrupt/solid-client-authn-node": "^2.0.0",
+    "@inrupt/solid-client-authn-browser": "^2.2.3",
+    "@inrupt/solid-client-authn-node": "^2.2.3",
     "@inrupt/solid-client-vc": "^1.0.0",
     "@popperjs/core": "^2.11.8",
     "@rdfjs/types": "^1.1.0",

src/components/Header/Header.tsx

@@ -34,6 +34,16 @@ export default function Header({
 }) {
   const { logout, session } = useContext(SessionContext);
 
+  const isUrl = (candidateUrl: string | URL) => {
+    try {
+      // If url is not URL-shaped, this will throw
+      new URL(candidateUrl);
+      return true;
+    } catch (_e) {
+      return false;
+    }
+  };
+
   /**
    * Handles the logout process of the application, which can be of two types: app logout and idp logout.
    *
@@ -48,7 +58,7 @@ export default function Header({
   const handleLogout = async () => {
     if (
       session.info.clientAppId !== undefined &&
-      URL.canParse(session.info.clientAppId)
+      isUrl(session.info.clientAppId)
     ) {
       await logout({
         logoutType: "idp",