Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Use RawSqlStatement instead of RawParameterizedSqlStatement for Saml encryption update#45347

Merged
mposolda merged 1 commit intokeycloak:mainfrom
rmartinc:issue-45107
Jan 12, 2026
Merged

Use RawSqlStatement instead of RawParameterizedSqlStatement for Saml encryption update#45347
mposolda merged 1 commit intokeycloak:mainfrom
rmartinc:issue-45107

Conversation

@rmartinc
Copy link
Contributor

Closes #45107

It seems that RawParameterizedSqlStatement is not logged OK when exporting to a file and should be substituted by RawSqlStatement. It wa smy falt because I used the parametrized version because the other one was deprecated. It seems that there is an issue and that one is not exported OK when using the file (I suppose because the parameters are not created).

I tested that all migrations work OK and tested the export to file with Oracle.

@rmartinc rmartinc requested a review from a team as a code owner January 12, 2026 12:05
Copy link
Contributor

@graziang graziang left a comment

Choose a reason for hiding this comment

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

@rmartinc thanks!

@mposolda mposolda self-assigned this Jan 12, 2026
Copy link
Contributor

@mposolda mposolda left a comment

Choose a reason for hiding this comment

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

@rmartinc Thanks for fixing this!

@mposolda mposolda merged commit e6fb3aa into keycloak:main Jan 12, 2026
82 checks passed
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.

Failed upgrade to 26.4.7 - sql generated for manual database upgrade contains invalid statements

3 participants