Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

User unknown in password recovery (zauth) #3274

Closed
rallek opened this issue Dec 6, 2016 · 3 comments · Fixed by #3538
Closed

User unknown in password recovery (zauth) #3274

rallek opened this issue Dec 6, 2016 · 3 comments · Fixed by #3538
Assignees
@Guite
Labels
Bug
Milestone
1.4.4

Comments

@rallek
Copy link
Contributor

rallek commented Dec 6, 2016

Q A
Zikula Version 1.4.4 RC5
PHP Version 5.5.30

Expected behavior

password recovery is working

Actual behavior

The user is not know anymore if he not logged in one time after the zauth is introduced (after upgrade the core).

Steps to reproduce

Upgrade from 1.4.1 to 1.4.4 and try to recover password of an already registered user.

#3256 is not solved completely. It is only solved 70%. But this is not a blocker for RC5
@Guite Guite added this to the 1.4.5 milestone Dec 6, 2016
@Guite Guite mentioned this issue Dec 6, 2016
Merged
@Guite
Copy link
Member

Guite commented Dec 6, 2016

@rallek can you please test whether #3277 fixes this issue for you? It aims not only on lost password, but also on lost user name. If it works, we can still put that into 1.4.4.

@Guite Guite self-assigned this Dec 6, 2016
@rallek
Copy link
Contributor Author

rallek commented Dec 6, 2016

tested both. Great job! That is working well now for both cases.

@Guite
Copy link
Member

Guite commented Dec 7, 2016

@craigh put into 1.4.4?

@craigh craigh modified the milestones: 1.4.4, 1.4.5 Dec 7, 2016
@craigh craigh mentioned this issue Dec 7, 2016
Closed
@craigh craigh closed this as completed in c4761dd Dec 7, 2016
craigh added a commit that referenced this issue Dec 7, 2016
@craigh
Merge pull request #3277 from zikula/issue-3274
d174eb6 
search also for user entities during lost user name and lost password processes, fixed #3274
@craigh craigh mentioned this issue Feb 8, 2017
Closed
craigh added a commit that referenced this issue Apr 9, 2017
@craigh
revert 'search also for user entities during lost user name and lost …
22e5e62 
…password processes, fixed #3274' (reverted from commit c4761dd)
@craigh craigh mentioned this issue Apr 9, 2017
Merged
craigh added a commit that referenced this issue Apr 9, 2017
@craigh
revert 'search also for user entities during lost user name and lost …
effeb0f 
…password processes, fixed #3274' (reverted from commit c4761dd) (#3538)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Guite Guite

Labels
Bug
Projects
None yet
Milestone
1.4.4
Development

Successfully merging a pull request may close this issue.

Revert #3277 zikula/core
3 participants
@Guite @craigh @rallek