search

Kunzisoft / KeePassDX

Lightweight vault and password manager for Android, KeePassDX allows editing encrypted data in a single file in KeePass format and fill in the forms in a secure way.
https://www.keepassdx.com/
GNU General Public License v3.0
4.35k stars 262 forks source link

The notification remains active while the database is closed #979

Closed Vstory closed 3 years ago

Vstory commented 3 years ago

Describe the bug The database unlock status in the notification is incorrect.

To Reproduce Steps to reproduce the behavior:

  1. Set the database unlock timeout to 5 minutes.
  2. Unlock the database.
  3. Press the home button to exit KeePassDX Pro and wait for the database to be automatically locked after timeout.
  4. The notification of the database is still "locked".
  5. The database has been locked, but the notification still prompts "locked".

KeePass Database

KeePassDX:

Android:

J-Jamet commented 3 years ago

I have just tested and the timeout works well. What do you mean by :

the notification still prompts "locked"

"locked" is not an available message for the database notification. There is only "database opened" and the "lock" button when the timeout has not sent the close request.

Vstory commented 3 years ago

"locked" is not an available message for the database notification. There is only "database opened" and the "lock" button when the timeout has not sent the close request.

What I mean is that after the database is automatically locked, the notification still shows that the database can be locked.

Is this a known feature?

J-Jamet commented 3 years ago

No this is not normal, the notification should disappear with the lock. But what disturbs me the most is that I can't reproduce the problem. I hope this is not specific to Xiaomi.

Vstory commented 3 years ago

No this is not normal, the notification should disappear with the lock. But what disturbs me the most is that I can't reproduce the problem. I hope this is not specific to Xiaomi.

I have allowed self-starting and set the battery optimization whitelist. Normally, it should not be restricted by the system. Don't kill my app!

Vstory commented 3 years ago

No this is not normal, the notification should disappear with the lock. But what disturbs me the most is that I can't reproduce the problem. I hope this is not specific to Xiaomi.

Wait a minute, I will try to reproduce and get a logcat log.

Vstory commented 3 years ago

Sorry, I tried a few times just now and it hasn’t recurred.

Vstory commented 3 years ago

logcat.zip The problem reappeared.

J-Jamet commented 3 years ago

I don't even know what I'm looking for in your logs if there's no crash. Can you tell me when you saw the problem because your log is a file over two whole days.

Vstory commented 3 years ago

I don't even know what I'm looking for in your logs if there's no crash. Can you tell me when you saw the problem because your log is a file over two whole days.

It was the next day. At that time, I crawled the log, but it was inconvenient for feedback, so I kept it. This does not affect the use, mainly because the notification status is displayed incorrectly. It occasionally appears on my device.

J-Jamet commented 3 years ago

I fixed a theoretical case where it could happen, I upload version 2.10.0 and you will tell me if you still have the problem.

Vstory commented 3 years ago

I fixed a theoretical case where it could happen, I upload version 2.10.0 and you will tell me if you still have the problem.

OK. Wait for the push of the Google Play Store.

Vstory commented 3 years ago

This problem should be fixed and has not been discovered recently. Thank you.

J-Jamet commented 3 years ago

Thank you for your feedback! ;)