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

Prompt to migrate KeePassHTTP settings only if Browser Integration is enabled #3327

Closed
janteau opened this issue Jun 27, 2019 · 3 comments · Fixed by #3344
Closed

Prompt to migrate KeePassHTTP settings only if Browser Integration is enabled #3327

janteau opened this issue Jun 27, 2019 · 3 comments · Fixed by #3344
Assignees

Comments

@janteau
Copy link

janteau commented Jun 27, 2019

I would like to retain my KeePassHTTP settings for use on Windows. I use auto-type in KeePassXC, so I do not want to migrate my KeePassHTTP settings. Unfortunately, I am prompted every time that I unlock my database with KeePassXC (even though Browser Integration is disabled). Would it be possible for the migration prompt to only be displayed when the Browser Integration settings are enabled and KeePassHTTP attributes are found?

Current Behavior

When a database with KeePassHTTP entries is unlocked, the user is prompted to migrate these settings. If "No" is chosen, the user will again be prompted on next database unlock.

Expected Behavior

Only prompt to migrate KeePassHTTP settings when Browser Integration is enabled in the settings.

@janteau janteau changed the title Remember do not migrate KeePassHTTP settings Prompt to migrate KeePassHTTP settings only if Browser Integration is enabled Jun 27, 2019
@phoerious
Copy link
Member

I strongly discourage the use of KeePassHTTP and the browser extensions that support it. A lot of work has been put into making our own browser extension as secure as possible. You can use KeePassXC on Windows as well.

@janteau
Copy link
Author

janteau commented Jun 27, 2019

I understand the dislike of KeePassHTTP. I typically use auto-type. I would just prefer to leave my database entries untouched, especially since I'm not using the browser integration feature.

@droidmonkey
Copy link
Member

I don't see anything wrong with this request, if the feature is disabled we should not be prompting the user about the feature.

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

Successfully merging a pull request may close this issue.

4 participants