search

SpiderOak / Encryptr

Encryptr is a zero-knowledge cloud-based password manager / e-wallet powered by Crypton
GNU General Public License v3.0
1.57k stars 137 forks source link

Node crashed while changing password. #242

Closed HellerCommaA closed 8 years ago

HellerCommaA commented 8 years ago

Node crashed while changing password from a complex (20+ character password) to a 10+ character password. OSX 10.11.1

`Uncaught node.js Error

TypeError: Cannot read property 'toJSON' of undefined at changePassphraseCallback (file:///Applications/Encryptr.app/Contents/Resources/app.nw/js/Encryptr.js:2218:72) at file:///Applications/Encryptr.app/Contents/Resources/app.nw/js/crypton.js:822:9 at Request.callback (file:///Applications/Encryptr.app/Contents/Resources/app.nw/js/crypton.js:7741:3) at Request. (file:///Applications/Encryptr.app/Contents/Resources/app.nw/js/crypton.js:7466:10) at Request.Emitter.emit (file:///Applications/Encryptr.app/Contents/Resources/app.nw/js/crypton.js:6952:20) at XMLHttpRequest.xhr.onreadystatechange (file:///Applications/Encryptr.app/Contents/Resources/app.nw/js/crypton.js:7811:10)`

daviddahl commented 8 years ago

I just tried to reproduce, but was unable to. Is this with Encryptr 2.0? Were you able to re-login?

HellerCommaA commented 8 years ago

Yes, it looks like the password was actually reset. I'm on 2.0.0. ...Not entirely sure what happened to be honest. I also just attempted to reproduce and wasn't able to. Will close the issue.

daviddahl commented 8 years ago

Ok, feel free to re-open if it happens again. I am curious what commit of Crypton @devgeeks is using in this 2.0.0 build

devgeeks commented 8 years ago

I don't think it's a Crypton issue... the toJSON seems backbone related...

@HellerCommaA - Make sure to reopen if it happens again :/

HellerCommaA commented 8 years ago

Will do. Wish I had more info to give you guys. Sorry :cry: I even tried to reset to the same passwords I had when this first occurred in order to reproduce. No luck.