/qubes-issues

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

Backup should accept a stdin password or a file key #1653

Closed
Rudd-O opened this Issue Jan 17, 2016 · 4 comments

Comments

Assignees
No one assigned
Labels
Projects
None yet
Milestone
  Release 3.2
3 participants
@Rudd-O @cfcs @marmarek
@Rudd-O

Rudd-O commented Jan 17, 2016

Right now there is no way to avoid typing the password in the backup. This bodes badly for automated backups, where the password is stored in the TCB (dom0) as a keyfile or perhaps piped into the backup program from the calling script.

This feature would make it so easy to do automated backups without having to deal with the cancer that is expect.

Thanks!
@cfcs

This comment has been minimized.

Show comment
Hide comment
@cfcs

cfcs Jan 17, 2016

Also useful for stuff like this: https://github.com/cfcs/scramlkb

cfcs commented Jan 17, 2016

Also useful for stuff like this: https://github.com/cfcs/scramlkb
@Rudd-O

This comment has been minimized.

Show comment
Hide comment
@Rudd-O

Rudd-O Jan 19, 2016

Agreed. No need to use the stupid getpass module. There's no additional security by demanding reads from a tty.

Rudd-O commented Jan 19, 2016

Agreed. No need to use the stupid getpass module. There's no additional security by demanding reads from a tty.

@marmarek marmarek added enhancement C: core P: major labels Jan 19, 2016

@marmarek marmarek added this to the Release 3.2 milestone Jan 19, 2016

marmarek added a commit to marmarek/old-qubes-core-admin that referenced this issue Jan 20, 2016

@marmarek
backup: add options useful for scripted backups 
1. No confirmation mode
2. Read password from stdin, instead of TTY

Fixes QubesOS/qubes-issues#1653
Verified
This commit was signed with a verified signature.
@marmarek marmarek Marek Marczykowski-Górecki
GPG key ID: 063938BA42CFA724 Learn about signing commits
161ddc2

@marmarek marmarek referenced this issue in marmarek/old-qubes-core-admin Jan 20, 2016

Closed

backup: add options useful for scripted backups #11

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Jan 20, 2016

Member

Take a look at https://github.com/marmarek/qubes-core-admin/pull/11
Member

marmarek commented Jan 20, 2016

Take a look at https://github.com/marmarek/qubes-core-admin/pull/11

marmarek added a commit to marmarek/old-qubes-core-admin that referenced this issue Jan 25, 2016

@marmarek
backup: add options useful for scripted backups 
1. No confirmation mode
2. Read password from stdin, instead of TTY

Fixes QubesOS/qubes-issues#1653
Verified
This commit was signed with a verified signature.
@marmarek marmarek Marek Marczykowski-Górecki
GPG key ID: 063938BA42CFA724 Learn about signing commits
34816ae
@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek May 15, 2016

Member

Fixed in QubesOS/qubes-core-admin#20
Member

marmarek commented May 15, 2016

Fixed in QubesOS/qubes-core-admin#20

@marmarek marmarek closed this May 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment