? Success
Pull Request for # 6380

User tests: Successful: Unsuccessful:

avatar n9iels
n9iels
10 Mar 2015

This is a fix for issue #6380.

The problem

A password reset on front-end ignore the password criteria giving in the User manager configuration

How to test this patch

  1. Go to User Manager --> options and set:
    • minimum password length: 8
    • Password Minimum Integers: 1
    • Password Minimum Symbols: 1
    • Password Upper Case Minimum: 1
  2. Go to fron-end and reset your password. Give a password what doesn't contain any of the criteria
  3. Notice the password reset was successfully
  4. Apply this patch and reset your password again
  5. Notice the password criteria are no longer ignored
avatar n9iels n9iels - open - 10 Mar 2015
avatar joomla-cms-bot joomla-cms-bot - change - 10 Mar 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 10 Mar 2015
Title
fix for issue #6380
password requirements are ignored when password reset(fix for #6380)
Rel_Number 6380
Relation Type Pull Request for
Easy No Yes
avatar zero-24 zero-24 - change - 10 Mar 2015
Category Authentication
avatar joomdonation
joomdonation - comment - 11 Mar 2015

@test: Success.

Before patch, I can enter any password to reset

After patch, I can only reset the password if the entered password match the requirement settings in the component.

Nice find !

avatar bertmert
bertmert - comment - 11 Mar 2015

@test Success

avatar zero-24 zero-24 - change - 11 Mar 2015
Status Pending Ready to Commit
avatar zero-24 zero-24 - alter_testresult - 12 Mar 2015 - joomdonation: Tested successfully
avatar zero-24 zero-24 - alter_testresult - 12 Mar 2015 - bertmert: Tested successfully
avatar zero-24
zero-24 - comment - 12 Mar 2015

Moving to RTC. Thanks for coding and testing!


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6382.
avatar brianteeman brianteeman - change - 12 Mar 2015
Labels Added: ?
avatar gwsdesk
gwsdesk - comment - 13 Mar 2015

@test works4me. can be merged imho

avatar zero-24 zero-24 - close - 13 Mar 2015
avatar phproberto phproberto - change - 13 Mar 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-03-13 09:27:10
avatar phproberto phproberto - close - 13 Mar 2015
avatar phproberto phproberto - close - 13 Mar 2015
avatar phproberto
phproberto - comment - 13 Mar 2015

Merged. Thanks!

avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment