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

1337 keycloak request retries #1386

Merged
merged 4 commits into from Nov 7, 2019
Merged

1337 keycloak request retries #1386

merged 4 commits into from Nov 7, 2019

Conversation

@rocketeerbkw
Copy link
Member

rocketeerbkw commented Nov 7, 2019

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated.
  • Changelog entry has been written

More changes to help #1337

Changelog Entry

Bugfix - Shared keycloak client and parallel calls caused API timeouts/errors (#1337)
Bugfix - loadAllUsers api query only returns 100 users

Closing issues

Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

rocketeerbkw added 4 commits Nov 4, 2019
There is a bug with the keycloak-admin client related to expiring access
tokens that somehow can cause failures when parallel keycloak requests are in
flight and a token expires.
@Schnitzel Schnitzel marked this pull request as ready for review Nov 7, 2019
@Schnitzel Schnitzel merged commit 7c67acf into master Nov 7, 2019
1 check passed
1 check passed
continuous-integration/jenkins/pr-merge This commit looks good
Details
@tobybellwood tobybellwood added this to the v1.2.0 milestone Nov 26, 2019
@rocketeerbkw rocketeerbkw deleted the 1337-keycloak-request-retries branch Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.