Skip to content

Comments

Docs for Keycloak Deployment#642

Merged
GPortas merged 4 commits intoauthentication-oidcfrom
doc/keycloak_for_builtin_users_auth
Apr 4, 2025
Merged

Docs for Keycloak Deployment#642
GPortas merged 4 commits intoauthentication-oidcfrom
doc/keycloak_for_builtin_users_auth

Conversation

@GPortas
Copy link
Contributor

@GPortas GPortas commented Apr 2, 2025

What this PR does / why we need it:

Which issue(s) this PR closes:

  • Closes #

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

@GPortas GPortas marked this pull request as draft April 2, 2025 18:33
@coveralls
Copy link

coveralls commented Apr 2, 2025

Coverage Status

coverage: 97.592% (-0.5%) from 98.075%
when pulling 6f87b5b on doc/keycloak_for_builtin_users_auth
into b9f3906 on develop.

README.md Outdated
>
> On the SPA side, ensure that the PKCE environment variables are set up to connect to the chosen OIDC provider for authentication.
>
> Additionally, to allow the SPI to authenticate users against the Dataverse database, the Dataverse database must be accessible from the Keycloak service within the deployed infrastructure.
Copy link
Member

Choose a reason for hiding this comment

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

@donsizemore can we please try this on https://beta.dataverse.org ? Make the database accessible to https://beta-keycloak.dataverse.org ?

Choose a reason for hiding this comment

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

absolutely. I was waiting on somebody from IQSS to tell me "yes, make this change for beta"

@GPortas GPortas changed the base branch from develop to authentication-oidc April 4, 2025 09:09
@GPortas GPortas requested a review from g-saracca April 4, 2025 09:09
@GPortas GPortas moved this to Ready for Review ⏩ in IQSS Dataverse Project Apr 4, 2025
@GPortas GPortas added SPA.Q1 Not related to any specific Q1 feature Size: 3 A percentage of a sprint. 2.1 hours. Original size: 3 GREI Re-arch GREI re-architecture-related labels Apr 4, 2025
@GPortas GPortas marked this pull request as ready for review April 4, 2025 09:11
@github-project-automation github-project-automation bot moved this from Ready for Review ⏩ to Ready for QA ⏩ in IQSS Dataverse Project Apr 4, 2025
@GPortas GPortas merged commit b1def53 into authentication-oidc Apr 4, 2025
7 of 9 checks passed
@github-project-automation github-project-automation bot moved this from Ready for QA ⏩ to Merged 🚀 in IQSS Dataverse Project Apr 4, 2025
@GPortas GPortas deleted the doc/keycloak_for_builtin_users_auth branch April 4, 2025 11:56
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GREI Re-arch GREI re-architecture-related Original size: 3 Size: 3 A percentage of a sprint. 2.1 hours. SPA.Q1 Not related to any specific Q1 feature

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

5 participants