No Code Attached Yet
avatar robertaonnis
robertaonnis
2 Mar 2022

Expected result

During reset password if I use same password the system ignore if passowrd is equal and permit me to continue.

Actual result

message if password is equal to the previus

System information (as much as possible)

joomla 3.10.6

Additional comments

the problem is on com_user->model->reset funtion processResetComplete();
if ($user->requireReset == 1 && JUserHelper::verifyPassword($data['password1'], $user->password))

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
4.00

avatar robertaonnis robertaonnis - open - 2 Mar 2022
avatar robertaonnis robertaonnis - change - 2 Mar 2022
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 Mar 2022
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 2 Mar 2022
avatar robertaonnis robertaonnis - change - 2 Mar 2022
The description was changed
avatar robertaonnis robertaonnis - edited - 2 Mar 2022
avatar chmst
chmst - comment - 3 Mar 2022

It is expected behaviour to make sure that the password is your own password.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37176.

avatar robertaonnis
robertaonnis - comment - 3 Mar 2022

Yes you are right, in the function veryfypassword.
@.***

I have print the data:

$data[‘password1’]= for example is test
$user->password= is the encrypted password save on db. That I have put to test.

But the function do not return true so continue without alert.

If before this check I decrypt $user->password and in the function I pass the encrypted password from the db so the function return true and told me that the password is equal. Giving me an alert.

I don’t know why is not working.

Do you think is my problem in my joomla? Strange is the last version.

Da: Christiane Maier-Stadtherr @.>
Inviato: giovedì 3 marzo 2022 13:04
A: joomla/joomla-cms @.
>
Cc: robertaonnis @.>; Author @.>
Oggetto: Re: [joomla/joomla-cms] Seams bug on com_user to reset password (Issue #37176)

It is expected behaviour to make sure that the password is your own password.


This comment was created with the J!Tracker Applicationhttps://github.com/joomla/jissues at issues.joomla.org/tracker/joomla-cms/37176https://issues.joomla.org/tracker/joomla-cms/37176.


Reply to this email directly, view it on GitHub#37176 (comment), or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARP6TEYNPQU6XSGKHZ6KDMLU6CTC3ANCNFSM5PX3GY3Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.@.>>

avatar brianteeman
brianteeman - comment - 3 Mar 2022

This is a bug. There is no point in an admin setting a user account to require a password reset if we let them reset to the same password..

avatar robertaonnis
robertaonnis - comment - 3 Mar 2022

sorry I didn't understand what you mean.

I have created a platform. not yet online where users can register freely and request password recovery.

so they can do the reset independently. are you saying this is a bug in com_user?

Da: Brian Teeman @.>
Inviato: giovedì 3 marzo 2022 13:13
A: joomla/joomla-cms @.
>
Cc: robertaonnis @.>; Author @.>
Oggetto: Re: [joomla/joomla-cms] Seams bug on com_user to reset password (Issue #37176)

This is a bug. There is no pint in an admin setting a user account to require a password reset if we let them reset to the same password..


Reply to this email directly, view it on GitHub#37176 (comment), or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARP6TE7SJVXJGSZDNETMGADU6CUD3ANCNFSM5PX3GY3Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.@.>>

avatar brianteeman
brianteeman - comment - 3 Mar 2022

Ah I thought you were talking about the option for an admin to require a user to reset their password

image

avatar robertaonnis
robertaonnis - comment - 3 Mar 2022

Not yet check.

I was talk when user forgot a password and the system send to the user a link to set a new password!

Il giorno 3 mar 2022, alle ore 13:47, Brian Teeman @.***> ha scritto:



Ah I thought you were talking about the option for an admin to require a user to reset their password

[image]https://user-images.githubusercontent.com/1296369/156567576-6ea51588-8602-4f8a-9fbb-bb6d5ade7a59.png


Reply to this email directly, view it on GitHub#37176 (comment), or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARP6TEYAH3265KVHQD3C2YDU6CYG3ANCNFSM5PX3GY3Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

avatar robertaonnis
robertaonnis - comment - 3 Mar 2022

Hi Brian I have discover the issue:
// Check if the user is reusing the current password if required to reset their password
if ($user->requireReset == 1 && JUserHelper::verifyPassword($data['password1'], $user->password,$user->id))
{
$this->setError(JText::_('JLIB_USER_ERROR_CANNOT_REUSE_PASSWORD'));

        return false;
    }

In this point to go inside is necessary that $user->requiReset is true.

But this is never true at this point.

If I delete this is seams work. This function is called only in case of reset asked from end user. So not admin. For admin request the function is in the model of profile.php

Da: Roberta Onnis @.>
Inviato: giovedì 3 marzo 2022 13:50
A: joomla/joomla-cms @.
>
Cc: joomla/joomla-cms @.>; Author @.>
Oggetto: Re: [joomla/joomla-cms] Seams bug on com_user to reset password (Issue #37176)

Not yet check.

I was talk when user forgot a password and the system send to the user a link to set a new password!

Il giorno 3 mar 2022, alle ore 13:47, Brian Teeman @.@.>> ha scritto:


Ah I thought you were talking about the option for an admin to require a user to reset their password

[image]https://user-images.githubusercontent.com/1296369/156567576-6ea51588-8602-4f8a-9fbb-bb6d5ade7a59.png


Reply to this email directly, view it on GitHub#37176 (comment), or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARP6TEYAH3265KVHQD3C2YDU6CYG3ANCNFSM5PX3GY3Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.@.>>

avatar chmst
chmst - comment - 4 Mar 2022

@robertaonnis sorry but I do not understand. So I am unsure if this is still an issue for you or if I can close this.

You say that your problem is:
If a user forgot a password, he can activate the link "forgot my password" in the frontend.
He gets an email with a link and a key.
If he follows the link in the email he can enter a new password.

Expected result

During reset password if I use same password the system ignore if passowrd is equal and permit me to continue.

This is normal bevaviour.

So we can close this issue?

avatar robertaonnis
robertaonnis - comment - 4 Mar 2022

In your opinion if the user ask to reset password and inserting the new password the newest is equal to the old is not a bug sure you can close.

In my opinion is a bug.

Thanks for your assistance.

Roberta Onnis

Il giorno 4 mar 2022, alle ore 21:32, Christiane Maier-Stadtherr @.***> ha scritto:



@robertaonnishttps://github.com/robertaonnis sorry but I do not understand. So I am unsure if this is still an issue for you or if I can close this.

You say that your problem is:
If a user forgot a password, he can activate the link "forgot my password" in the frontend.
He gets an email with a link and a key.
If he follows the link in the email he can enter a new password.

Expected result

During reset password if I use same password the system ignore if passowrd is equal and permit me to continue.

This is normal bevaviour.

So we can close this issue?


Reply to this email directly, view it on GitHub#37176 (comment), or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARP6TE2KUSY6L4IDYYZH3JTU6JXMTANCNFSM5PX3GY3Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

avatar chmst chmst - close - 5 Mar 2022
avatar chmst
chmst - comment - 5 Mar 2022

I think this is correct, as the password was secret before and is secret also after the change.

If an administrator gives a password and requires a change, the new password must be different because the administrator knows it.

avatar chmst chmst - change - 5 Mar 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-03-05 09:52:49
Closed_By chmst

Add a Comment

Login with GitHub to post a comment