Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix dashboard logout #3468

Merged
merged 1 commit into from
Jun 19, 2023
Merged

Conversation

dbutenhof
Copy link
Member

@dbutenhof dbutenhof commented Jun 16, 2023

PBENCH-1185

Starting relatively recently, Keycloak has a distinct callback URI setting for "post logout", which I noticed while debugging my Postman configuration on the staging server. Adding the proper "post logout callback URI" solved the problem and allowed a clean logout. Then it only took me an hour of searching to figure out how to specify this in the REST call; to say that it's "not excessively well documented" is understating the point ...

PBENCH-1185

Starting relatively recently, Keycloak has a distinct callback URI setting for
"post logout", which I noticed while debugging my Postman configuration on
the staging server. Adding the proper "post logout callback URI" solved the
problem and allowed a clean logout. Then it only took me an hour of searching
to figure out how to specify this in the REST call; to say that it's "not
excessively well documented" is understanding the point ...
@dbutenhof dbutenhof added Server Dashboard Of and relating to the Dashboard GUI Users Of and relating to working with users. labels Jun 16, 2023
@dbutenhof dbutenhof self-assigned this Jun 16, 2023
Copy link
Member

@webbnh webbnh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it works, I'm for it.

(Did you set this manually on the Staging server?)

server/pbenchinacan/load_keycloak.sh Show resolved Hide resolved
Copy link
Member Author

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Did you set this manually on the Staging server?)

Yeah, I did that first before I even started figuring out how to set it in the API, and then tested on a runlocal deployment.

So, yes, Logout should work on the staging server dashboard now. At least, it did (multiple times) for me, and with luck will for you as well.

server/pbenchinacan/load_keycloak.sh Show resolved Hide resolved
@dbutenhof dbutenhof merged commit b091da3 into distributed-system-analysis:main Jun 19, 2023
@dbutenhof dbutenhof deleted the logout branch June 19, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dashboard Of and relating to the Dashboard GUI Server Users Of and relating to working with users.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants