Skip to content

Conversation

@ap-wtioit
Copy link
Contributor

Migration of auth_oidc to 19.0

Info @wt-io-it

andreschenkels and others added 30 commits October 1, 2025 13:08
update manifest, add README, update requirements.txt
Add some install instructions and configuration instructions
for keycloak.
Avoid replacing the access token by the id token.
This may cause confusion.
Copy a little piece of code from auth_oauth() method,
to make the code easier to follow, and prepare for
implementing the authorization code flow.
This is not a recommended scenario, but this prepares
the code for using PKCE
create an upstream merge request after
OCA#393 is merged
@ap-wtioit ap-wtioit mentioned this pull request Oct 1, 2025
19 tasks
@pedrobaeza
Copy link
Member

/ocabot migration auth_oidc

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Oct 1, 2025
@ap-wtioit
Copy link
Contributor Author

ap-wtioit commented Oct 1, 2025

Tests are failing because of demo data no longer loaded by default (they still are in our doodba). Will split up the demo keycloak client data and test client data.

Edit: test data is now separated from demo data

@ap-wtioit ap-wtioit force-pushed the 19.0-mig-auth_oidc branch 2 times, most recently from 11b5217 to a5830da Compare October 1, 2025 12:59
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@KirolosMFahem
Copy link

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@sbidoul

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.