Skip to content

Support OIDC registration prompt#11703

Open
tiran133 wants to merge 1 commit into
ILIAS-eLearning:release_10from
tiran133:oidc_register
Open

Support OIDC registration prompt#11703
tiran133 wants to merge 1 commit into
ILIAS-eLearning:release_10from
tiran133:oidc_register

Conversation

@tiran133

Copy link
Copy Markdown

Allow the OpenID Connect login endpoint to receive a register flag and carry that intent through the OIDC credentials object. When registration is requested, send prompt=create to the provider instead of the regular forced login prompt.

This lets ILIAS initiate the OIDC request itself, preserving state handling while delegating account registration to the identity provider.

https://openid.net/specs/openid-connect-prompt-create-1_0.html

Allow the OpenID Connect login endpoint to receive a register flag and
carry that intent through the OIDC credentials object. When registration
is requested, send prompt=create to the provider instead of the regular
forced login prompt.

This lets ILIAS initiate the OIDC request itself, preserving state
handling while delegating account registration to the identity provider.
@tiran133

Copy link
Copy Markdown
Author

Hi @mjansenDatabay,

hope this is alright to ping you here. This is another OIDC improvement I added. Maybe you can have look too?

@mjansenDatabay mjansenDatabay added improvement php Pull requests that update Php code labels Jun 30, 2026
@mjansenDatabay

Copy link
Copy Markdown
Contributor

Hi @tiran133 ,

thanks again for your contribution. Even if the change is not big and it also adds a missing impl. detail of the OIDC specification, we just cannot integrate this.

Reference for the implementation is the feature request https://docu.ilias.de/go/wiki/wpage_4827_1357 (you need to be logged in). I was not involved in the initial implementation and don't know why the registration was not part of the implementation, but it is how it is. Means: We need a feature request page for this change (see: https://docu.ilias.de/go/wiki/wpage_788_1357) and have to present this in in of the next JF meetings (see: https://docu.ilias.de/go/dcl/5316).

Best regards,
@mjansenDatabay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants