[Freeipa-devel] [freeipa PR#709][+ack] Fix s4u2self with adtrust

2017-04-11 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/709 Title: #709: Fix s4u2self with adtrust Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#709][comment] Fix s4u2self with adtrust

2017-04-11 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/709 Title: #709: Fix s4u2self with adtrust flo-renaud commented: """ Hi @simo5, I tested webUI authentication with a IPA user and it is working with this patch. """ See the full comment at

[Freeipa-devel] [freeipa PR#697][comment] Create system users for FreeIPA services during package installation

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation pvomacka commented: """ ipa-4-5: * e8a429d9e170955919f2e53e66b580be95e908d9 Create system users for FreeIPA services during package installation master: *

[Freeipa-devel] [freeipa PR#697][-ack] Create system users for FreeIPA services during package installation

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation Label: -ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#697][closed] Create system users for FreeIPA services during package installation

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/697 Author: dkupka Title: #697: Create system users for FreeIPA services during package installation Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/697/head:pr697

[Freeipa-devel] [freeipa PR#697][+pushed] Create system users for FreeIPA services during package installation

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#697][+ack] Create system users for FreeIPA services during package installation

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#709][opened] Fix s4u2self with adtrust

2017-04-11 Thread simo5
URL: https://github.com/freeipa/freeipa/pull/709 Author: simo5 Title: #709: Fix s4u2self with adtrust Action: opened PR body: """ When ADtrust is installed we add a PAC to all tickets, during protocol transition we need to generate a new PAC for the requested user ticket, not check the

[Freeipa-devel] [freeipa PR#697][comment] Create system users for FreeIPA services during package installation

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation stlaz commented: """ While I don't like to omit @adelton comments, this is a test blocker for us. I propose going with @dkupka's comment on adding the GID/UID

[Freeipa-devel] [freeipa PR#697][+ack] Create system users for FreeIPA services during package installation

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#697][synchronized] Create system users for FreeIPA services during package installation

2017-04-11 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/697 Author: dkupka Title: #697: Create system users for FreeIPA services during package installation Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#695][+ack] [4.4] Fix PKCS11 helper

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/695 Title: #695: [4.4] Fix PKCS11 helper Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#708][+ack] Minor typo in details.js

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/708 Title: #708: Minor typo in details.js Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#704][+pushed] WebUI: cert login: Configure name of parameter used to pass username

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/704 Title: #704: WebUI: cert login: Configure name of parameter used to pass username Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#704][closed] WebUI: cert login: Configure name of parameter used to pass username

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/704 Author: dkupka Title: #704: WebUI: cert login: Configure name of parameter used to pass username Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/704/head:pr704

[Freeipa-devel] [freeipa PR#704][comment] WebUI: cert login: Configure name of parameter used to pass username

2017-04-11 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/704 Title: #704: WebUI: cert login: Configure name of parameter used to pass username pvomacka commented: """ ipa-4-5: * a9721e529e7a02eeb40d29cb7820e69cd86d9337 WebUI: cert login: Configure name of parameter used to pass username master: *

Re: [Freeipa-devel] Checking OCSP and CRL during certificate login

2017-04-11 Thread Rob Crittenden
Pavel Vomacka wrote: > Hello, > > With the recent addition of certificate mapping and certificate login > support into WebUI, we need to handle also revoking of certificates > which are used for login. There is ticket which requests this > functionality: https://pagure.io/freeipa/issue/6370 > >

[Freeipa-devel] [freeipa PR#708][opened] Minor typo in details.js

2017-04-11 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/708 Author: Akasurde Title: #708: Minor typo in details.js Action: opened PR body: """ Fixes: https://pagure.io/freeipa/issue/6863 Signed-off-by: Abhijeet Kasurde """ To pull the PR as Git branch: git remote add ghfreeipa

[Freeipa-devel] [freeipa PR#682][closed] ipaserver/dcerpc: unify error processing

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/682 Author: abbra Title: #682: ipaserver/dcerpc: unify error processing Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/682/head:pr682 git checkout pr682 -- Manage

[Freeipa-devel] [freeipa PR#682][comment] ipaserver/dcerpc: unify error processing

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/682 Title: #682: ipaserver/dcerpc: unify error processing MartinBasti commented: """ master: * aef77b3529540ad12939a2cc54996c341c5d49d3 ipaserver/dcerpc: unify error processing * e560899cce20ca7773a5ce46a1c29db1349e8ec7 trust: always use oddjobd

[Freeipa-devel] [freeipa PR#682][+pushed] ipaserver/dcerpc: unify error processing

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/682 Title: #682: ipaserver/dcerpc: unify error processing Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] Checking OCSP and CRL during certificate login

2017-04-11 Thread Alexander Bokovoy
On ti, 11 huhti 2017, Pavel Vomacka wrote: Hello, With the recent addition of certificate mapping and certificate login support into WebUI, we need to handle also revoking of certificates which are used for login. There is ticket which requests this functionality:

[Freeipa-devel] [freeipa PR#682][comment] ipaserver/dcerpc: unify error processing

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/682 Title: #682: ipaserver/dcerpc: unify error processing MartinBasti commented: """ Code LGTM """ See the full comment at https://github.com/freeipa/freeipa/pull/682#issuecomment-293241513 -- Manage your subscription for the Freeipa-devel mailing

[Freeipa-devel] [freeipa PR#682][+ack] ipaserver/dcerpc: unify error processing

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/682 Title: #682: ipaserver/dcerpc: unify error processing Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#705][edited] [WIP] ipatests: Add krbpasswordexpiration tests

2017-04-11 Thread redhatrises
URL: https://github.com/freeipa/freeipa/pull/705 Author: redhatrises Title: #705: [WIP] ipatests: Add krbpasswordexpiration tests Action: edited Changed field: title Original value: """ ipatests: Add krbpasswordexpiration tests """ -- Manage your subscription for the Freeipa-devel mailing

[Freeipa-devel] [freeipa PR#697][synchronized] Create system users for FreeIPA services during package installation

2017-04-11 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/697 Author: dkupka Title: #697: Create system users for FreeIPA services during package installation Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#700][+pushed] Use entry_points for ipa CLI

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/700 Title: #700: Use entry_points for ipa CLI Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#700][comment] Use entry_points for ipa CLI

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/700 Title: #700: Use entry_points for ipa CLI MartinBasti commented: """ master: * bf67974459f093487a1c5a49234769803780ecbe Use entry_points for ipa CLI ipa-4-5: * 1e1e4e8ef2d2486068e17228c8a0f8b1a2b099f5 Use entry_points for ipa CLI """ See

[Freeipa-devel] [freeipa PR#700][closed] Use entry_points for ipa CLI

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/700 Author: tiran Title: #700: Use entry_points for ipa CLI Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/700/head:pr700 git checkout pr700 -- Manage your

[Freeipa-devel] Checking OCSP and CRL during certificate login

2017-04-11 Thread Pavel Vomacka
Hello, With the recent addition of certificate mapping and certificate login support into WebUI, we need to handle also revoking of certificates which are used for login. There is ticket which requests this functionality: https://pagure.io/freeipa/issue/6370 We (me, David and Jan) are

[Freeipa-devel] [freeipa PR#697][comment] Create system users for FreeIPA services during package installation

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/697 Title: #697: Create system users for FreeIPA services during package installation stlaz commented: """ Travis reports wrong usage of the `useradd` command. """ See the full comment at

[Freeipa-devel] [freeipa PR#707][+ack] Minor typo fixes

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/707 Title: #707: Minor typo fixes Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#707][synchronized] Minor typo fixes

2017-04-11 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/707 Author: Akasurde Title: #707: Minor typo fixes Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/707/head:pr707 git checkout pr707 From

[Freeipa-devel] [freeipa PR#707][comment] Minor typo fixes

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/707 Title: #707: Minor typo fixes stlaz commented: """ Thanks, ACK. """ See the full comment at https://github.com/freeipa/freeipa/pull/707#issuecomment-293225820 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#682][synchronized] ipaserver/dcerpc: unify error processing

2017-04-11 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/682 Author: abbra Title: #682: ipaserver/dcerpc: unify error processing Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/682/head:pr682 git checkout pr682 From

[Freeipa-devel] [freeipa PR#703][comment] Remove surplus 'the' in output of ipa-adtrust-install

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/703 Title: #703: Remove surplus 'the' in output of ipa-adtrust-install MartinBasti commented: """ master: * bad0f608c4f44cb36556f305f1290020d37439c6 Remove surplus 'the' in output of ipa-adtrust-install ipa-4-5: *

[Freeipa-devel] [freeipa PR#703][+pushed] Remove surplus 'the' in output of ipa-adtrust-install

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/703 Title: #703: Remove surplus 'the' in output of ipa-adtrust-install Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#703][closed] Remove surplus 'the' in output of ipa-adtrust-install

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/703 Author: MartinBasti Title: #703: Remove surplus 'the' in output of ipa-adtrust-install Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/703/head:pr703 git checkout

[Freeipa-devel] [freeipa PR#702][comment] Correct PyPI package dependencies

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/702 Title: #702: Correct PyPI package dependencies MartinBasti commented: """ For backport to 4.5 ticket and agreement on ticket triage must be done """ See the full comment at https://github.com/freeipa/freeipa/pull/702#issuecomment-293220360 --

[Freeipa-devel] [freeipa PR#707][synchronized] Minor typo fixes

2017-04-11 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/707 Author: Akasurde Title: #707: Minor typo fixes Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/707/head:pr707 git checkout pr707 From

[Freeipa-devel] [freeipa PR#707][-ack] Minor typo fixes

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/707 Title: #707: Minor typo fixes Label: -ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#697][synchronized] Create system users for FreeIPA services during package installation

2017-04-11 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/697 Author: dkupka Title: #697: Create system users for FreeIPA services during package installation Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#707][+ack] Minor typo fixes

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/707 Title: #707: Minor typo fixes Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#707][opened] Minor typo fixes

2017-04-11 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/707 Author: Akasurde Title: #707: Minor typo fixes Action: opened PR body: """ Fixes https://pagure.io/freeipa/issue/6865 Signed-off-by: Abhijeet Kasurde """ To pull the PR as Git branch: git remote add ghfreeipa

[Freeipa-devel] [freeipa PR#697][synchronized] Create system users for FreeIPA services during package installation

2017-04-11 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/697 Author: dkupka Title: #697: Create system users for FreeIPA services during package installation Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#703][synchronized] Remove surplus 'the' in output of ipa-adtrust-install

2017-04-11 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/703 Author: MartinBasti Title: #703: Remove surplus 'the' in output of ipa-adtrust-install Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/703/head:pr703 git

[Freeipa-devel] [freeipa PR#703][comment] Remove surplus 'the' in output of ipa-adtrust-install

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/703 Title: #703: Remove surplus 'the' in output of ipa-adtrust-install stlaz commented: """ We just got an issue opened, you can add it: https://pagure.io/freeipa/issue/6864. """ See the full comment at

[Freeipa-devel] [freeipa PR#671][comment] Slim down dependencies

2017-04-11 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/671 Title: #671: Slim down dependencies tiran commented: """ Now depends on PR #702 """ See the full comment at https://github.com/freeipa/freeipa/pull/671#issuecomment-293195296 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#686][edited] Stabilize make pypi_packages

2017-04-11 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/686 Author: tiran Title: #686: Stabilize make pypi_packages Action: edited Changed field: body Original value: """ Parallel make or flags like IPA_OMIT_INSTALL and IPA_SERVER_WHEELS could like to bad packages for PyPI. Only build the packages we

[Freeipa-devel] [freeipa PR#686][synchronized] Stabilize make pypi_packages

2017-04-11 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/686 Author: tiran Title: #686: Stabilize make pypi_packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/686/head:pr686 git checkout pr686 From

[Freeipa-devel] [freeipa PR#706][comment] Fix CA-less to CA-full upgrade

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/706 Title: #706: Fix CA-less to CA-full upgrade stlaz commented: """ I am not a big fan of creating directories on object initialization, thus this change. Another way of doing this would be of course to check whether the pki-tomcat/alias dir

[Freeipa-devel] [freeipa PR#706][opened] Fix CA-less to CA-full upgrade

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/706 Author: stlaz Title: #706: Fix CA-less to CA-full upgrade Action: opened PR body: """ CertDB would have always created a directory on initialization. This behavior changes here by replacing the truncate argument with create which will only

[Freeipa-devel] [freeipa PR#675][comment] [4.5, master] Fix PKCS11 helper

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/675 Title: #675: [4.5, master] Fix PKCS11 helper stlaz commented: """ I ran the integration test and the result was fine, ACK. """ See the full comment at https://github.com/freeipa/freeipa/pull/675#issuecomment-293180206 -- Manage your

[Freeipa-devel] [freeipa PR#675][+ack] [4.5, master] Fix PKCS11 helper

2017-04-11 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/675 Title: #675: [4.5, master] Fix PKCS11 helper Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#682][synchronized] ipaserver/dcerpc: unify error processing

2017-04-11 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/682 Author: abbra Title: #682: ipaserver/dcerpc: unify error processing Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/682/head:pr682 git checkout pr682 From