Drupal\user\Tests\UserPasswordResetTest	65	1	0
Message	Group	Filename	Line	Function	Status
One-time link is no longer valid.	Other	UserPasswordResetTest.php	81	Drupal\user\Tests\UserPasswordResetTest->testUserPasswordReset()

alrwady seen that twice today

CommentFileSizeAuthor
#4 2017217-userpasswordreset-random-fail.patch782 bytesalexpott

Comments

ParisLiakos’s picture

ParisLiakos commented
Issue tags: +Random test failure

tagging

ParisLiakos’s picture

ParisLiakos commented
Priority: Critical » Normal

lets lower the priority for now, i tried tens of time locally and it does not fail, and i cant see why it could fail

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Priority: Normal » Major

I get this almost every other day, its major enough.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2017217-userpasswordreset-random-fail.patch782 bytes

Talked about this with @chx ... the one time login link is based on the last login in time... if our testbots are running quick and the log and the first usage happen during the same second then the last login link will still be valid... so we need to ensure that the last login time is in the past.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community
dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

Automatically closed -- issue fixed for 2 weeks with no activity.