This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Update keycloak `docker-compose.yml` #63

Merged

KowalskiPiotr98 merged 3 commits into main from keycloak-docker-compose-fix

Nov 25, 2022

Contributor

KowalskiPiotr98 commented Nov 25, 2022 •

edited

Loading

Closes #35

KowalskiPiotr98 added 3 commits

November 25, 2022 22:10


          Migrate to new version of keycloak

ec51631


          Change keycloak authority paths to new format

68a85b3


          Fix logout path no longer working

0f91d17

pull-request-size bot added the size/M label

KowalskiPiotr98 mentioned this pull request

Adjust authority path to new keycloak rinkudesu/rinkudesu-links#195

Merged

sonarcloud bot commented Nov 25, 2022

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

KowalskiPiotr98 enabled auto-merge

November 25, 2022 21:36

github-advanced-security bot found potential problems

View reviewed changes

Rinkudesu.Gateways.Webui/Rinkudesu.Gateways.Webui/Controllers/UserSessionController.cs

@@ @@ -37,7 +37,7 @@ @@
                       {
                           if (!User.Identity!.IsAuthenticated) return Redirect("/");
                           await HttpContext.SignOutAsync();
-                          return Redirect($"{KeycloakSettings.Current.Authority}/protocol/openid-connect/logout?redirect_uri={HttpContext.GetEncodedBasePath()}");
+                          return Redirect($"{KeycloakSettings.Current.Authority}/protocol/openid-connect/logout?client_id={KeycloakSettings.Current.ClientId}&post_logout_redirect_uri={HttpContext.GetEncodedBasePath()}");

Check warning

Code scanning / CodeQL

URL redirection from remote source

Untrusted URL redirection due to [user-provided value](1). Untrusted URL redirection due to [user-provided value](2). Untrusted URL redirection due to [user-provided value](3).

KowalskiPiotr98 merged commit 2eda5e1 into main

KowalskiPiotr98 deleted the keycloak-docker-compose-fix branch

November 25, 2022 21:39

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels