search

Narigo / keepass-diff

A CLI-tool to diff Keepass (.kdbx) files. Useful, if syncing with Dropbox or NextCloud and getting multiple files due to conflicts.
https://keepass-diff.narigo.dev/
MIT License
306 stars 28 forks source link

Any updates planned? #54

Closed mrghosti3 closed 1 year ago

mrghosti3 commented 1 year ago

Libraries (below) have received updates:

Plus it is unclear which edition the project is using. Maybe I could help with some stuff?

Narigo commented 1 year ago

Hi @mrghosti3 - I haven't come around to update to the newer versions. I know @sseemayer even mentioned me to update and give feedback, but I was lacking time to put into this tool.

If you want to try and update, you can give it a go, I don't think I'm able to do anything in this project for the next 2 weeks at least.

mrghosti3 commented 1 year ago

Oki and is there any tips on how I could approach testing or about pull request submissions?

Narigo commented 1 year ago

I have written some tests in the run-tests.sh shell script. I haven't come around to create a GitHub action that builds and runs this though yet. I suppose creating an action for tests and creating binaries is probably a good idea to do next...

mrghosti3 commented 1 year ago

So updating rpassword and clap crates doesn't require a lot of refactoring, so made separate PRs:

And began working on updating to keepass-rs@0.6.1: https://github.com/mrghosti3/keepass-diff/tree/update-kepass

Narigo commented 1 year ago

Sorry for not getting back to this lately, had lots of stuff on my plate. I have done some work on updating crates before your PRs came in, but I remember being stuck when trying to update keepass-rs. I've added some tests for keepass-rs that showcased my issue and from the looks it should be fixed by now. I need to check this again - I hope I can get back to this weekend.

Narigo commented 1 year ago

Finally got around to finalize the obfuscation feature for the passwords and updated the dependencies as well (see #66). Thanks for pushing me to do it @mrghosti3 !