10000 Sequential transactions instead of nested transactions by ahus1 · Pull Request #40224 · keycloak/keycloak · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Sequential transactions instead of nested transactions #40224

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

Conversation

ahus1
Copy link
Contributor
@ahus1 ahus1 commented Jun 4, 2025

Closes #40171

@sschu
Copy link
Contributor
sschu commented Jun 4, 2025

@ahus1 I wonder if this would be worth a backport to KC26? Could be really annoying for people going to that version.

Closes keycloak#40171

Signed-off-by: Alexander Schwartz <aschwart@redhat.com>
@ahus1 ahus1 force-pushed the is-40171-sequential-instead-of-nested-transactions branch from 341145e to 4aa70f4 Compare June 4, 2025 09:21
@ahus1
Copy link
Contributor Author
ahus1 commented Jun 4, 2025

@sschu - I assume you mean backporting to 26.2.x? That would be a yes. Older versions are not affected.

@sschu
Copy link
Contributor
sschu commented Jun 4, 2025

8000 @ahus1 Ah, ok. I thought they might be affected since Argon2 was introduced earlier. Then 26.2 would be great.

Copy link
Contributor
@rmartinc rmartinc left a comment

Choose a reason for hiding this comment

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

Thanks @ahus1 for your work on this! LGTM!

@ahus1 ahus1 marked this pull request as ready for review June 4, 2025 12:21
@ahus1 ahus1 requested review from a team as code owners June 4, 2025 12:21
@ahus1
Copy link
Contributor Author
ahus1 commented Jun 4, 2025

@mposolda - can you please approve and merge this one based on @rmartinc's review?

@ahus1 ahus1 enabled auto-merge (squash) June 4, 2025 12:23
@ahus1 ahus1 merged commit 3578499 into keycloak:main Jun 5, 2025
76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SQL error when logging in for first time (per user) after Keycloak upgrade
4 participants
0