borgmatic-collective/borgmatic
borgmatic-collective
/
borgmatic
15
57
Fork 40
Code Issues 54 Pull Requests 1 Actions Packages Releases 121 Activity

Update docs/how-to/provide-your-passwords.md #805

Merged
witten merged 1 commits from axel/borgmatic:docs-password-add-pass-command into main 2023-12-30 23:48:33 +00:00
Conversation 3 Commits 1 Files Changed 1 +16 -4
axel commented 2023-12-30 23:12:28 +00:00
Contributor
Copy Link

Provide an explanation of encryption_passcommand.
Also, adjust headers for consistency.

Provide an explanation of encryption_passcommand. Also, adjust headers for consistency.
axel added 1 commit 2023-12-30 23:12:29 +00:00
f23001aa8d Update docs/how-to/provide-your-passwords.md 
Provide an explanation of encryption_passcommand.
Also, adjust headers for consistency.
witten requested changes 2023-12-30 23:18:48 +00:00
witten left a comment
Owner
Copy Link

Thanks for taking the time to do this! I have just a few minor changes requested and then I'd be happy to merge.

Thanks for taking the time to do this! I have just a few minor changes requested and then I'd be happy to merge.
docs/how-to/provide-your-passwords.md Outdated
@ -22,0 +22,4 @@
### Delegating to a another application
borgmatic supports calling another application such as a password manager to
obtain the borg passphrase to a repository.
witten commented 2023-12-30 23:13:59 +00:00
Owner
Copy Link

While borgmatic is all lowercase, Borg isn't. :D

While borgmatic is all lowercase, Borg isn't. :D
axel commented 2023-12-30 23:45:24 +00:00
Author
Contributor
Copy Link

Good point!

Good point!
axel marked this conversation as resolved
docs/how-to/provide-your-passwords.md Outdated
@ -22,0 +24,4 @@
borgmatic supports calling another application such as a password manager to
obtain the borg passphrase to a repository.
For example, to ask password manager to provide the passphrase:
witten commented 2023-12-30 23:16:17 +00:00
Owner
Copy Link

Seems like this should be "a password manager" or "your password manager" or even "the Pass password manager".

Seems like this should be "_a_ password manager" or "_your_ password manager" or even "the Pass password manager".
axel commented 2023-12-30 23:45:39 +00:00
Author
Contributor
Copy Link

Good catch. Correcting.

Good catch. Correcting.
axel marked this conversation as resolved
axel force-pushed docs-password-add-pass-command from f23001aa8d to 4176532317 2023-12-30 23:46:16 +00:00 Compare
axel commented 2023-12-30 23:46:35 +00:00
Author
Contributor
Copy Link

Thanks for taking the time to do this! I have just a few minor changes requested and then I'd be happy to merge.

Should be good now. Thanks!

> Thanks for taking the time to do this! I have just a few minor changes requested and then I'd be happy to merge. Should be good now. Thanks!
witten approved these changes 2023-12-30 23:47:55 +00:00
docs/how-to/provide-your-passwords.md
@ -19,6 +19,18 @@ encryption_passphrase: yourpassphrase
But if you'd rather store them outside of borgmatic, whether for convenience
or security reasons, read on.
### Delegating to a another application
witten commented 2023-12-30 23:47:23 +00:00
Owner
Copy Link

Just noticed this "a another". I'll fix it though after merging to save you from another round of back-and-forth.

Just noticed this "a another". I'll fix it though after merging to save you from another round of back-and-forth.
👍 1
witten merged commit a03c2744e5 into main 2023-12-30 23:48:33 +00:00
witten commented 2023-12-30 23:49:10 +00:00
Owner
Copy Link

Thanks again!

Thanks again!
axel commented 2023-12-30 23:52:09 +00:00
Author
Contributor
Copy Link

Happy to contribute

Happy to contribute
axel deleted branch docs-password-add-pass-command 2023-12-30 23:52:45 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
witten
Labels
Clear labels   
bug

   
data loss

   
design finalized

   
good first issue

   
new feature area

   
question / support

   
security

   
waiting for response
No Label
bug
data loss
design finalized
good first issue
new feature area
question / support
security
waiting for response
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#805
No description provided.
Delete Branch "axel/borgmatic:docs-password-add-pass-command"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?