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

Password confirmation #706

Merged
merged 1 commit into from
Nov 29, 2019
Merged

Conversation

pspeter
Copy link
Contributor

@pspeter pspeter commented Oct 31, 2019

Stacked on #705. Rebased on Master.

When creating a new password (using --encrypt), jrnl will now ask you to repeat the password. It will only encrypt it after you entered the same password correctly twice.
I've read some people complaining about permanently encrypting their journal because they mistyped their password, this should prevent that.

This does not work during the install yet, but will in #708.

Fixes #481.

Checklist

  • The code change is tested and works locally.
  • Tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?

@pspeter pspeter marked this pull request as ready for review November 2, 2019 01:14
@pspeter pspeter force-pushed the password-confirmation branch from dadbcbf to b5d0081 Compare November 2, 2019 01:27
@pspeter pspeter changed the title Password confirmation [Stacked on #705] Password confirmation Nov 2, 2019
@wren wren added this to the v2.1.1 - Doctor's Orders milestone Nov 9, 2019
@pspeter pspeter force-pushed the password-confirmation branch from 7c74099 to b49ceba Compare November 12, 2019 09:14
@pspeter pspeter changed the title [Stacked on #705] Password confirmation Password confirmation Nov 12, 2019
@wren
Copy link
Member

wren commented Nov 19, 2019

v2.1.1 is getting a bit big, so I'm moving this to v2.1.2.

@wren
Copy link
Member

wren commented Nov 19, 2019

@pspeter Can you add some non-happy-path tests? Like what happens if you mistype the password on the second try?

@pspeter pspeter force-pushed the password-confirmation branch 5 times, most recently from 7f05ea1 to 5103827 Compare November 19, 2019 13:11
@pspeter pspeter force-pushed the password-confirmation branch from 5103827 to e925fe2 Compare November 19, 2019 13:24
@pspeter
Copy link
Contributor Author

pspeter commented Nov 19, 2019

@wren Done 👍

@wren wren changed the base branch from master to develop November 29, 2019 14:40
Copy link
Member

@wren wren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@wren wren merged commit abb81ac into jrnl-org:develop Nov 29, 2019
@pspeter pspeter deleted the password-confirmation branch November 29, 2019 14:43
@wren wren added the enhancement New feature or request label Jan 10, 2020
@lock
Copy link

lock bot commented May 21, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot added the 🔒 Outdated label May 21, 2020
@lock lock bot locked as resolved and limited conversation to collaborators May 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request 🔒 Outdated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ask to confirm password
2 participants