If you use Drupal as a backend to a non-Drupal Frontend, the frontend does not know about the password and there can be no testing.

Drupal redirects the newly logged in user to the /user page for the account, but if that is on the frontend, there is no redirect to the /user edit page.

CommentFileSizeAuthor
#7 password_policy-redirect_after_login-3158097-7.patch1.17 KBit-cru
#5 password_policy-redirect_after_login-3158097-5.patch1.17 KBit-cru
#4 3158097-redirect-after-login-alpha5.patch1.23 KBkilles@www.drop.org
#3 3158097-redirect-after-login.patch1.25 KBkilles@www.drop.org

Comments

killes@www.drop.org created an issue. See original summary.

killes@www.drop.org’s picture

killes@www.drop.org commented
Title: Add support for redirect after login » Add support for explicit redirect after login
killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Active » Needs review
StatusFileSize
new 3158097-redirect-after-login.patch1.25 KB

Attached patch tries to implement this.

killes@www.drop.org’s picture

killes@www.drop.org commented
StatusFileSize
new 3158097-redirect-after-login-alpha5.patch1.23 KB

Just in case somebody else still uses alpha5...

it-cru’s picture

it-cru
Primary language English
Location Munich
commented
StatusFileSize
new password_policy-redirect_after_login-3158097-5.patch1.17 KB

The 3158097-redirect-after-login.patch doesn't apply anymore.

Refactored 3158097-redirect-after-login.patch to apply again.

Status: Needs review » Needs work

The last submitted patch, 5: password_policy-redirect_after_login-3158097-5.patch, failed testing. View results

it-cru’s picture

it-cru
Primary language English
Location Munich
commented
StatusFileSize
new password_policy-redirect_after_login-3158097-7.patch1.17 KB

Replace usage of drupal_set_message() with \Drupal::messenger()->addError().

it-cru’s picture

it-cru
Primary language English
Location Munich
commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 7: password_policy-redirect_after_login-3158097-7.patch, failed testing. View results

it-cru’s picture

it-cru
Primary language English
Location Munich
commented

Is this maybe obsolete with 4.0.x ?

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented
Status: Needs work » Postponed (maintainer needs more info)

Thanks to everyone for the work on this issue.

I'm going through all the 8.x issues.

As the 8.x is no longer supported, I'm postponing this issue for now and need feedback as to whether or not this issue is relevant to 4.0.x.

If it is, please reopen and change the version, make sure the issue summary is clear and complete, including steps to reproduce, and reroll the patch. If it's not, please close.

If there is no response to this in a month addressing the above, it can be closed.