Re: [Freeipa-devel] [PATCH] 78 Use ldapi: instead of unsecured ldap: in ipa core tools.

2011-02-23 Thread Pavel Zůna
On 2011-02-15 16:36, JR Aquino wrote: On 2/15/11 6:52 AM, "Simo Sorce" wrote: On Tue, 15 Feb 2011 15:19:50 +0100 Pavel Zuna wrote: I can't reproduce this. :-/ For me it goes fine: [root@ipadev tools]# ./ipa-nis-manage enable Directory Manager password: Enabling plugin This setting will n

Re: [Freeipa-devel] Localization patches.

2011-02-23 Thread Pavel Zůna
On 2011-02-23 20:26, Rob Crittenden wrote: Pavel Zůna wrote: Rebased patch 81 and 83 (pygettext). Created a new patch to fix these latest test failures - it was easier than doing a complex rebase. All latest versions of localization patches are attached to this email for review. I tried to

Re: [Freeipa-devel] [PATCH] 78 Use ldapi: instead of unsecured ldap: in ipa core tools.

2011-02-25 Thread Pavel Zůna
On 2011-02-25 18:12, JR Aquino wrote: On 2/25/11 5:58 AM, "Pavel Zuna" wrote: On 02/23/2011 11:53 PM, Simo Sorce wrote: On Wed, 23 Feb 2011 23:41:33 +0100 Pavel Zůna wrote: On 2011-02-15 16:36, JR Aquino wrote: On 2/15/11 6:52 AM, "Simo Sorce"wrote: On Tue,

[Freeipa-devel] [PATCH] Add a new user-add flag param to disable the creation of UPG.

2011-03-28 Thread Pavel Zůna
This patch handles the issue in a kind of stupid way, but I couldn't think of anything better. It adds a new flag parameter to user-add (--noprivate). With this flag, the command marks the private group about to be created for deletion and is deleted after the user is created. The only excepti

Re: [Freeipa-devel] [PATCH] Add a new user-add flag param to disable the creation of UPG.

2011-03-28 Thread Pavel Zůna
On 2011-03-28 23:05, Rob Crittenden wrote: Pavel Zůna wrote: This patch handles the issue in a kind of stupid way, but I couldn't think of anything better. It adds a new flag parameter to user-add (--noprivate). With this flag, the command marks the private group about to be create

Re: [Freeipa-devel] [PATCHES] Bring back old outputting functionality

2010-02-11 Thread Pavel Zůna
Rob Crittenden wrote: Pavel Zuna wrote: Rob Crittenden wrote: Pavel Zuna wrote: I compiled 3 patches, that effectively bring back all the functionality we had before Jasons big patch (i.e. before introducing output validation and the common output interface). --all and --raw are back, but t

Re: [Freeipa-devel] [PATCHES] Bring back old outputting functionality

2010-02-11 Thread Pavel Zůna
Jason Gerard DeRose wrote: On Wed, 2010-02-10 at 10:30 -0500, Rob Crittenden wrote: Pavel Zuna wrote: What I'm saying is that the Env object stores all strings as str and the env command uses the same output_for_cli as LDAP commands, that only use str for binary. So, we either need to override

Re: [Freeipa-devel] [PATCH] 387 fix test failures

2010-02-23 Thread Pavel Zůna
Rob Crittenden wrote: This fixes the failures in the Env due to switching to unicode internally. Now that --all works this also adds the dn to the output in the XML-RPC tests. rob ack. Pavel ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

[Freeipa-devel] [PATCH] Fix bug where parameter cloning didn't clone validating rules.

2010-02-23 Thread Pavel Zůna
I thought we had validation fixed, but this little bit was still missing. Pavel 0001-Fix-bug-where-parameter-cloning-didn-t-clone-validat.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.co

[Freeipa-devel] [PATCH] Replace incorrect use of str.index with str.find in host plugin.

2010-02-23 Thread Pavel Zůna
index was used as if it was find in the validation function. Pavel 0002-Replace-incorrect-use-of-str.index-with-str.find-in-.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/lis

[Freeipa-devel] [PATCH] Complete netgroup attributes.

2010-02-23 Thread Pavel Zůna
Add missing attributes to the netgroup plugin. The plugin will now correctly display membership information and allow searching for netgroups by UUID. Pavel 0003-Complete-netgroup-attributes.patch Description: application/mbox ___ Freeipa-devel mail

[Freeipa-devel] [PATCH] Make the --all option work in Add/Remove Member commands.

2010-02-23 Thread Pavel Zůna
Add/Remove Member commands didn't work with the --all option. They do now. Pavel 0001-Make-the-all-option-work-in-Add-Remove-Member-comman.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.c

[Freeipa-devel] [DOC] about netgroups

2010-02-23 Thread Pavel Zůna
I was asked to complete the documentation of IPA commands on the internal wiki. Unfortunatly, I currently don't have access to it and have decided to put some of the information I've been gathering here on freeipa-devel. It's not a secret after all and is easier to review by other team members.

[Freeipa-devel] [PATCH] Add plugin versioning and dependency checking

2010-03-02 Thread Pavel Zůna
Since there weren't any comments on my proposal about plugin versioning and dependencies, I decided to make a first implementation, that shows what I had in mind. This patch adds two class variables to plugable.Plugin: VERSION and DEPENDENCIES. VERSION is supposed to be a tuple with 2 ints spe

Re: [Freeipa-devel] [PATCH] Add plugin versioning and dependency checking

2010-03-02 Thread Pavel Zůna
Rob Crittenden wrote: Pavel Zůna wrote: Since there weren't any comments on my proposal about plugin versioning and dependencies, I decided to make a first implementation, that shows what I had in mind. Sorry, I had looked at it and never responded. Seeing an implementation does help t

[Freeipa-devel] [PATCH] Add interface for baseldap plugins to register additional callbacks.

2010-03-04 Thread Pavel Zůna
This is somewhat of a tech-preview patch. It works, but the whole concept might need some more work/thinking done. It adds another way to extend plugins without resorting to the versioning system. Until now, every baseldap command had two callbacks. The pre-callback called before data was pa

Re: [Freeipa-devel] [PATCH] Add interface for baseldap plugins to register additional callbacks.

2010-03-04 Thread Pavel Zůna
Rob Crittenden wrote: Pavel Zůna wrote: This is somewhat of a tech-preview patch. It works, but the whole concept might need some more work/thinking done. It adds another way to extend plugins without resorting to the versioning system. Until now, every baseldap command had two callbacks

Re: [Freeipa-devel] [PATCH] Enable LDAPObject subclasses to disable DN normalization in their methods.

2010-03-18 Thread Pavel Zůna
Rob Crittenden wrote: Pavel Zuna wrote: New attribute in LDAPObject: normalize_dn Defaults to True. If False, LDAP* methods won't normalize DNs before passing them to python-ldap. Pavel This also makes entries returned by LDAPSearch a list instead of a tuple. What is the purpose of that?

Re: [Freeipa-devel] [PATCH] 404 ensure priority is unique

2010-03-22 Thread Pavel Zůna
Rob Crittenden wrote: Pavel Zuna wrote: Jason Gerard DeRose wrote: On Fri, 2010-03-12 at 18:01 -0500, Rob Crittenden wrote: Ensure that the group policy priority is unique. We use CoS to determine the order in which group policy is applied. The behavior in CoS is undefined for multiple entri

[Freeipa-devel] about DNs

2010-03-22 Thread Pavel Zůna
Hi, I didn't want to quote the already over-quoted e-mail about DNs from Rich, so I'm "starting a new thread". :) Anyway, if I understand correctly, we should stop using quoted strings in DNs and use escapes instead, so: This: cn="dc=example,dc=com",dc=example,dc=com Becomes this: cn=dc\=exa

[Freeipa-devel] [PATCH] Fix output of commands, that do not return entries.

2010-03-24 Thread Pavel Zůna
This patch fixes Command.output_for_cli for the env plugin. Before we assumed, that a list/tuple is always a list of entries and a dict is always an entry. Still, this solution isn't perfect. I think, that in the future, we should allow Output subclasses to control the way we output values in

Re: [Freeipa-devel] [PATCH] Fix output of commands, that do not return entries.

2010-03-24 Thread Pavel Zůna
Pavel Zůna wrote: This patch fixes Command.output_for_cli for the env plugin. Before we assumed, that a list/tuple is always a list of entries and a dict is always an entry. Still, this solution isn't perfect. I think, that in the future, we should allow Output subclasses to control th

[Freeipa-devel] [PATCH] Fix output of env plugin. It displayed more than it should.

2010-03-24 Thread Pavel Zůna
Some outputs were missing the 'no_display' flag. Pavel 0001-Fix-output-of-env-plugin.-It-displayed-more-than-it-.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeip

Re: [Freeipa-devel] [PATCH] Fix output of commands, that do not return entries.

2010-03-24 Thread Pavel Zůna
Rob Crittenden wrote: Pavel Zůna wrote: Pavel Zůna wrote: This patch fixes Command.output_for_cli for the env plugin. Before we assumed, that a list/tuple is always a list of entries and a dict is always an entry. Still, this solution isn't perfect. I think, that in the future, we s

Re: [Freeipa-devel] [PATCH] Fix DNS plugin: proper output definitions, --all, dns-add-rr overwritting

2010-04-14 Thread Pavel Zůna
On 4/13/2010 10:51 PM, Rob Crittenden wrote: Pavel Zuna wrote: The DNS plugin is getting old, tired and already looking forward to his pension in the Carribean. It will be replaced soon by a younger, faster, safer, shorter (in terms of code) and more maintainable version. Until that happens, her

Re: [Freeipa-devel] [PATCH] Fix DNS plugin: proper output definitions, --all, dns-add-rr overwritting

2010-04-14 Thread Pavel Zůna
On 4/14/2010 5:36 PM, Rob Crittenden wrote: Pavel Zůna wrote: On 4/13/2010 10:51 PM, Rob Crittenden wrote: Pavel Zuna wrote: The DNS plugin is getting old, tired and already looking forward to his pension in the Carribean. It will be replaced soon by a younger, faster, safer, shorter (in

Re: [Freeipa-devel] Use ldap2 instead of legacy LDAP code from v1 in installer scripts.

2010-04-14 Thread Pavel Zůna
On 4/14/2010 4:35 PM, Rob Crittenden wrote: Pavel Zuna wrote: On 03/30/2010 10:27 PM, Rob Crittenden wrote: Pavel Zuna wrote: On 03/23/2010 09:40 PM, Rob Crittenden wrote: Pavel Zuna wrote: This is the first in a series of patches, that replace all the legacy code from v1 related to LDAP. I

[Freeipa-devel] [PATCH] Fix ipa-dns-install. It was failing when DNS was reinstalling.

2010-04-14 Thread Pavel Zůna
I noticed a few bugs when DNS was reinstalling: - Service.move_service returned None, because the service entry was already in the right place - BindInstance didn't expect that. - We were passing a unicode string to python-ldap although we know it hates that. - We were catching all exceptio

Re: [Freeipa-devel] Use ldap2 instead of legacy LDAP code from v1 in installer scripts.

2010-04-16 Thread Pavel Zůna
On 4/15/2010 8:18 PM, Rob Crittenden wrote: Pavel Zůna wrote: On 4/14/2010 4:35 PM, Rob Crittenden wrote: Pavel Zuna wrote: On 03/30/2010 10:27 PM, Rob Crittenden wrote: Pavel Zuna wrote: On 03/23/2010 09:40 PM, Rob Crittenden wrote: Pavel Zuna wrote: This is the first in a series of

Re: [Freeipa-devel] [PATCH] Use escapes in DNs instead of quoting.

2010-04-16 Thread Pavel Zůna
On 4/16/2010 5:09 PM, Rob Crittenden wrote: Pavel Zuna wrote: This patch effectively removes all LDAPv2 style quoted DNs and makes sure we don't use them anymore. KDC doesn't seem to have any problems with LDAPv3 style DNs, but I kept the option to disable DN normalization for now. I also had

[Freeipa-devel] [PATCH] Add new pwpolicy plugin based on baseldap classes

2010-04-16 Thread Pavel Zůna
Don't mind the numbering. This is a completely independent patch. It adds a new pwpolicy plugin based on baseldap.py classes. It has the same functionality as the current pwpolicy plugin, but a more clean and consistent interface, fine grained search capabilities, etc. This is actually an upd

[Freeipa-devel] [PATCH] Add file with example plugins/tutorial.

2010-04-20 Thread Pavel Zůna
Note that this is still work in progress and will be finished in another patch. Specifically, it currently doesn't cover baseldap.py classes. Pavel From b0e269d1616cc5283866b9d830f76d20c223e94b Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Tue, 20 Apr 2010 18:32:11 +0200 Subject: [PATCH] Add f

Re: [Freeipa-devel] [PATCH] 432 add default open HBAC on install

2010-05-05 Thread Pavel Zůna
On 2010-05-04 21:30, Rob Crittenden wrote: Create an HBAC that allows all users to access all hosts from any host. This should make initial installation and testing easier. It is expected that this rule (allow_all) will be removed before deployment. In case you know you don't want this you can p

Re: [Freeipa-devel] [PATCH] 433 improve hbac output

2010-05-05 Thread Pavel Zůna
On 2010-05-04 21:31, Rob Crittenden wrote: This patch adds more attributes to the default output and fixes up some labels. rob ack. Pavel ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] Add exception callback (exc_callback) to baseldap.py classes.

2010-05-12 Thread Pavel Zůna
On 2010-05-10 20:38, Rob Crittenden wrote: Pavel Zuna wrote: The new callback enables plugin authors to supply their own handler for ExecutionError exceptions generated by calls to ldap2 made from the execute method of baseldap.py classes that extend CallbackInterface. Pavel I don't see any r

Re: [Freeipa-devel] [PATCH] Correctly handle EmptyModlist exception in pwpolicy2-mod.

2010-05-12 Thread Pavel Zůna
On 2010-05-10 15:44, Pavel Zuna wrote: EmptyModlist exception was generated by pwpolicy2-mod when modifying policy priority only. It was because the priority attribute is stored outside of the policy entry (in a CoS entry) and there was nothing left to be changed in the policy entry. This patch

Re: [Freeipa-devel] [PATCH] 439 spec file cleanups

2010-06-03 Thread Pavel Zůna
On 2010-05-10 20:11, Rob Crittenden wrote: Remove references to Fedora < 10 and add some tests for RHEL 6. rob ack. Pavel ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 451 fix i18n test

2010-06-03 Thread Pavel Zůna
On 2010-06-02 21:01, Rob Crittenden wrote: Pavel Zuna wrote: On 05/21/2010 11:35 PM, Rob Crittenden wrote: Fix this test to work from source tree root It would work if you ran the test from its location in tests/test_ipalib but this isn't the most common method. If you want to run it individua

Re: [Freeipa-devel] [Patch] Simple-plugin-for-reflecting-user-principal

2010-08-05 Thread Pavel Zůna
On 2010-08-04 01:49, Adam Young wrote: This is a required patch for the UI code. Basically, the Kerberos authentication method does not provide any way for the web ui to know who logged in. With this patch, we can do the equivalent of 'ipa whoami' that returns the user principal in the summary f

Re: [Freeipa-devel] [PATCH] 502 hosts can fetch keytabs

2010-08-09 Thread Pavel Zůna
On 2010-08-06 04:50, Rob Crittenden wrote: Enable a host to retrieve a keytab for all its services. Using the host service principal one should be able to retrieve a keytab for other services for the host using ipa-getkeytab. This required a number of changes: - allow hosts in the service's man

[Freeipa-devel] [PATCH] Make LDAPObject classes JSON serializable

2010-08-09 Thread Pavel Zůna
Allow LDAPObject classes (and sub-classes) to be serialized into a JSON string using: json.dumps(obj, default=ipalib.util.json_serialize) Pavel From 209162028b58ba8cc59e8c90409082eb8478a0dd Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Mon, 9 Aug 2010 16:45:26 -0400 Subject: [PATCH 1/4] Mak

[Freeipa-devel] [PATCH] Change the behaviour of addattr/setattr parameters

2010-08-09 Thread Pavel Zůna
setattr and addattr can now be used both to set all values of ANY attribute. the last setattr always resets the attribute to the specified value and all addattr append to it. Examples: user-mod testuser --setattr=title=msc title: msc user-mod testuser --setattr=title=msb title: msb user-mod t

[Freeipa-devel] [PATCH] Fix bug: not found exc. handler was failing for singleton objects

2010-08-09 Thread Pavel Zůna
Very minor bug, that would probably never happen in production anyway, but why not fix it. Pavel From 1492f14a59f0e0a1a818b6b920c77ede3f5e2a36 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Mon, 9 Aug 2010 19:50:15 -0400 Subject: [PATCH 3/4] Fix bug: not found exc. handler was failing for sing

[Freeipa-devel] [PATCH] Add new parameters to LDAPSearch: timelimit and sizelimit.

2010-08-09 Thread Pavel Zůna
Doesn't include Robs loading of default values from cn=ipaConfig. Pavel From a81e2b42d16473b0102969246dcf81820b1930f0 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Mon, 9 Aug 2010 19:56:14 -0400 Subject: [PATCH 4/4] Add new parameters to LDAPSearch: timelimit and sizelimit. --- ipalib/plugin

Re: [Freeipa-devel] [Patch] Changes to the python infrastructure-supporting-the-web ui

2010-08-09 Thread Pavel Zůna
On 2010-08-06 17:57, Adam Young wrote: On 08/06/2010 11:51 AM, Rob Crittenden wrote: Adam Young wrote: This code contains the changes to the python code necessary to support the new web ui. It handles the changes to the baseldap code necessary to expose the meta data to the front end, as well a

Re: [Freeipa-devel] [PATCH] Fix bug: not found exc. handler was failing for singleton objects

2010-08-09 Thread Pavel Zůna
On 2010-08-09 23:00, Rob Crittenden wrote: Pavel Zůna wrote: Very minor bug, that would probably never happen in production anyway, but why not fix it. Can you describe the situation this could occur in? How useful would the error be if the key isn't available? rob Well, I caught th

Re: [Freeipa-devel] [PATCH] Improve serialization to JSON.

2010-08-11 Thread Pavel Zůna
On 2010-08-10 21:45, Rob Crittenden wrote: Pavel Zuna wrote: - Make it recursive. - Make Param classes serializable. - Take python native data types into account. Pavel Oops, sorry, commented on wrong patch. This looks ok but you have a serious whitespace issue in the def __json__ method. a

Re: [Freeipa-devel] [PATCH] Add a new INTERNAL plugin that exports plugin meta-data into JSON.

2010-08-11 Thread Pavel Zůna
On 2010-08-10 21:47, Rob Crittenden wrote: Pavel Zuna wrote: This is required for the webUI, since we're dropping Genshi. *ehm* :) You can't use this command on the CLI. It takes one optional argument: the name of an IPA object. If not specified, meta-data for all objects are returned. Note: I

Re: [Freeipa-devel] [PATCH] 509 add BuildRequries for authconfig

2010-08-11 Thread Pavel Zůna
On 2010-08-10 22:41, Rob Crittenden wrote: In ipapython/dnsclient.py we import acutil which is provided by authconfig. This can get invoked by setup.py so we need to have it at build time. rob ack. Pavel ___ Freeipa-devel mailing list Freeipa-deve

Re: [Freeipa-devel] [PATCH] Improve serialization to JSON.

2010-08-12 Thread Pavel Zůna
On 2010-08-11 22:55, Rob Crittenden wrote: Pavel Zůna wrote: On 2010-08-10 21:45, Rob Crittenden wrote: Pavel Zuna wrote: - Make it recursive. - Make Param classes serializable. - Take python native data types into account. Pavel Oops, sorry, commented on wrong patch. This looks ok but

Re: [Freeipa-devel] [PATCH] Change the behaviour of addattr/setattr parameters

2010-08-12 Thread Pavel Zůna
On 2010-08-12 04:46, Rob Crittenden wrote: Pavel Zůna wrote: setattr and addattr can now be used both to set all values of ANY attribute. the last setattr always resets the attribute to the specified value and all addattr append to it. Examples: user-mod testuser --setattr=title=msc title: msc

Re: [Freeipa-devel] [PATCH] Change the behaviour of addattr/setattr parameters

2010-08-12 Thread Pavel Zůna
On 2010-08-12 14:38, Rob Crittenden wrote: Pavel Zůna wrote: On 2010-08-12 04:46, Rob Crittenden wrote: Pavel Zůna wrote: setattr and addattr can now be used both to set all values of ANY attribute. the last setattr always resets the attribute to the specified value and all addattr append to

Re: [Freeipa-devel] [PATCH] Hash Params Redux

2010-08-19 Thread Pavel Zůna
On 2010-08-18 22:59, Adam Young wrote: The patch replaces the earlier Hash Params patch. It fixs the build issues, and fixes the group details page as well. Git still reports white space errors, but that's hardly a show stopper. ACK. Pavel ___ Fr

Re: [Freeipa-devel] [PATCH] Fix script tags in index.xhtml. End tag is required.

2010-08-19 Thread Pavel Zůna
On 2010-08-20 00:48, Pavel Zůna wrote: The paste server had some issues with it and end tags are required by the standard anyway. Pavel I forgot to mention that this applies after Adam's 0009 patch (updated Hash Params). Pavel ___ Freeipa-

Re: [Freeipa-devel] BA-BBQ

2010-08-23 Thread Pavel Zůna
On 2010-08-23 16:41, Adam Young wrote: On 08/23/2010 07:51 AM, Pavel Zuna wrote: On 08/23/2010 04:12 AM, Adam Young wrote: Pavel, Thoughts of jquery.ba-bbq have been marinating in the back of my brain. I know that the Back button doesn't work so well with hjashparams, and that BBQ purports to

Re: [Freeipa-devel] [PATCH] Fix script tags in index.xhtml. End tag is required.

2010-08-23 Thread Pavel Zůna
On 2010-08-20 02:26, Adam Young wrote: On 08/19/2010 06:51 PM, Pavel Zůna wrote: On 2010-08-20 00:48, Pavel Zůna wrote: The paste server had some issues with it and end tags are required by the standard anyway. Pavel I forgot to mention that this applies after Adam's 0009 patch (up

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0015 hashchange

2010-08-25 Thread Pavel Zůna
On 2010-08-24 23:57, Adam Young wrote: This is a fairly substantial patch that makes use of the hashchange event to drive the site. It affcets just about all places where click events are triggered. Just started reviewing the patch a few moments ago. Just 2 notes in advance: 1) It doesn't app

[Freeipa-devel] [PATCH] Make changes to details.js. See commit message.

2010-08-25 Thread Pavel Zůna
First off: I know I promised this patch yesterday, but I just didn't make it - I felt asleep on the keyboard and there were still some things to fix. :) This patch has some of the most important things, that we discussed on Tuesday last week, namely: - Add/Remove links are now only available

[Freeipa-devel] [PATCH] Add 'hint' kwarg to Param. Used for context help in webUI.

2010-08-25 Thread Pavel Zůna
Used like this: Str('someparam', cli_name='sp', label='Some Param', hint='Use at own risk!', ), After the following patch (pzuna-freeipa-0021-details), it will show up in the webUI as (details page): Some Param: [ ] Hint: Use at own risk! Pavel pzuna-fre

Re: [Freeipa-devel] [PATCH] Make changes to details.js. See commit message.

2010-08-25 Thread Pavel Zůna
On 2010-08-25 13:54, Pavel Zůna wrote: First off: I know I promised this patch yesterday, but I just didn't make it - I felt asleep on the keyboard and there were still some things to fix. :) This patch has some of the most important things, that we discussed on Tuesday last week, n

Re: [Freeipa-devel] [PATCH] Add 'hint' kwarg to Param. Used for context help in webUI.

