[Freeipa-devel] [freeipa PR#804][comment] krb5: make sure KDC certificate is readable

2017-05-22 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/804 Title: #804: krb5: make sure KDC certificate is readable HonzaCholasta commented: """ I have already "solved" this by changing the permissions of the cert file in the `renew_kdc_cert`. Your solution is definitely better, please remove the chmod

[Freeipa-devel] [freeipa PR#804][-ack] krb5: make sure KDC certificate is readable

2017-05-22 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/804 Title: #804: krb5: make sure KDC certificate is readable Label: -ack ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] [freeipa PR#805][+ack] Fix rare race condition with missing ccache file

2017-05-22 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/805 Title: #805: Fix rare race condition with missing ccache file Label: +ack ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] [freeipa PR#805][comment] Fix rare race condition with missing ccache file

2017-05-22 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/805 Title: #805: Fix rare race condition with missing ccache file abbra commented: """ LGTM """ See the full comment at https://github.com/freeipa/freeipa/pull/805#issuecomment-303289438 ___ FreeIPA-devel

[Freeipa-devel] [freeipa PR#805][opened] Fix rare race condition with missing ccache file

2017-05-22 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/805 Author: simo5 Title: #805: Fix rare race condition with missing ccache file Action: opened PR body: """ In some circumstances the ccache file may disappear while mod_auth_gssapi still has a valid cookie and the client is performing a json

[Freeipa-devel] [freeipa PR#804][synchronized] krb5: make sure KDC certificate is readable

2017-05-22 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/804 Author: abbra Title: #804: krb5: make sure KDC certificate is readable Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/804/head:pr804 git checkout pr804

[Freeipa-devel] [freeipa PR#792][comment] named.conf template: add modification warning

2017-05-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/792 Title: #792: named.conf template: add modification warning tomaskrizek commented: """ - fixed the broken comment - mentioned any changes should be documented in FreeIPA or Red Hat IdM documentation I don't think going into more detail is

[Freeipa-devel] [freeipa PR#623][comment] client install: do not assume /etc/krb5.conf.d exists

2017-05-22 Thread frozencemetery
URL: https://github.com/freeipa/freeipa/pull/623 Title: #623: client install: do not assume /etc/krb5.conf.d exists frozencemetery commented: """ Adding on to my previous comment: I've talked with the Debian maintainers, and they plan to add the same `includedir` after the Stretch release.

[Freeipa-devel] [freeipa PR#800][comment] Change python-cryptography to python2-cryptography

2017-05-22 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/800 Title: #800: Change python-cryptography to python2-cryptography MartinBasti commented: """ Voting for explicit `python2-*` prefix """ See the full comment at https://github.com/freeipa/freeipa/pull/800#issuecomment-303077114

[Freeipa-devel] [freeipa PR#771][comment] cert-show: check if certificate_out is in options

2017-05-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/771 Title: #771: cert-show: check if certificate_out is in options martbab commented: """ @frasertweedale @HonzaCholasta can you please continue review of this PR? """ See the full comment at

[Freeipa-devel] [freeipa PR#797][comment] ipa-replica-conncheck: handle ssh not installed

2017-05-22 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/797 Title: #797: ipa-replica-conncheck: handle ssh not installed flo-renaud commented: """ Hi @martbab I initially thought that the code never worked and always triggered an exception, but you are right, it used to skip the check when ssh client is

[Freeipa-devel] [freeipa PR#797][synchronized] ipa-replica-conncheck: handle ssh not installed

2017-05-22 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/797 Author: flo-renaud Title: #797: ipa-replica-conncheck: handle ssh not installed Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/797/head:pr797 git checkout

[Freeipa-devel] [freeipa PR#794][+pushed] ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt

2017-05-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/794 Title: #794: ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt Label: +pushed ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] [freeipa PR#794][closed] ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt

2017-05-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/794 Author: flo-renaud Title: #794: ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/794/head:pr794 git

[Freeipa-devel] [freeipa PR#794][comment] ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt

2017-05-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/794 Title: #794: ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt martbab commented: """ master: * d93264247563937d6d8e3f030a2bffac10572612 ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt ipa-4-5: *

[Freeipa-devel] [freeipa PR#794][synchronized] ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt

2017-05-22 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/794 Author: flo-renaud Title: #794: ipa-ca-install: append CA cert chain into /etc/ipa/ca.crt Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/794/head:pr794 git