[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Freeipa-devel] Turn off "all" for searches



On 03/29/2011 10:16 AM, Adam Young wrote:
When we do search or details, we blindly add {all: false} to the JSON params. This is a mistake: all triggers far more server side work than we want to do. In the case of SUDO, it is causing enough problems that JrAquino's server errors out. Aside from correctness issues, we want to avoid the additional overhead.

Gonna recommend we change:

/usr/share/ipa/ui/search.js: line 272 to look like this:

 IPA.cmd(
          'find', [filter], {all: false}, on_success, on_error,
            that.entity_name);


That will break the user search, but should leave the rest of the site untouched (need to confirm). We can over ride search for users to use {all: true}

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel redhat com
https://www.redhat.com/mailman/listinfo/freeipa-devel
Correction: we blindly add {all:True}...recommending we change to all : false for all searches but user.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]