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

utility to launch keepassx with database passwords fetched from kdewa… #55

Merged
merged 1 commit into from
Oct 25, 2016

Conversation

jkt628
Copy link
Contributor

@jkt628 jkt628 commented Oct 25, 2016

Description

this utility opens KeePass databases automagically:

  • find kdbx files in the filesystem,
  • look up their passwords stored in kdewallet,
  • use the --pw-stdin feature to launch keepassx.

Motivation and Context

i use lengthy randomly generated passwords for my kdbx files. instead of keeping the plaintext passwords in a file or email or some such i use the kwallet system to encrypt them and this script to decrypt and launch keepassx on the fly. the --pw-stdin feature keeps the passwords out of the environment where they might be visible.

@droidmonkey and i discussed this briefly at #54 (comment)

How Has This Been Tested?

i use this script every day with multiple databases.

since the script is outside keepassx the worst that can happen is a failed launch.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

…llet

Sorry, something went wrong.

Copy link
Member

@droidmonkey droidmonkey left a comment

Choose a reason for hiding this comment

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

I would move the path to search for keepass db's into an env variable or a variable defined at the top of the script. Otherwise great!

@droidmonkey droidmonkey merged commit e97add0 into keepassxreboot:develop Oct 25, 2016
@jkt628
Copy link
Contributor Author

jkt628 commented Oct 25, 2016

good call, will do, thanks.

@phoerious phoerious modified the milestone: v2.1.0 Jan 14, 2017
@Germano0 Germano0 mentioned this pull request Dec 15, 2021
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

Successfully merging this pull request may close these issues.

None yet

3 participants