Re: [Freeipa-devel] [PATCH] Bulk

2010-11-05 Thread Rob Crittenden
Adam Young wrote: Renamed the plugin to 'batch' which is a better name than bulk. Added the example to the docs, put a header on it, and removed the changes to internal.py This will blow up if args ends up being empty so we'll need to address that at some point. This particular error I think

Re: [Freeipa-devel] [PATCH] Bulk

2010-11-05 Thread Adam Young
On 11/05/2010 11:15 AM, Rob Crittenden wrote: Adam Young wrote: Renamed the plugin to 'batch' which is a better name than bulk. Added the example to the docs, put a header on it, and removed the changes to internal.py This will blow up if args ends up being empty so we'll need to address tha

[Freeipa-devel] [PATCH] freeipa-admiyo-0072-3-rights-check.patch

2010-11-05 Thread Adam Young
REbased, got the links for add and removed chacked as well, and set defautl to 'rsc' From 1d86318f0129487f55297ad754713192f7b836e8 Mon Sep 17 00:00:00 2001 From: Adam Young Date: Fri, 5 Nov 2010 13:11:56 -0400 Subject: [PATCH] rights check if the field does not have a 'w' for writable in its rig

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-0072-3-rights-check.patch

2010-11-05 Thread Adam Young
On 11/05/2010 02:29 PM, Adam Young wrote: REbased, got the links for add and removed chacked as well, and set defautl to 'rsc' ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel A couple not

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-0072-3-rights-check.patch

2010-11-05 Thread Rob Crittenden
Adam Young wrote: REbased, got the links for add and removed chacked as well, and set defautl to 'rsc' ack ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-0072-3-rights-check.patch

2010-11-05 Thread Rob Crittenden
Adam Young wrote: On 11/05/2010 02:29 PM, Adam Young wrote: REbased, got the links for add and removed chacked as well, and set defautl to 'rsc' ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-d

[Freeipa-devel] [PATCH] 604 revoke certs when disabling and deleting hosts

2010-11-05 Thread Rob Crittenden
When a host is deleted we revoke its certificate, if any. When a host keytab is disabled we disable all the keytabs and revoke the certificates of its services. I've also tried to make it more universal to display certificate details when viewing a record with a certificate in it. rob >From

Re: [Freeipa-devel] [PATCH] 598 add rights to LDAPUpdate base class

2010-11-05 Thread Adam Young
On 11/04/2010 10:47 AM, Rob Crittenden wrote: This makes --rights available to all _mod commands. https://fedorahosted.org/freeipa/ticket/437 rob ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-0072-3-rights-check.patch

2010-11-05 Thread Adam Young
On 11/05/2010 03:16 PM, Rob Crittenden wrote: Adam Young wrote: On 11/05/2010 02:29 PM, Adam Young wrote: REbased, got the links for add and removed chacked as well, and set defautl to 'rsc' ___ Freeipa-devel mailing list Freeipa-devel@redhat.com ht

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-0072-3-rights-check.patch

2010-11-05 Thread Adam Young
On 11/05/2010 04:42 PM, Adam Young wrote: On 11/05/2010 03:16 PM, Rob Crittenden wrote: Adam Young wrote: On 11/05/2010 02:29 PM, Adam Young wrote: REbased, got the links for add and removed chacked as well, and set defautl to 'rsc' ___ Freeipa-dev

[Freeipa-devel] Patches needing review

2010-11-05 Thread Adam Young
Rob's patches: 593 595 599 600 601 602 603 604 Adam's 0073 (Clear fields upon add) Jakub's patch 004 Have I missed any? ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-freeipa-0075-Ticket-Expiration.patch

2010-11-05 Thread Adam Young
On 11/02/2010 03:35 PM, Adam Young wrote: Ticket Expiration This patch handles Kerberos ticket expiration in the UI. Additionally it removes the mod_auth_kerb authorization for elements in the static directory, cutting down on the number of round trips ___

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-freeipa-0073-Clear-fields-after-add.patch

2010-11-05 Thread Endi Sukma Dewata
On 11/3/2010 12:03 PM, Endi Sukma Dewata wrote: On 11/3/2010 8:55 AM, Adam Young wrote: I suspect then that the service add dialog is wrong. This behaviour has been spec'ed and working for a long time. What does it break? The field.setup(dialog, IPA_ADD_UPDATE) should be called before the add

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-freeipa-0075-Ticket-Expiration.patch

2010-11-05 Thread Endi Sukma Dewata
On 11/5/2010 4:03 PM, Adam Young wrote: Ticket Expiration This patch handles Kerberos ticket expiration in the UI. Additionally it removes the mod_auth_kerb authorization for elements in the static directory, cutting down on the number of round trips Rebased It still doesn't apply. Could you

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-freeipa-0075-Ticket-Expiration.patch

2010-11-05 Thread Adam Young
On 11/05/2010 05:26 PM, Endi Sukma Dewata wrote: On 11/5/2010 4:03 PM, Adam Young wrote: Ticket Expiration This patch handles Kerberos ticket expiration in the UI. Additionally it removes the mod_auth_kerb authorization for elements in the static directory, cutting down on the number of round tr

Re: [Freeipa-devel] [PATCH] freeipa-admiyo-freeipa-0075-Ticket-Expiration.patch

2010-11-05 Thread Endi Sukma Dewata
On 11/5/2010 6:52 PM, Adam Young wrote: Here it is. ACK-ish.. :) The error handling on ticket expiration works, but there are some minor issues: 1. The IPA variable is declared twice. +var IPA; var IPA = ( function () { 2. IPA_DEFAULT_JSON_URL should be deleted. var IPA_DEFAULT_JSON_UR

[Freeipa-devel] [PATCH] HBAC Access Time

2010-11-05 Thread Endi Sukma Dewata
Hi, Please review the attached patch. Thanks! https://fedorahosted.org/reviewboard/r/102/ IPA commands now can be defined in these classes: - ipa_command: a single IPA command - ipa_batch_command: a batch command for executing multiple commands on the server side using the new batch plugin