fix # for logout urls

cultuurnet · Jun 28, 2024 · 68517b2 · 68517b2
1 parent bb2fd05
commit 68517b2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/Keycloak/Converters/IntegrationUrlConverter.php b/app/Keycloak/Converters/IntegrationUrlConverter.php
@@ -54,6 +54,6 @@ public static function buildLogoutUrls(Integration $integration, Environment $en
         }
 
         $logoutUrls = $integration->urlsForTypeAndEnvironment(IntegrationUrlType::Logout, $environment);
-        return implode('#', array_map(static fn ($url) => $url->url, $logoutUrls));
+        return implode('##', array_map(static fn ($url) => $url->url, $logoutUrls));
     }
 }
diff --git a/tests/Keycloak/Converters/IntegrationUrlConverterTest.php b/tests/Keycloak/Converters/IntegrationUrlConverterTest.php
@@ -99,7 +99,7 @@ public function test_convert_for_first_party_logout_urls(): void
             new IntegrationUrl(Uuid::uuid4(), $integration->id, Environment::Production, IntegrationUrlType::Logout, 'https://wrong.com/'),
         );
         $this->assertSame(
-            'https://example.com/logout1#https://example.com/logout2',
+            'https://example.com/logout1##https://example.com/logout2',
             IntegrationUrlConverter::buildLogoutUrls($integration, $this->client->environment)
         );
     }