-
Notifications
You must be signed in to change notification settings - Fork 28
feat!: Add masterkey as credential option #468
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
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files
🚀 New features to boost your workflow:
|
|
Great job! I assume the key should be given (unencrypted) as a new global option, isn't it? Many tools accept command-line arguments, environment variables, and password files (with proper permission check, eg, 600). I don't know if this is a requirement here (or maybe you already implemented that!-)
(emphasis mine) I think the password credentials with the master key encrypted in the repository should remain the default strategy. It is reasonably safe, depending on the SLA of the underlying storage provider. Can the user change the strategy after the repo init? |
The default will be what the user selects. If they don't select anything, The strategy can be changed any time. You can always add new repository keys using |
|
This feature has now also been added to |
Adds the possibility to use a master key directly as credential to open/initialize a repository.
Using the masterkey has the following advantages:
scryptalgorithm.scryptalgorithm, opening a repository is much faster and needs much less resources (CPU/Memory) making this option interesting for some use cases.Advantages of the still supported password credential are the possibility to backup the masterkey in the repository - and of course allowing to access the repo using a noticeable password.
Note that this change allows to use repositories with only the masterkey, but additionally allows to access a "normal" repository with keyfiles with either a password or the masterky.
As a side effect, most integration tests are now much faster as most now use the masterkey.
This is a breaking change as it changes
Repositorymethods.