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

feat(keycloak-integration): create authenticator for keycloak #6150

Merged
merged 15 commits into from
Sep 28, 2023

Conversation

uprightech
Copy link
Contributor

@uprightech uprightech commented Sep 28, 2023

This pull request contains the code for the janssen authenticator api for keycloak , which allows keycloak users to authenticate via janssen-auth
Closes #6151,

@uprightech uprightech added the kind-feature Issue or PR is a new feature request label Sep 28, 2023
@uprightech uprightech self-assigned this Sep 28, 2023
@mo-auto
Copy link
Member

mo-auto commented Sep 28, 2023

Error: Hi @uprightech, You did not reference an open issue in your PR. I attempted to create an issue for you.
Please update that issues' title and body and make sure I correctly referenced it in the above PRs body.

Signed-off-by: Rolain Djeumen <uprightech@gmail.com>
Copy link
Member

@moabu moabu left a comment

Choose a reason for hiding this comment

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

At this point the PR looks good but because its introducing a new folder in our monorepo we need to include it in all our processes so we don't miss it.

  1. Build in jenkins which is critical before merging
  2. Security scan which can be added after merging
  3. Setup integration which can be added after merging

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
@moabu moabu merged commit bcecc16 into main Sep 28, 2023
1 check failed
@moabu moabu deleted the keycloak-integration branch September 28, 2023 17:16
devrimyatar pushed a commit that referenced this pull request Dec 30, 2023
* feat(keycloak-integration): added maven parent pom for subprojects

* feat(keycloak-integration): added janssen authenticator pom

* feat(keycloak-integration): added maven assembly xml for authenticator dependencies

* feat(keycloak-integration): added spi factory discovery meta for keycloak
* added authenticator spi factory discovery meta for keycloak
* added rest service spi factory discovery meta for keycloak

* feat(keycloak-integration): added authenticator message mapping file

* feat(keycloak-integration): added authenticator image resources

* feat(keycloak-integration): added page templates for authenticator

* feat(keycloak-integration): added authenticator core coded
* added authenticator code
* added authenticator factory code
* added authenticator config properties code

* feat(keycloak-integration): added missing file containing session attribute definitions

* feat(keycloak-integration): added nimbus based minimalist oidc client code

* chore(keycloak-integration): minor change to the authenticator factory

* feat(keycloak-integration): added authn completion rest service

Signed-off-by: Rolain Djeumen <uprightech@gmail.com>

* ci: add scanning

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>

* ci: fix CODEOWNERS

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>

* ci: fix CODEOWNERS

Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>

---------

Signed-off-by: Rolain Djeumen <uprightech@gmail.com>
Signed-off-by: moabu <47318409+moabu@users.noreply.github.com>
Co-authored-by: moabu <47318409+moabu@users.noreply.github.com>
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind-feature Issue or PR is a new feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(keycloak-integration): create authenticator for keycloak -autocreated
3 participants