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

Add an option to keep users logged in for longer #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mdebski
Copy link

@mdebski mdebski commented Mar 2, 2023

Add new optional config option:
$wgOAuth2Client['configuration']['remember_me']

It controls whether the users should be logged in the way they would if they checked the "Keep me logged in" checkbox in password-based authentication. Basically, the cookies are then set a bit differently, as described in https://www.mediawiki.org/wiki/Manual:$wgCookieExpiration

The default is false, so the behaviour from before this patch is preserved.

This is useful, as without it in, some setups the user will be logged in only for a short period of time, which makes e.g. editing experience pretty annoying.

Add new optional config option:
$wgOAuth2Client['configuration']['remember_me']

It controls whether the users should be logged in the way they would if they
checked the "Keep me logged in" checkbox in password-based authentication.
Basically, the cookies are then set a bit differently, as described in
https://www.mediawiki.org/wiki/Manual:$wgCookieExpiration

The default is false, so the behaviour from before this patch is
preserved.

This is useful, as without it in, some setups the user will be logged in
only for a short period of time, which makes e.g. editing experience
pretty annoying.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant