Add create command to keepassxc-cli#2352
Closed
coelebs wants to merge 10 commits intokeepassxreboot:developfrom
Closed
Add create command to keepassxc-cli#2352coelebs wants to merge 10 commits intokeepassxreboot:developfrom
coelebs wants to merge 10 commits intokeepassxreboot:developfrom
Conversation
Member
|
@Vi-n thanks for the PR! I think you incorporated changes already merged into develop in this PR. Could you fix that? |
phoerious
requested changes
Oct 20, 2018
2c11ee0 to
9d39f5f
Compare
Command can be used to create a database using the command line. The database can be secured by a key file and/or password. A key file can be specified that will be created when it does not exist. The password will be read from stdin, the same way as it's currently being done for unlocking the database. When no key is specifed the function will fail. Challenge response to unlock the database is not supported. This functionality has been inspired by issue keepassxreboot#2220, and solves point 1 of that issue.
In pull request keepassxreboot#2351 phoerious updated the coding style for cli components. This commit implements similar changes for the new Create component
Command can be used to create a database using the command line. The database can be secured by a key file and/or password. A key file can be specified that will be created when it does not exist. The password will be read from stdin, the same way as it's currently being done for unlocking the database. When no key is specifed the function will fail. Challenge response to unlock the database is not supported. This functionality has been inspired by issue keepassxreboot#2220, and solves point 1 of that issue.
In pull request keepassxreboot#2351 phoerious updated the coding style for cli components. This commit implements similar changes for the new Create component
…to create_command
This was referenced Dec 7, 2018
Member
|
Closing in favor of #2540 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Command can be used to create a database using the command line. The
database can be secured by a key file and/or password. A key file can be
specified that will be created when it does not exist. The password will
be read from stdin, the same way as it's currently being done for
unlocking the database. When no key is specifed the function will fail.
Challenge response to unlock the database is not supported.
This functionality has been inspired by issue #2220, and solves point 1
of that issue.
Examples
A few examples of this command
Creating a database with a password
keepassxc-cli create /path/to/database.kdbxInsert password used to encrypt database (Press enter leave blank):Creating a database with a keyfile
keepassxc-cli create -k /path/to/keyfile.key /path/to/database.kdbxInsert password used to encrypt database (Press enter leave blank):Motivation and context
This functionality was missing from the keepassxc-cli and specifically requested in #2220. A more complete keepassxc-cli improves the scriptability of keepassxc-cl
How has this been tested?
Tested on Linux following edge cases
Types of changes
Checklist:
-DWITH_ASAN=ON. [REQUIRED]