2010-08-25 Thread Pavel Zůna
On 2010-08-25 15:52, Rob Crittenden wrote: Pavel Zůna wrote: Used like this: Str('someparam', cli_name='sp', label='Some Param', hint='Use at own risk!', ), After the following patch (pzuna-freeipa-0021-details), it will show up in the webUI as (deta

Re: [Freeipa-devel] [PATCH] 517 updated command documentation

2010-08-26 Thread Pavel Zůna
On 2010-08-25 05:52, Rob Crittenden wrote: David O'Brien reviewed all the per-command documentation last month and provided a ton of great feedback. I've applied all his changes. rob ACK. Pavel ___ Freeipa-devel mailing list Freeipa-devel@redhat.co

Re: [Freeipa-devel] [PATCH] admiyo-freeipa-0016-sampledata.patch

2010-08-30 Thread Pavel Zůna
On 2010-08-30 16:01, Adam Young wrote: THis patch usese the protocol part of the URL to decide whether or not to use sample data. Thus, file: -> use sample data http:, https: -> Don't. This will prevent the use of sample data in the lite-server. It also reduces by one the number of page reques

Re: [Freeipa-devel] Proposed Javascript coding standards

2010-09-14 Thread Pavel Zůna
On 2010-09-13 23:46, Simo Sorce wrote: On Mon, 13 Sep 2010 17:02:19 -0400 Adam Young wrote: The is a really nasty bug that the "same line" policy avoids. Javascript often attempts to guess where you meant to put semicolons, and puts the in for you, without telling you. return { status:

Re: [Freeipa-devel] [PATCH] Check if attribute is single-value before trying to add values to it.

2010-10-14 Thread Pavel Zůna
On 2010-10-14 19:20, Rob Crittenden wrote: Pavel Zuna wrote: On 10/14/2010 12:01 AM, Rob Crittenden wrote: Pavel Zuna wrote: This patch adds a check in ldap2 for single-value attributes. DS doesn't seem to care much about attributes being defined as SINGLE-VALUE except for things like uidNumbe

[Freeipa-devel] [PATCH] Rename parent LDAPObject pkeys in child LDAPObject methods.

2010-11-08 Thread Pavel Zůna
If the parent and child entries have the same attribute as primary key (such as in the DNS schema), we need to rename the parent key to prevent a param name conflict. It has no side effects, because the primary key name is always taken from the LDAPObject params, never from the method params. Pav

[Freeipa-devel] [PATCH] Add new version of DNS plugin: complete rework with baseldap + unit tests.

2010-11-08 Thread Pavel Zůna
Finally, there it is. :) I redesigned the whole thing to fit the baseldap model. Here's some example on how it's used: # create zone 'example.com' # ipa dnszone-add example.com --name=ns.example.com --admin=ad...@example.com # create a resource in zone 'example.com' named 'machine1' # (machi

[Freeipa-devel] [PATCH] Replace 'Locking' in `ipa help user` with 'Disabling'.

2010-11-08 Thread Pavel Zůna
Ticket #452 Pavel >From e03d082523a677b1d57e829ba10c31fd7f8d87b5 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Mon, 8 Nov 2010 22:45:27 -0500 Subject: [PATCH] Replace 'Locking' in `ipa help user` with 'Disabling'. Ticket #452 --- ipalib/plugins/user.py |2 +- 1 files changed, 1 insertion

Re: [Freeipa-devel] [PATCH] Add new version of DNS plugin: complete rework with baseldap + unit tests.

2010-11-08 Thread Pavel Zůna
On 2010-11-09 05:07, Pavel Zůna wrote: Finally, there it is. :) I redesigned the whole thing to fit the baseldap model. Here's some example on how it's used: # create zone 'example.com' # ipa dnszone-add example.com --name=ns.example.com --admin=ad...@example.com # creat

Re: [Freeipa-devel] [PATCH] Add new version of DNS plugin: complete rework with baseldap + unit tests.

2010-11-24 Thread Pavel Zůna
On 2010-11-24 16:21, Adam Young wrote: On 11/24/2010 03:46 AM, Pavel Zuna wrote: On 11/24/2010 03:26 AM, Adam Young wrote: On 11/23/2010 09:37 AM, Pavel Zuna wrote: Finally managed to rewrite the DNS plugin again. Sorry, it took so long, we had training in the office and I also had a nasty bug

[Freeipa-devel] [PATCH] Enable filtering search results by member attributes.

2010-11-28 Thread Pavel Zůna
LDAPSearch base class has now the ability to generate additional options for objects with member attributes. These options are used to filter search results - search only for objects without the specified members. Any class that extends LDAPSearch can benefit from this functionality. This patch e

Re: [Freeipa-devel] [PATCH] Enable filtering search results by member attributes.

2010-11-30 Thread Pavel Zůna
On 2010-11-30 04:06, Rob Crittenden wrote: Pavel Zůna wrote: LDAPSearch base class has now the ability to generate additional options for objects with member attributes. These options are used to filter search results - search only for objects without the specified members. Any class that

[Freeipa-devel] [PATCH] Fix search filter generator in ldap2 for NOT operator.

2010-11-30 Thread Pavel Zůna
Search filters generated from attributes with multiple values were incorrect when the NOT operator was used (ldap.MATCH_NONE). Pavel >From 332fe9fc4d1c10fd20107afdb9328faea7d5c97c Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Tue, 30 Nov 2010 21:53:28 -0500 Subject: [PATCH 1/2] Fix search fil

[Freeipa-devel] [PATCH] Enable custom list of attributes to retrieve effective rights.

2010-12-30 Thread Pavel Zůna
LDAPObject sub-classes can define a custom list of attributes for effective rights retrieval. Fix #677 Pavel >From 98b7456f1c4d872e265ea4e8c26ab42e206375e8 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Thu, 30 Dec 2010 08:48:12 -0500 Subject: [PATCH 2/4] Enable custom list of attributes to r

