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

Correct multiple snap issues #934

Merged
merged 1 commit into from
Sep 19, 2017
Merged

Correct multiple snap issues #934

merged 1 commit into from
Sep 19, 2017

Conversation

droidmonkey
Copy link
Member

@droidmonkey droidmonkey commented Sep 10, 2017

Description

Provide fixes to multiple snap issues reported.

Note: In order to use Yubikey you must manually connect the raw-usb interface using: sudo snap connect keepassxc:raw-usb core:raw-usb

How has this been tested?

Manually and unit tests.

Types of changes

  • ✅ Bug fix (non-breaking change which fixes an issue)

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]
  • ✅ I have compiled and verified my code with -DWITH_ASAN=ON. [REQUIRED]

@droidmonkey droidmonkey added this to the v2.2.1 milestone Sep 10, 2017
@droidmonkey droidmonkey changed the base branch from develop to release/2.2.1 September 10, 2017 14:27
@TheZ3ro
Copy link
Contributor

TheZ3ro commented Sep 10, 2017

Note: In order to use Yubikey you must manually connect the raw-usb interface using: sudo snap connect keepassxc:raw-usb core:raw-usb

This needs to go in the documentation and in the FAQ page on the website.

@droidmonkey
Copy link
Member Author

droidmonkey commented Sep 10, 2017

Btw, it looks like the new travis image causes asan to blow up.

@louib
Copy link
Member

louib commented Sep 16, 2017

@droidmonkey the travis build should be passing if you rebase on release/2.2.1

@TheZ3ro TheZ3ro merged commit daef0a3 into release/2.2.1 Sep 19, 2017
@TheZ3ro TheZ3ro deleted the hotfix/snap branch September 19, 2017 11:29
droidmonkey added a commit that referenced this pull request Oct 1, 2017
- Corrected multiple snap issues [#934, #1011]
- Corrected multiple custom icon issues [#708, #719, #994]
- Corrected multiple Yubikey issues [#880]
- Fixed single instance preventing load on occasion [#997]
- Keep entry history when merging databases [#970]
- Prevent data loss if passwords were mismatched [#1007]
- Fixed crash after merge [#941]
- Added configurable auto-type default delay [#703]
- Unlock database dialog window comes to front [#663]
- Translation and compiling fixes
@josechrist
Copy link

josechrist commented Nov 16, 2017

Command line:
It would be nice to have similar options than keepass, that is
keepass <db_URL> -keyfile:<path_to_file> -iousername:<username> -iopassword:<password> -ioscomplete -minimize
However, it would be much better to have -iousername:<path_to_file>
where file contains "username passwd" to hide username and passwd from process details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants