Skip to content

Feature Auto Login (in conjunction with direct login feature)#50

Closed
ghost wants to merge 2 commits into4.x-devfrom
unknown repository
Closed

Feature Auto Login (in conjunction with direct login feature)#50
ghost wants to merge 2 commits into4.x-devfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Mar 16, 2021

fixes #49

@ghost
Copy link
Copy Markdown
Author

ghost commented Apr 8, 2021

Is there anything I can do to help with reviewing / evaluating this PR?

@ghost
Copy link
Copy Markdown
Author

ghost commented Jun 1, 2021

Is there anything I can do to help with reviewing / evaluating this PR?

1 similar comment
@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 28, 2021

Is there anything I can do to help with reviewing / evaluating this PR?

@scott-amgen
Copy link
Copy Markdown

@DonCarlosGraz Beautiful feature, sadly the feature doesn't work on latest 4.12.3.

Issue #81 still stands.

@scott-amgen
Copy link
Copy Markdown

Not related to auto-login, but I've been able to workaround #81 with #83.

When I logged the events, I noticed with the current Login & LoginOIDC plugins, when an unauthenticated user visits an auth-required page, the user will get bounced around Login:login and LoginOIDC:loginMod until 500/403.

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 3, 2023

Since there is no activity anymore, I guess this pull request will not be accepted?

(Otherwise I will close the PR)

@ghost ghost closed this Feb 19, 2023
This pull request was closed.
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.

Auto Login

1 participant