[Freeipa-devel] [PATCH] Translate IA5Str paramaters the editable text fields in the webUI.

2010-12-30 Thread Pavel Zůna
Fix #684 Pavel >From ea2cd3c1c97c11c216c0c2baff07615d2f14298e Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Thu, 30 Dec 2010 09:11:58 -0500 Subject: [PATCH 3/4] Translate IA5Str paramaters the editable text fields in the webUI. Fix #684 --- install/static/details.js |3 +++ 1 files chang

[Freeipa-devel] [PATCH] Fix 'ipa help permissions'; add 'dns' in allowed types.

2010-12-30 Thread Pavel Zůna
Pavel >From 08534b8e0fe9ab9713599de166a5bfb4232c1776 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Thu, 30 Dec 2010 09:12:35 -0500 Subject: [PATCH 4/4] Fix 'ipa help permissions'; add 'dns' in allowed types. --- ipalib/plugins/permission.py |2 +- 1 files changed, 1 insertions(+), 1 del

Re: [Freeipa-devel] [PATCH] Translate IA5Str paramaters the editable text fields in the webUI.

2010-12-30 Thread Pavel Zůna
On 2010-12-30 10:29, Pavel Zůna wrote: Fix #684 Pavel Left some debugging output in the original patch. Fixed version attached. Pavel >From 508a68cc41f812683cbf62640c3dc2de02dd2981 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Thu, 30 Dec 2010 09:11:58 -0500 Subject: [PATCH] Transl

[Freeipa-devel] [PATCH] Disable action panel links when the selected entry is deleted.

2010-12-30 Thread Pavel Zůna
Fix #685 Pavel >From 2dd2304d8f10272d3e376090cbe2565e21ff627d Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Thu, 30 Dec 2010 16:41:50 -0500 Subject: [PATCH] Disable action panel links when the selected entry is deleted. Fix #685 --- install/static/search.js |6 +- 1 files changed, 5

Re: [Freeipa-devel] [PATCH] admiyo-0127-add-missing-files-in-rpm

2011-01-06 Thread Pavel Zůna
On 2011-01-05 20:57, Adam Young wrote: Had to move some files around, and added to both Makefile.am and ipa.spec ACK. Pavel ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH] Fix crash when displaying values composed of white chars only in CLI.

2011-01-21 Thread Pavel Zůna
Fix #825 Pavel >From 8a7e6119399aa974457eda41b998cb765186d4eb Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Fri, 21 Jan 2011 09:30:23 -0500 Subject: [PATCH] Fix crash when displaying values composed of white-space chars only in CLI. Ticket #825 --- ipalib/cli.py |2 ++ 1 files changed, 2

Re: [Freeipa-devel] [PATCH] Remove deprecated i18n code from ipalib.request and all references to it.

2011-02-04 Thread Pavel Zůna
On 2011-02-04 16:24, Rob Crittenden wrote: Pavel Zuna wrote: On 02/04/2011 04:03 PM, Rob Crittenden wrote: Pavel Zuna wrote: On 02/02/2011 09:36 PM, Rob Crittenden wrote: Pavel Zuna wrote: This ticket effectively fixes the translation of exception messages. Ticket #903 Pavel On hold for

Re: [Freeipa-devel] [PATCH] Send Accept-Language header over XML-RPC and translate on server.

2011-02-04 Thread Pavel Zůna
On 2011-02-04 16:23, Rob Crittenden wrote: Pavel Zuna wrote: This patch makes the ipa client send the Accept-Language header, so that the server can translate things like exceptions, that cannot be translated on the client. It also fixes the language recognition for the webUI. The values in Acc

Re: [Freeipa-devel] [PATCH] Send Accept-Language header over XML-RPC and translate on server.

2011-02-16 Thread Pavel Zůna
On 2011-02-04 18:35, Pavel Zůna wrote: On 2011-02-04 16:23, Rob Crittenden wrote: Pavel Zuna wrote: This patch makes the ipa client send the Accept-Language header, so that the server can translate things like exceptions, that cannot be translated on the client. It also fixes the language

[Freeipa-devel] [PATCH] Translate docstrings.

2011-02-16 Thread Pavel Zůna
This patch prepares the built-in help system for localized docstrings. Pavel freeipa-pzuna-80-docstringloc.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-deve

[Freeipa-devel] [PATCH] Fix translatable strings in ipalib plugins.

2011-02-16 Thread Pavel Zůna
Some translatable strings were in a wrong format a there were some more related issues. This patch tries to fix all of them. Needed for xgettext/pygettext processing. Pavel freeipa-pzuna-81-fixlocstrings.patch Description: application/mbox ___ Freei

[Freeipa-devel] [PATCH] Fix i18n related failures in unit tests.

2011-02-16 Thread Pavel Zůna
Fixes unit test failures cause by the changes introduced in my other localization related patches. Pavel freeipa-pzuna-82-fixlocutests.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/

Re: [Freeipa-devel] [PATCH] Fix translatable strings in ipalib plugins.

2011-02-17 Thread Pavel Zůna
On 2011-02-16 16:25, Pavel Zůna wrote: Some translatable strings were in a wrong format a there were some more related issues. This patch tries to fix all of them. Needed for xgettext/pygettext processing. Pavel Rebased version attached. Pavel freeipa-pzuna-81-2-fixlocstrings.patch

Re: [Freeipa-devel] Localization patches.

2011-02-17 Thread Pavel Zůna
On 2011-02-17 05:09, Rob Crittenden wrote: Pavel Zůna wrote: My efforts in fixing localization all around the framework and preparing it for localizing docstrings have resulted in a lot of patches. Because I understand they have become a bit hard to track, I decided to post them all together in

Re: [Freeipa-devel] Localization patches.

2011-02-18 Thread Pavel Zůna
On 2011-02-17 22:52, Rob Crittenden wrote: Pavel Zůna wrote: On 2011-02-17 05:09, Rob Crittenden wrote: Pavel Zůna wrote: My efforts in fixing localization all around the framework and preparing it for localizing docstrings have resulted in a lot of patches. Because I understand they have

[Freeipa-devel] [PATCH] Use pygettext to generate translatable strings from plugin files.

2011-02-21 Thread Pavel Zůna
This goes on top of my other localization patches! This patch replaces xgettext with a custom pygettext to generate translatable strings from plugin files in ipalib/plugins. pygettext was modified to handle plural forms (credit goes to Jan Hendrik Goellner) and had some bugs fixed by myself. W

[Freeipa-devel] [PATCHES] All-around improvements to baseldap.py classes.

2009-07-31 Thread Pavel Zůna
0001: Enable attribute re-mapping and ordering when printing entries. Also print multiple values on one line separated by commas. --- 0002: Prevent double encoding/decoding when processing compound types. --

[Freeipa-devel] [PATCH] Rewrite pwpolicy plugin based on baseldap.py.

2009-07-31 Thread Pavel Zůna
Fix bugs: 510740, 510739, 510735, 510733, 510532 Pavel 0005-Rewrite-pwpolicy-plugin-based-on-baseldap.py.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH] Replace TYPE_ERROR by ValidationError

2009-07-31 Thread Pavel Zůna
No more tracebacks when an INT parameter is beyond the type limit. Fix bug: 5107333 There will probably be more patches similar to this one in the future. TYPE_ERROR is used in a several places where a better suited exception is available. Pavel 0006-Replace-TYPE_ERROR-by-ValidationError.

[Freeipa-devel] [PATCH] Fix typo bug in aci.py

2009-07-31 Thread Pavel Zůna
Pavel 0007-Fix-typo-bug-in-aci.py.patch Description: application/mbox ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] Add wrapping when printing multi-value attributes that don't fit on one line.

2009-08-05 Thread Pavel Zůna
Nathan Kinder wrote: On 08/04/2009 09:32 AM, Pavel Zuna wrote: This is how it looks like in practice: ./ipa user-show pzuna --all --raw -- user-show: -- ... memberof: cn=ipausers,cn=groups,cn=accounts,dc=pzuna, cn=dráčci,cn=groups,cn=accounts,dc=pzuna For an attr

[Freeipa-devel] [PATCH] Make ldap2.add_entry proof to None values, because python-ldap hate'em.

2009-09-08 Thread Pavel Zůna
I posted this patch on freeipa-devel a while ago. We had a discussion with Jason about converting unused parameter values to None, but it died for some reason. Anyway this patch is required if we want to prevent things from blowing up when someone accidentally passes a None value to ldap2.add_e

[Freeipa-devel] [PATCH] Fix bug in dns_find - execute() returned different value than expected.

2009-09-08 Thread Pavel Zůna
dns_find.execute() wasn't returning the truncated (truncated search results) flag. It threw an exception when invoked. Pavel 0002-Fix-bug-in-dns_find-execute-returned-different-v.patch Description: application/mbox ___ Freeipa-devel mailing list Free

[Freeipa-devel] [PATCH] Improve attribute printing in the CLI.

2009-09-08 Thread Pavel Zůna
This patch makes printing of LDAP attributes more versatile. The caller can now choose if attributes with more than one value should be printed on one line (each value separated by ', ') or on several lines (each value on its own line). In the later case, the patch also introduces smart word w

[Freeipa-devel] [PATCH] Remove obsolete CRUD base classes as they aren't used anymore.

2009-09-08 Thread Pavel Zůna
This removes legacy CRUD base classes (Add, Get, Del, Mod, Find) that aren't (and won't be) used anymore. They have been replaced by Create, Retrieve, Update, Delete and Search (and their LDAP oriented extensions). Note: This patch doesn't remove unit tests associated with these classes, but I

[Freeipa-devel] [PATCHES] Improve ipalib.plugins.baseldap classes.

2009-09-08 Thread Pavel Zůna
- remove obsolete code related to PluginProxy - remove parent_key attribute, for the purpose of nested objects the parent's primary key is retrieved automatically - added support for auto-generating UUIDs - make use of the improved attribute printing in CLI !!! depends on previous patch !!! - ma

  1   2   >