zeruniverse / Password-Manager

An online keepass-like tool to manage password. client-side AES encryption!
Other
170 stars 44 forks source link

"last changed on " #199

Closed Pofilo closed 6 years ago

Pofilo commented 6 years ago

I just found a weird bug (reproductible on the demo).

Steps to reproduce it:

Screenshots:

image

image

Solution: In the plugin showPasswordAge.js, in account["other"], there is no edititempasswordlastchanged. If we take a look at where/when is edititempasswordlastchanged setted. It is in main.js. We set it when we create a password (as it should). The problem comes from $("#editbtn").click(function(). When we change the password, the date is modified, otherwise forgotten. Solution: do not forget the information.