keepass-rs icon indicating copy to clipboard operation
keepass-rs copied to clipboard
verified publisher icon sseemayer

Make `Database::open` open database and key files

Open joezak11 opened this issue 4 years ago • 2 comments

joezak11 avatar Apr 15 '22 20:04 joezak11

Thanks for the PR! Can you comment on your rationale for making this change?

sseemayer avatar Apr 15 '22 20:04 sseemayer

Since there is no need to open those files in the main code as I don't think there would any further use for it, it should be cleaner to let the open method handle it

joezak11 avatar Apr 15 '22 20:04 joezak11

As the database and/or keyfile might come from another source than from disk, the currently-existing solution is more general, so I would like to keep it as-is.

sseemayer avatar Dec 29 '22 14:12 sseemayer