Skip to content
This repository has been archived by the owner on Mar 15, 2022. It is now read-only.

bug: Password reset not working #40

Closed
NSchmittUniWue opened this issue Apr 30, 2019 · 2 comments
Closed

bug: Password reset not working #40

NSchmittUniWue opened this issue Apr 30, 2019 · 2 comments
Assignees
@vikin91

Comments

@NSchmittUniWue
Copy link

Hello,
the password resets are not working properly (version 0.5.2). After a password reset, the user object variable pass has the new plaintext password, yet check_password in Function/Core.pm assumes an encrypted and salted password in pass.
As far as I can see, this is due to line 324 in Controller/Login.pm setting the variable to the plaintext password instead of the encrypted and salted password generated in line 323.
@vikin91 vikin91 self-assigned this Apr 30, 2019
@vikin91
Copy link
Owner

vikin91 commented Apr 30, 2019

Thanks for reporting. I will give it a look in the next days.

vikin91 added a commit that referenced this issue May 1, 2019
@vikin91
Fix: Store hashed password after password reset.
2f7d3ce 
Fixes #40
@vikin91 vikin91 closed this as completed in 4570282 May 1, 2019
vikin91 added a commit that referenced this issue May 1, 2019
@vikin91
Fix: Store hashed password after password reset.
9fd7168 
Fixes #40
vikin91 added a commit that referenced this issue May 1, 2019
@vikin91
Fix: Store hashed password after password reset.
ac6700a 
Fixes #40
@vikin91
Copy link
Owner

vikin91 commented May 1, 2019 •
edited
Loading

This issue should be now fixed for versions >= 0.5.2 in their respective branches.

Versions0.5.2 and 0.5.3 have now their own branches to enable backporting bugfixes.

I would recommend you to do the migration to 0.5.3 and later to 0.6.0 when it is released – I solved a lot of internal problems in 0.6.0. I am happy to support you in this migration in case you need help.

vikin91 added a commit that referenced this issue May 1, 2019
@vikin91
Fix: Store hashed password after password reset.
d81785a 
Fixes #40
vikin91 added a commit that referenced this issue May 24, 2019
@vikin91
Feature: Flat repository (#35)
9eacc08 
* Move from layered repo to a single layer. Add FlatRespository and its Facade
* Refactor: Run tidyall -a
* Fix builds on Travis by updating Text::BibTeX (#37)
* Enable verbose installation
* Remove traces of `Dist::Zilla` support
* Use Text::BibTeX version 0.88
* Remove has 'dateTimeFormat' from BibSpaceDTO
* Name routes for Teams. Remove Arch generator
* Remove non-existing route for teams. Change URL for /unrelated_papers
* Fix Dockerhub builds (#39)
* Add wait-for script
* Fix: Store hashed password after password reset. Fixes #40
* Fix bug for deleting teams. Add tests for uncovered code. Update changelog
* Remove unused code. Refactor + add tests for a disabled registration
* Fix attachment_url test
* Replace eval LocalBibStyle.pm in _function_arith
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@vikin91 vikin91

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vikin91 @NSchmittUniWue