[Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-12 Thread Petr Viktorin
This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not ignored as the ticket asks. Instead it's printed, but not formatted nicely as the client doesn't have enough info for that.

Re: [Freeipa-devel] [PATCH] 0050 Fail on unknown Command options

2012-06-12 Thread Petr Viktorin
On 06/07/2012 11:37 AM, Petr Vobornik wrote: On 05/28/2012 04:16 PM, Martin Kosek wrote: On Mon, 2012-05-28 at 15:46 +0200, Petr Vobornik wrote: On 05/25/2012 09:20 AM, Petr Vobornik wrote: On 05/16/2012 02:11 PM, Martin Kosek wrote: On Wed, 2012-05-16 at 10:37 +0200, Petr Viktorin wrote:

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-12 Thread Simo Sorce
On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not ignored as the ticket asks. Instead it's printed, but not formatted nicely as

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-12 Thread Petr Viktorin
On 06/12/2012 02:38 PM, Simo Sorce wrote: On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not ignored as the ticket asks. Instead it's

Re: [Freeipa-devel] [PATCH] 1025 set fixed primary IPA server in client

2012-06-12 Thread Rob Crittenden
Dmitri Pal wrote: On 06/11/2012 03:45 PM, Rob Crittenden wrote: Add --fixed-primary flag to control the order of ipa_server in sssd.conf. When set the discovered (or passed) server will be set first rather than _srv_. The default is to have _srv_ set first. rob

Re: [Freeipa-devel] [PATCH] 1024 add client session support

2012-06-12 Thread Petr Viktorin
On 06/11/2012 06:49 PM, Martin Kosek wrote: On Thu, 2012-06-07 at 22:55 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: This adds client session support. The session key is stored in the kernel key ring. Your first request should go to /ipa/session/xml where it

Re: [Freeipa-devel] [PATCH] 274 Password change capability for form-based auth

2012-06-12 Thread Martin Kosek
On Mon, 2012-06-11 at 13:52 -0400, Rob Crittenden wrote: Martin Kosek wrote: On Mon, 2012-06-11 at 10:36 +0200, Martin Kosek wrote: On Thu, 2012-06-07 at 23:07 -0400, Simo Sorce wrote: On Thu, 2012-06-07 at 22:28 -0400, Rob Crittenden wrote: Martin Kosek wrote: You can use the attached

Re: [Freeipa-devel] [PATCH] 1024 add client session support

2012-06-12 Thread Rob Crittenden
Petr Viktorin wrote: On 06/11/2012 06:49 PM, Martin Kosek wrote: On Thu, 2012-06-07 at 22:55 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: This adds client session support. The session key is stored in the kernel key ring. Your first request should go to

[Freeipa-devel] Python debugging tip

2012-06-12 Thread John Dennis
Every so often I'm confronted with being unable to determine where an exception is being raised. Unfortunately pdb (Python debugger) cannot be set to break on an exception (oddly missing functionality). It turns out that pdb is built upon a simple Python API, a callback that is invoked

Re: [Freeipa-devel] [PATCH] 274 Password change capability for form-based auth

2012-06-12 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2012-06-11 at 13:52 -0400, Rob Crittenden wrote: Martin Kosek wrote: On Mon, 2012-06-11 at 10:36 +0200, Martin Kosek wrote: On Thu, 2012-06-07 at 23:07 -0400, Simo Sorce wrote: On Thu, 2012-06-07 at 22:28 -0400, Rob Crittenden wrote: Martin Kosek wrote: You can

Re: [Freeipa-devel] [PATCH] 0061 Improve ipa-client-install debug output

2012-06-12 Thread Rob Crittenden
Petr Viktorin wrote: I went through all the output/debug messages in ipa-client-install, removed duplicates, and routed them through the log manager. I used these log levels: DEBUG - detailed messages the user probably doesn't want to see (only printed to console with --debug) INFO - progress

Re: [Freeipa-devel] [PATCH] 156 Action panel for service provisioning

2012-06-12 Thread Endi Sukma Dewata
On 6/7/2012 3:52 AM, Petr Vobornik wrote: Service provisioning status widget was modified only to display the has_keytab status. Button for 'delete key,unprovision' was moved as action to newly created action panel in the same section. This required to moved the creation of the unprovisioning

Re: [Freeipa-devel] [PATCH] 155 Action panel for host enrollment

2012-06-12 Thread Endi Sukma Dewata
On 6/6/2012 10:59 AM, Petr Vobornik wrote: Widgets in host enrollment sections were modified. They now serve only for displaying of has_key and has_password status. Functionality for setting otp and unprovisioning was moved to separate dialogs. Execution points for opening of these dialogs are

Re: [Freeipa-devel] [PATCH] 157 Added password reset capabilities to unauthorized dialog

2012-06-12 Thread Endi Sukma Dewata
On 6/8/2012 10:52 AM, Petr Vobornik wrote: and now the patch... On 06/08/2012 05:51 PM, Petr Vobornik wrote: For those of you who are only interest in user perspective I prepared a set of screenshots to demonstrate workflow of password reset:

Re: [Freeipa-devel] [PATCH] 158 Separate reset password page

2012-06-12 Thread Endi Sukma Dewata
On 6/8/2012 10:54 AM, Petr Vobornik wrote: This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the