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

Add sqlserver to keycloak role jdbc configurations #78

Merged
merged 1 commit into from
May 9, 2023

Conversation

hwo-wd
Copy link
Collaborator

@hwo-wd hwo-wd commented May 9, 2023

Fix #74

Tested locally inside a docker container and a "real" SQL server

@hwo-wd hwo-wd changed the title #74 - add sqlserver support to keycloak role Close #74 - add sqlserver support to keycloak role May 9, 2023
Copy link
Member

@guidograzioli guidograzioli left a comment

Choose a reason for hiding this comment

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

LGTM!

@guidograzioli guidograzioli merged commit e8e0f67 into ansible-middleware:main May 9, 2023
1 check passed
@guidograzioli guidograzioli changed the title Close #74 - add sqlserver support to keycloak role Add sqlserver to keycloak role jdbc configurations May 9, 2023
@guidograzioli guidograzioli added the minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix label May 9, 2023
@hwo-wd hwo-wd deleted the fix/74 branch May 11, 2023 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor_changes New parameters added to modules, or non-breaking behavior changes to existing parameters; no bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Keycloak/RHSSO role should support sqlserver jdbc
2 participants