Allow other modules to use this login for different user role authentication scenarios

Created on 11 March 2024, about 1 year ago

Updated 30 April 2024, 11 months ago

Problem/Motivation

In #3238067-2: gsislogin project open to new maintainer applications on 5 Oct, 2021 → I had introduced some hooks I would like to add to the module code:

hook_gsislogin_failure_redirect_url_alter(): Alters the url to redirect upon GSIS failure.
hook_gsislogin_success_redirect_url_alter(): Alters the url to redirect upon GSIS success login/registration.
hook_gsislogin_redirect_uri_query_alter(): Allows modules to change the redirect uri's query parameters.
hook_gsislogin_user_exists(): Allows modules to say if the user with the provided details exists.
hook_gsislogin_user_create(): Allows modules to say if the user with the provided data should be created.
hook_gsislogin_user_module_alter(): Allows modules to change the namespace used for the external login.
hook_gsislogin_create_user_details(): Allows modules to act upon a user created/logged-in (eg. if user is authenticated, also gets the role with rid X).

The concept is to allow other modules to provide custom authentication forms or links to the /gsis login page so that, upon return from GSIS, the user gets created with a specific role. This of course could be expanded more as a feature, by allowing other modules to add extra query parameters and use them again upon return.

PS: Some of the hooks above might not be needed anymore in D8+, due to events handling for request data etc. We will see how it goes...

Remaining tasks

Add the hooks to the code.

User interface changes

None

API changes

Hooks to be added. Nothing breaking backwards-compatibility.

✨ Feature request

Status

Needs work

Version

2.0

Component

Code

Created by

🇬🇷Greece vensires

Live updates comments and jobs are added and updated live.

Merge Requests

!2Allow other modules to use this login for different user role authentication scenarios
Open
🇬🇷Greece thanos ntelis
updated about 1 year ago

Comments & Activities

Issue created by @vensires
Comment about 1 year ago →
🇬🇷Greece thanos ntelis Athens
Thanos Ntelis → made their first commit to this issue’s fork.
Merge request !2Allow other modules to use this login for different user role authentication scenarios → (Open) created by thanos ntelis
Status changed to Needs review about 1 year ago9:13am 13 March 2024
Comment about 1 year ago →
🇬🇷Greece thanos ntelis Athens
Status changed to Needs work 11 months ago8:01am 30 April 2024
Comment 11 months ago →
🇬🇷Greece vensires
@thanos-ntelis what's the status of this MR? I still see some TODO comments in the code.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024