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

Smarter handling of "Last Accessed" #2005

Open
ftorto opened this issue May 28, 2018 · 7 comments
Open

Smarter handling of "Last Accessed" #2005

ftorto opened this issue May 28, 2018 · 7 comments

Comments

@ftorto
Copy link

ftorto commented May 28, 2018

Expected Behavior

When I copy the password or when I use autofill (from keepassxc, not from browser with plugin), I expect the accessed date of the entry to be updated to now.

Current Behavior

The access date seems to be equal to the last modified date

Possible Solution

  • Precise the usage of access date if it's not related to my expected behaviour
  • Add a new checkbox option in settings to activate the update upon autofill/copy password to clipboard actions

Operating system: Ubuntu 16.04
CPU architecture: x86_64
Kernel: 4.13.0-41

@droidmonkey
Copy link
Member

The problem with "accessed" is that every time you interact with the database it would "dirty" it and require a save. If autosave is on, then it would save everytime you did anything. This is not a good behavior, especially if you are using a robust key derivation like Argon2.

My opinion is to get rid of accessed all together.

@ftorto
Copy link
Author

ftorto commented May 28, 2018

I'm not against the idea to get rid of it. My objective was to track the unused websites.
Another way to do this could be to set a default expiration date to X months and check very old expired passwords.
I can't be confident with "created date" because I just imported to keepassxc from a CSV format which didn't had this field.

@TheZ3ro
Copy link
Contributor

TheZ3ro commented May 29, 2018

My opinion is to get rid of accessed all together.

We can't really do this since the accessed property is defined in the KDBX standard.

Using access date to track unused entry seems a great idea to me, but like droidmonkey said, this property will be updated every time and so the kdbx will.

Sincerely, I see the accessed attributes as an attribute tracked when an entry is opened (open the edit interface but no edit is made).

We should look at what other clients do with it.

@droidmonkey
Copy link
Member

By get rid of it, I meant in the GUI only.

@droidmonkey droidmonkey changed the title "Accessed" date update Smarter handling of "Last Accessed" Jul 10, 2019
@spotlesscoder
Copy link

I'd like to have this feature working, not getting rid of it in the GUI.
It's a good measure to check after you let someone use your PC while you forgot to lock keepass whether the person did not steal a credential.

@ghost
Copy link

ghost commented Feb 20, 2021

I agree that if there's no easy way to fix this it should be removed from the GUI because it does nothing and is misleading.

@jbond00747
Copy link

I understand not wanting to change this value on every access and thus need to re-write the file. Would it be possible to set an option along the lines of "Update last accessed value if it's older than (Hours|Days|etc.)" While this wouldn't show you the absolute latest access, it would provide a good balance between figuring out which accounts aren't being used over a long time and minimizing disk writes. (Conceptually this is similar to the relatime option for filesystems on Linux. (https://blog.confirm.ch/mount-options-atime-vs-relatime/))

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

No branches or pull requests

5 participants