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

Fix: Regenerate transform seed and transform master key on save. #1068

Merged
merged 1 commit into from
Oct 19, 2017
Merged

Fix: Regenerate transform seed and transform master key on save. #1068

merged 1 commit into from
Oct 19, 2017

Conversation

rockihack
Copy link
Contributor

Description

Fix: #1051

Motivation and context

How has this been tested?

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:

  • ✅ 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]
  • ✅ My change requires a change to the documentation and I have updated it accordingly.
  • ✅ I have added tests to cover my changes.

@phoerious
Copy link
Member

Would you be so kind to rebase this against release/2.2.2?

@phoerious phoerious added this to the v2.2.2 milestone Oct 13, 2017
@rockihack
Copy link
Contributor Author

@phoerious I can do that but I think cherry-picking is better?

@phoerious
Copy link
Member

This patch should be targeted at release/2.2.2 which will then be merged back into develop. So no need to post against develop directly.

@rockihack rockihack changed the base branch from develop to release/2.2.2 October 13, 2017 11:33
@droidmonkey droidmonkey merged commit c6f83b9 into keepassxreboot:release/2.2.2 Oct 19, 2017
phoerious added a commit that referenced this pull request Oct 21, 2017
- Fixed entries with empty URLs being reported to KeePassHTTP clients [#1031]
- Fixed YubiKey detection and enabled CLI tool for AppImage binary [#1100]
- Added AppStream description [#1082]
- Improved TOTP compatibility and added new Base32 implementation [#1069]
- Fixed error handling when processing invalid cipher stream [#1099]
- Fixed double warning display when opening a database [#1037]
- Fixed unlocking databases with --pw-stdin [#1087]
- Added ability to override QT_PLUGIN_PATH environment variable for AppImages [#1079]
- Fixed transform seed not being regenerated when saving the database [#1068]
- Fixed only one YubiKey slot being polled [#1048]
- Corrected an issue with entry icons while merging [#1008]
- Corrected desktop and tray icons in Snap package [#1030]
- Fixed screen lock and Google fallback settings [#1029]
@phoerious phoerious added pr: bugfix Pull request that fixes a bug and removed bug labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: bugfix Pull request that fixes a bug security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants