[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 g

[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 https

[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 08/09/201

[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 as Git branch: git remo

[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 cainstance.__spawn_instance() exits in

[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: https://www.redhat.com/mailman/listinf

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

2016-08-23 Thread mbasti-rh
Akasurde's pull request #11: "Removed unwanted line break from RefererError Dialog message" was closed See the full pull-request at https://github.com/freeipa/freeipa/pull/11 ... or pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/11/

[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 li

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 Cu

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 ne

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, A

[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 pull/12/h

[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: https://www.redhat.com/mailman/listinfo

[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 li

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

2016-08-23 Thread Tomas Krizek
In that case, the first version of the patch solves the issue. I'm attaching the patch once again, but it's the same as the one in the original message. On 08/23/2016 11:53 AM, Jan Cholasta wrote: On 22.8.2016 19:08, Tomas Krizek wrote: I've attached the updated patch. Hopefully I didn't fo

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 t

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 Tweed

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 16.0

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: https://www.redhat.com/mailman/listinfo/fr

[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. https://fedorahos

[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: https://www.redhat.com/mailman/listinfo/f