Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Fix for plaintext passwords in DB if using CLI for user creation 😱 #619

Merged
scriptsrc merged 1 commit into from
Mar 17, 2017
Merged

Fix for plaintext passwords in DB if using CLI for user creation 😱 #619

scriptsrc merged 1 commit into from
Mar 17, 2017

Conversation

@mikegrima
Copy link
Contributor

@mikegrima mikegrima commented Mar 17, 2017 •
edited
Loading

Note: This has an alembic DB script that will find all unencrypted DB passwords, and DELETE THE PASSWORDS FOR THE USER.

The user must use the recover password feature if affected.

@mikegrima mikegrima self-assigned this Mar 17, 2017
@mikegrima mikegrima requested a review from scriptsrc March 17, 2017 21:51
@mikegrima mikegrima changed the title Fix for ptxt passwords in db if using CLI for user creation 😱 Fix for plaintext passwords in db if using CLI for user creation 😱 Mar 17, 2017
@mikegrima mikegrima mentioned this pull request Mar 17, 2017
Closed
@mikegrima mikegrima changed the title Fix for plaintext passwords in db if using CLI for user creation 😱 Fix for plaintext passwords in DB if using CLI for user creation 😱 Mar 17, 2017
@coveralls
Copy link

coveralls commented Mar 17, 2017

Coverage Status

Coverage increased (+0.004%) to 56.567% when pulling 7587d34 on mikegrima:fix_bcrypt into dff1ef9 on Netflix:develop.

@scriptsrc scriptsrc merged commit 90c0263 into Netflix:develop Mar 17, 2017
@mikegrima mikegrima deleted the fix_bcrypt branch March 17, 2017 23:21
This was referenced Apr 14, 2017
Merged
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@scriptsrc scriptsrc Awaiting requested review from scriptsrc

Assignees

@mikegrima mikegrima

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mikegrima @coveralls @scriptsrc