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

keycloak: cache refresh_token #679

Merged
merged 1 commit into from
Sep 27, 2022
Merged

keycloak: cache refresh_token #679

merged 1 commit into from
Sep 27, 2022

Conversation

michalskrivanek
Copy link
Member

  • Fix RPM build warnings around cinderlib
  • Use oVirt buildcontainer image
  • Enable EL9 builds
  • Use offline maven build
  • Install maven metadata without xmvn
  • adjust bump_release.sh
  • [DO NOT MERGE] log refresh_token

@michalskrivanek
Copy link
Member Author

/ost

@michalskrivanek michalskrivanek changed the title offline rpm build [DO NOT MERGE] log refresh_token Sep 27, 2022
@michalskrivanek michalskrivanek force-pushed the offline-rpm-build branch 2 times, most recently from 886d1dc to ff6c36c Compare September 27, 2022 10:23
@michalskrivanek
Copy link
Member Author

/ost

@michalskrivanek michalskrivanek force-pushed the offline-rpm-build branch 2 times, most recently from df24005 to 1ceb869 Compare September 27, 2022 10:57
@michalskrivanek michalskrivanek changed the title [DO NOT MERGE] log refresh_token keycloak: cache refresh_token Sep 27, 2022
@michalskrivanek
Copy link
Member Author

/ost basic-suite-master el9stream

During logout the session content gets asynchronously invalidated during
logout and when we call out to keycloak we may no longer have refresh_token
available, and cause an exception on keycloak side, failing the whole
logout flow.
@michalskrivanek
Copy link
Member Author

/ost

@michalskrivanek
Copy link
Member Author

/ost basic-suite-master el9stream

@michalskrivanek michalskrivanek merged commit 6f3f56a into master Sep 27, 2022
@michalskrivanek michalskrivanek deleted the offline-rpm-build branch September 27, 2022 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants