[Freeipa-devel] [freeipa PR#13] Handled empty hostname in server-del command (synchronize)

2016-08-23 Thread Akasurde
Akasurde's pull request #13: "Handled empty hostname in server-del command" was synchronize See the full pull-request at https://github.com/freeipa/freeipa/pull/13 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/13/head:pr13

[Freeipa-devel] [freeipa PR#14] Tests: Failing intree tests (opened)

2016-08-23 Thread mirielka
mirielka's pull request #14: "Tests: Failing intree tests" was opened PR body: Fixing failing tests in: test_ipalib/test_plugable test_ipalib/test_rpc test_ipaserver/test_ldap All issues were discussed with Jan Cholasta and are results of thin client implementation. See the full pull-request at

[Freeipa-devel] [bind-dyndb-ldap PR#1] [WIP] Port bind-dyndb-ldap to BIND 9.11 (opened)

2016-08-23 Thread pspacek
pspacek's pull request #1: "[WIP] Port bind-dyndb-ldap to BIND 9.11" was opened PR body: This is work in progress patch set, do not merge it yet. See the full pull-request at https://github.com/freeipa/bind-dyndb-ldap/pull/1 ... or pull the PR as Git branch: git remote add ghbind-dyndb-ldap

[Freeipa-devel] [freeipa PR#13] Handled empty hostname in server-del command (comment)

2016-08-23 Thread mbasti-rh
mbasti-rh commented on a pull request NACK, see inline comments See the full comment at https://github.com/freeipa/freeipa/pull/13#issuecomment-241742765 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] [PATCH 0213] support multiple uid values in slapi-nis users map

2016-08-23 Thread thierry bordaz
On 08/23/2016 12:41 PM, Petr Vobornik wrote: On 08/10/2016 05:27 PM, thierry bordaz wrote: On 08/10/2016 04:37 PM, thierry bordaz wrote: On 08/10/2016 12:51 PM, Alexander Bokovoy wrote: On Wed, 10 Aug 2016, Alexander Bokovoy wrote: On Wed, 10 Aug 2016, thierry bordaz wrote: On

[Freeipa-devel] [freeipa PR#13] Handled empty hostname in server-del command (opened)

2016-08-23 Thread Akasurde
Akasurde's pull request #13: "Handled empty hostname in server-del command" was opened PR body: Fixes: https://fedorahosted.org/freeipa/ticket/6248 Signed-off-by: Abhijeet Kasurde See the full pull-request at https://github.com/freeipa/freeipa/pull/13 ... or pull the PR

[Freeipa-devel] [PATCH 0065] Fix ugly quit during external CA installation

2016-08-23 Thread Standa Laznicka
https://fedorahosted.org/freeipa/ticket/6230 From 33d25d76d71ede4b4d4ac3f57663132ac4c6decb Mon Sep 17 00:00:00 2001 From: Stanislav Laznicka Date: Tue, 23 Aug 2016 13:43:24 +0200 Subject: [PATCH] Make installer quit more nicely on external CA installation

[Freeipa-devel] [freeipa PR#11] Removed unwanted line break from RefererError Dialog message (+pushed)

2016-08-23 Thread mbasti-rh
Akasurde's pull request #11: "Removed unwanted line break from RefererError Dialog message" label *pushed* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/11 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#11] Removed unwanted line break from RefererError Dialog message (comment)

2016-08-23 Thread mbasti-rh
mbasti-rh commented on a pull request Fixed upstream master: https://fedorahosted.org/freeipa/changeset/d5a3f10a854f998087fe2c22fcbbbfdd2aba50ce See the full comment at https://github.com/freeipa/freeipa/pull/11#issuecomment-241702681 -- Manage your subscription for the Freeipa-devel mailing

Re: [Freeipa-devel] [PATCH 0034] Secure permissions of Custodia server.keys

2016-08-23 Thread Petr Vobornik
On 08/09/2016 01:53 PM, Martin Basti wrote: > > > On 08.08.2016 16:09, Christian Heimes wrote: >> I have split up patch 0032 into two smaller patches. This patch only >> addresses the server.keys file. >> >> Custodia's server.keys file contain the private RSA keys for encrypting >> and signing

Re: [Freeipa-devel] [PATCH 0035] Remove Custodia server keys from LDAP

2016-08-23 Thread Petr Vobornik
On 08/11/2016 04:13 PM, Martin Basti wrote: > > > On 08.08.2016 16:10, Christian Heimes wrote: >> The server-del plugin now removes the Custodia keys for encryption and >> key signing from LDAP. >> >> https://fedorahosted.org/freeipa/ticket/6015 >> >> > ACK for master > > For 4.3, it requires

Re: [Freeipa-devel] [PATCH 0213] support multiple uid values in slapi-nis users map

2016-08-23 Thread Petr Vobornik
On 08/10/2016 05:27 PM, thierry bordaz wrote: > > > On 08/10/2016 04:37 PM, thierry bordaz wrote: >> >> >> On 08/10/2016 12:51 PM, Alexander Bokovoy wrote: >>> On Wed, 10 Aug 2016, Alexander Bokovoy wrote: On Wed, 10 Aug 2016, thierry bordaz wrote: > > > On 08/09/2016 01:38 PM,

[Freeipa-devel] [freeipa PR#12] Tests: Duplicate declaration on variables in ID views tests (closed)

2016-08-23 Thread mbasti-rh
mirielka's pull request #12: "Tests: Duplicate declaration on variables in ID views tests" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/12 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#12] Tests: Duplicate declaration on variables in ID views tests (+pushed)

2016-08-23 Thread mbasti-rh
mirielka's pull request #12: "Tests: Duplicate declaration on variables in ID views tests" label *pushed* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/12 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#12] Tests: Duplicate declaration on variables in ID views tests (comment)

2016-08-23 Thread mbasti-rh
mbasti-rh commented on a pull request Fixed upstream master: https://fedorahosted.org/freeipa/changeset/fef4b953099b7d22f257bf32b6aa5d422a2831e5 See the full comment at https://github.com/freeipa/freeipa/pull/12#issuecomment-241691499 -- Manage your subscription for the Freeipa-devel mailing

Re: [Freeipa-devel] [PATCH] 0003 Validate key in otptoken-add

2016-08-23 Thread Jan Cholasta
On 22.8.2016 19:08, Tomas Krizek wrote: I've attached the updated patch. Hopefully I didn't forget anything else this time. On 08/22/2016 05:48 PM, Martin Basti wrote: On 22.08.2016 10:22, Tomas Krizek wrote: Seems like a good idea, I'm attaching the updated patch. Autofill does work when

Re: [Freeipa-devel] [PATCH] 0090, 0092..0094 cert-show: show subject alternative names

2016-08-23 Thread Fraser Tweedale
Thanks for review; rebased and updated patch attached. Only 0090 has substantive changes. Cheers, Fraser On Mon, Aug 22, 2016 at 09:22:08AM +0200, Jan Cholasta wrote: > On 19.8.2016 13:11, Fraser Tweedale wrote: > > Bump for review. > > > > On Mon, Aug 15, 2016 at 05:15:16PM +1000, Fraser

Re: [Freeipa-devel] [PATCH] 0003 Validate key in otptoken-add

2016-08-23 Thread Martin Basti
On 22.08.2016 19:05, Martin Basti wrote: On 22.08.2016 17:48, Martin Basti wrote: On 22.08.2016 10:22, Tomas Krizek wrote: Seems like a good idea, I'm attaching the updated patch. Autofill does work when the param is required. On 08/19/2016 04:19 PM, Martin Basti wrote: On

Re: [Freeipa-devel] [Design Review Request] V4/Automatic_Certificate_Request_Generation

2016-08-23 Thread Jan Cholasta
On 8.8.2016 22:23, Ben Lipton wrote: On 07/25/2016 07:45 AM, Jan Cholasta wrote: On 25.7.2016 13:11, Alexander Bokovoy wrote: On Mon, 25 Jul 2016, Jan Cholasta wrote: On 20.7.2016 16:05, Ben Lipton wrote: Hi, Thanks very much for the feedback! Some responses below; I hope you'll let me know

[Freeipa-devel] [freeipa PR#12] Tests: Duplicate declaration on variables in ID views tests (label change)

2016-08-23 Thread freeipa-github-notification
mirielka's pull request #12: "Tests: Duplicate declaration on variables in ID views tests" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/12 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#12] Tests: Duplicate declaration on variables in ID views tests (opened)

2016-08-23 Thread freeipa-github-notification
mirielka's pull request #12: "Tests: Duplicate declaration on variables in ID views tests" was opened PR body: In ipatests/test_xmlrpc/test_idviews_plugin several variables are declared twice, while never using the first declaration. The duplicate declaration is hereby removed.

[Freeipa-devel] [freeipa PR#11] Removed unwanted line break from RefererError Dialog message (label change)

2016-08-23 Thread freeipa-github-notification
Akasurde's pull request #11: "Removed unwanted line break from RefererError Dialog message" label *ack* has been added See the full pull-request at https://github.com/freeipa/freeipa/pull/11 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [PATCH] 0100 Track lightweight CAs on replica installation

2016-08-23 Thread Fraser Tweedale
Hi folks, Please review attached patch which fixes https://fedorahosted.org/freeipa/ticket/6019. Thanks, Fraser From 558ec02053154b472b0505e6c2279095f296cb9c Mon Sep 17 00:00:00 2001 From: Fraser Tweedale Date: Tue, 23 Aug 2016 16:14:30 +1000 Subject: [PATCH] Track