Discussion:
[Freeipa-devel] [freeipa PR#747][opened] vault: piped input for ipa vault-add fails
flo-renaud
2017-04-27 17:26:26 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Author: flo-renaud
Title: #747: vault: piped input for ipa vault-add fails
Action: opened

PR body:
"""
An exception is raised when using echo "Secret123\n" | ipa vault-add myvault

This happens because the code is using (string).decode(sys.stdin.encoding)
and sys.stdin.encoding is None when the input is read from a pipe.
The fix is using the prompt_password method defined by Backend.textui,
which gracefully handles this issue.

https://bugzilla.redhat.com/show_bug.cgi?id=1445358
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/747/head:pr747
git checkout pr747
Akasurde
2017-04-28 05:52:35 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

Label: +ack
stlaz
2017-04-28 07:54:01 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

Label: -ack
stlaz
2017-04-28 07:55:01 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

stlaz commented:
"""
@Akasurde: Don't add ACK label when the PR is not OK!
@flo-renaud: You will need to specify a ticket for this PR.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/747#issuecomment-297933288
flo-renaud
2017-04-28 08:02:59 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Author: flo-renaud
Title: #747: vault: piped input for ipa vault-add fails
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/747/head:pr747
git checkout pr747
flo-renaud
2017-04-28 08:04:32 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

flo-renaud commented:
"""
@stlaz
Thank you for the reminder. Commit msg updated with issue 6907
"""

See the full comment at https://github.com/freeipa/freeipa/pull/747#issuecomment-297935155
stlaz
2017-04-28 08:05:51 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

stlaz commented:
"""
Thank you for the brief action taken. Re-adding the ACK label.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/747#issuecomment-297935390
stlaz
2017-04-28 08:05:59 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

Label: +ack
MartinBasti
2017-04-28 11:20:58 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

MartinBasti commented:
"""
master:

* d5c41ed4ad370c7d74296a830993a5bd3fd32e5f vault: piped input for ipa vault-add fails


ipa-4-5:

* c8ca0f89a68b5d57c56344fdeb12fd436976c726 vault: piped input for ipa vault-add fails


"""

See the full comment at https://github.com/freeipa/freeipa/pull/747#issuecomment-297973494
MartinBasti
2017-04-28 11:21:02 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Title: #747: vault: piped input for ipa vault-add fails

Label: +pushed
MartinBasti
2017-04-28 11:21:05 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/747
Author: flo-renaud
Title: #747: vault: piped input for ipa vault-add fails
Action: closed

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/747/head:pr747
git checkout pr747

Loading...