search

square / keywhiz

A system for distributing and managing secrets
https://square.github.io/keywhiz/
Apache License 2.0
2.62k stars 216 forks source link

Instructions around db-seed data are unclear #166

Closed alexcole closed 8 years ago

alexcole commented 8 years ago

The docs say "(Optional) Import some development data:" but it seems like if you fail to do this you can't log in with the dev username/password. I just wasted a bunch of time trying to figure our what the username/password was because I failed to run db-seed. I think the instructions should be explicit that this is really necessary to set up the dev account.

alokmenghrajani commented 8 years ago

Agree. We should perhaps render some banner in the web UI saying the database is empty.

viniciusccarvalho commented 8 years ago

I still can't get it working. I ran the db-seed, I've opened the DB using H2 console, and I can see the keywhizAdmin user, and a hashed password, but trying the adminPass password yield a User authentication failed at login for keywhizAdmin

viniciusccarvalho commented 8 years ago

Hum, it does not work on Chrome, tried it on firefox and it works ...

mcpherrinm commented 8 years ago

Not working on Chrome is very strange! I do all my development on Chrome; it certainly WFM. Do you think you could try to capture the two requests to the Keywhiz server from the browsers and file a bug against Keywhiz?

viniciusccarvalho commented 8 years ago

My bad, I had a couple of plugins in chrome messing with cookies and headers, it worked on incognito mode. Sorry about that

alokmenghrajani commented 8 years ago

Glad to hear it now works.

mcpherrinm commented 8 years ago

I've added a command to explicitly add a user. I'll follow up with some new docs tomorrow that explicitly set up a user instead of relying on db-seed in https://github.com/square/keywhiz/pull/171