Comments (12)
I don't have a competitor, I wrote this plugin for me based on KeePassWinHello to fit my needs for permanent key storage and I decided to share it for free. You are asking for a removal of a warning I feel it should stay, informing the user that the credential retrieval was canceled by himself, you just need to push Enter, Esc or click the button to get the regular Key Prompt. Remember, according to the license, this software is offered as is and you are free to use whichever plugin fit your needs better. You can also just fork the repository and make the changes you want, as I did myself with this plugin.
from winhellounlock.
Yes, I am aware of the issue, it was introduced when closing #12.
After attempting to open the database, the plugin checks if the file was actually opened, If it was not, then the master key might have changed (Hence, the prompt from the plugin). If KeePass is checking for updates and finds any, it doesn't open the database right away but shows the obsolete components, that's why it fails.
I am testing another way of closing #12 that doesn't cause this issue.
from winhellounlock.
It would be very helpful if you could test this
WinHelloUnlock.zip build from EscButtonBugFix branch with my attempt to fix this issue.
I am testing but even though I downgraded some plugins, I can't get KeePass to check for update when I'm trying to unlock a database.
from winhellounlock.
Just tested. Still shows KeePass warning prompt at pressing Esc "Error decrypting the data. The request was cancelled by the user". Pressing OK in that prompt shows "Enter Password" prompt.
It sounds like you submit the request to decrypt data BEFORE finger scan is done and processed, instead of AFTER that. Esc works well with KeePassWinHello plugin.
Mark KeePass - Options - Advanced - Check for updates at KeePass startup. Then downgrade your plugin version, and re-launch KeePass.
from winhellounlock.
Just tested. Still shows KeePass warning prompt at pressing Esc "Error decrypting the data. The request was cancelled by the user". Pressing OK in that prompt shows "Enter Password" prompt.
That is actually expected behaviour! Esc key should be equivalent to clicking cancel button!
Mark KeePass - Options - Advanced - Check for updates at KeePass startup. Then downgrade your plugin version, and re-launch KeePass.
I did that but for some reason it doesn't work.
Anyway, I am asking about the issue in this thread. Is WinHelloUnlock working when it checks for updates?
from winhellounlock.
Is WinHelloUnlock working when it checks for updates?
For me it doesn't, and for some reason you close bugs without testing. I don't mind though, its your plugin. What happens now:
-
When KeePass shows updates available, WinHelloUnlock prompt appears. If you scan the finger, Update prompt comes to front. Once you Close it, KeePass doesn't open any file, and stays empty.
-
When there are no updates (or KeePass just waits a few days without reminders after showing once an update prompt), WinHelloUnlock prompt appears. If you Esc it, KeePass Error Prompt appears. You said this is expected behavior. But I don't expect such prompt, and KeePassWinHello plugin at Esc doesn't cause such error prompt.
The rest is up to you. I'm just trying to help by reporting errors to make it more convenient to use. If its enough for you to say "its expected" - this is your plugin. But some folks would move back to competitor. You also need to allow at least 48h for users to respond to your questions before closing any bug. :)
from winhellounlock.
Thanks for reporting! I closed because I could get Keepass to check for updates and the database was successfully opened, and don't worry, we can just reopen if needed. Are you using latest v1.3 release?
Also, let me understand correctly, do you push Esc key on the keyboard expecting to Unlock the database?
from winhellounlock.
Sorry, I was using previous build. With latest build, there is no update anymore unless you post a new test build. So I can't test the update behavior.
When Esc is pushed, I expect hidden behind KeePass Enter Password form to appear, not KeePass Error popup, it must proceed quietly with errors suppressed.
from winhellounlock.
To test this issue, downgrade a different plugin and test with latest WinHelloUnlock.
As for the other, I understand now what you mean, you just want the plugin to not report that the credentials were not retrieved because the user canceled! I fixed the plugin showing the error twice. I could add a setting for it to not show at all. Please let's continue this discussion in that issue thread
from winhellounlock.
Hope you can finish the fix without further help. :)
from winhellounlock.
Ooops, your competitor just got a new version, and it doesn't show KeePass window until finger is scanned, which is more logical, and shouldn't lead to problems like this bug.
KeePass shown Update prompt again. WinHelloUnlock finger scan worked, but after closing KeePass Update prompt, the DB remained locked. So your fix didn't work.
If KeePassWinHello plugin devs borrowed your method and code of storing password, they should reference you in the new release as per Github best practices.
from winhellounlock.
As for this issue, you are right, it is not fixed! thanks for pointing that out.
from winhellounlock.
Related Issues (20)
- Build instructions not in README, fresh clone fails with "type or namespace name 'KeePass' could not be found" HOT 1
- Ask for two Hello authentications HOT 2
- Can't decrypt HOT 1
- Error saving the composite key HOT 3
- Error with Keyprovider/Yubikey and composite key HOT 2
- Unhandled Excheption has occurred.
- Plugin incompatible with the current KeePass version (v2.51.1) HOT 1
- Hi - This is not an issue, but requesting if you can add a future. HOT 2
- New day first login error (Credential not retrieved: TPM 2.0 (0x8028008B)) HOT 4
- Hellow bug HOT 4
- The whole database is corrupted after choosing "delete key" in WinUnlock prompt!!! HOT 2
- multipel users / multipel biometric keys HOT 3
- Before install HOT 3
- WinHelloUnlock inkompatibel zu Keepas 2.54 HOT 1
- Windows Update: Plugin stops working HOT 1
- the plugin only allow login to the first password window (when using keepassOTP)
- Resolution
- WinHelloUnlock works only once HOT 1
- Master Key is Invalid Error HOT 3
- Unable to unlock via fingerprint after Windows was locked
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from winhellounlock.