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

Restore script, PBKDF2, and dependency updates #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

EnigmaCurry
Copy link

Thanks very much for this script, I've been hacking on it for the past few days. I've got some updates you may wish to re-incorporate in your branch. Sorry, they're all here in one branch together:

  • Updates to alpine 3.15
  • Updates to postgresql14-client
  • Makes new restore script (fixes Add restore command #7)
  • Introduces common.sh for parts common to both backup and restore
  • Turns on PBKF2 key derivation.

Before turning on PBKF2 I was seeing this warning (and now with this patch is absent):

2022/01/22 03:09:00 12: *** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Makes this warning go away:

```
2022/01/22 03:09:00 12: *** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.
```
@EnigmaCurry
Copy link
Author

I just now see theres other open PRs for the restore feature, I don't mean to step on anyones toes.

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

Successfully merging this pull request may close these issues.

Add restore command
1 participant