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.

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

New login screen fails with useEncryptedPasswords #611

Closed
Muesly opened this issue Dec 5, 2016 · 13 comments
Closed

New login screen fails with useEncryptedPasswords #611

Muesly opened this issue Dec 5, 2016 · 13 comments

Comments

@Muesly
Copy link

Muesly commented Dec 5, 2016

I've upgraded to 1.0.21 and the new login screen, which appears fine, but come up with this page when entering in my details:

Error: Illegal arguments: undefined, string
at Error (native)
at Object.bcrypt.compareSync (/Users/tony/Code/Lost/parse-dashboard/node_modules/bcryptjs/dist/bcrypt.js:230:19)
at /Users/tony/Code/Lost/parse-dashboard/node_modules/parse-dashboard/Parse-Dashboard/Authentication.js:91:65
at Array.find (native)
at Authentication.authenticate (/Users/tony/Code/Lost/parse-dashboard/node_modules/parse-dashboard/Parse-Dashboard/Authentication.js:88:21)
at /Users/tony/Code/Lost/parse-dashboard/node_modules/parse-dashboard/Parse-Dashboard/Authentication.js:40:21
at pass (/Users/tony/Code/Lost/parse-dashboard/node_modules/passport/lib/authenticator.js:347:9)
at Authenticator.deserializeUser (/Users/tony/Code/Lost/parse-dashboard/node_modules/passport/lib/authenticator.js:352:5)
at SessionStrategy.authenticate (/Users/tony/Code/Lost/parse-dashboard/node_modules/passport/lib/strategies/session.js:53:28)
at attempt (/Users/tony/Code/Lost/parse-dashboard/node_modules/passport/lib/middleware/authenticate.js:348:16)

I have the setting "useEncryptedPasswords" : true in my config, and login worked in 1.0.19 with the same config.

Any ideas?

@sidan5
Copy link

sidan5 commented Dec 5, 2016

Same here!

@jpm
Copy link

jpm commented Dec 5, 2016

Same here. Setting the parse-dashboard package version to 1.0.19 fixes the problem.

@drizzt
Copy link

drizzt commented Dec 6, 2016

Nice unit tests ;)

@richard4339
Copy link

richard4339 commented Dec 6, 2016

So Ubuntu only supports node 4.2.6, and parse-dashboard requires 4.3. I initially assumed it was related to this, but upgraded to the latest version and it still fails.

@bdevore17
Copy link

Having same issue. Getting "Internal Server Error"

@dvanwinkle
Copy link
Contributor

I'll look into this tomorrow morning. I wrote the original encrypted passwords part, but, haven't upgraded to the latest version with the login screen yet.

Thanks for reporting this @Muesly!

@dvanwinkle
Copy link
Contributor

Ah, looks like @lsohn already fixed this in #609. It has been accepted and merged into master already.

This can be closed @flovilmart.

@richard4339
Copy link

richard4339 commented Dec 8, 2016

Will there be a new release soon since this seems like a pretty major bug?

Edit: I manually applied the patch from #609 and I can confirm for those waiting on a release it does fix the problem

@jankcat
Copy link

jankcat commented Dec 11, 2016

Any ETA on this being put into a release?

@bdevore17
Copy link

Looks like @flovilmart just cut the release fixing this issue.

@flovilmart
Copy link
Contributor

yes, available on NPM: https://www.npmjs.com/package/parse-dashboard
and docker: https://hub.docker.com/r/parseplatform/parse-dashboard/tags/

I appologize to the other maintainers for cutting the line of the PR flow 🙇

@hatpick
Copy link

hatpick commented Jan 9, 2017

It's still failing on 1.0.22
After I enter my credential, it says invalid username or password.
Removing useEncryptedPasswords solves the issue. Let me know if you need more information.
I had the issue since updating from 1.0.18 to 1.0.19. #588

@mldb
Copy link

mldb commented Aug 14, 2023

same here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests