CachedLDAPConnection: Don't cache fields we don't use
authorW. Trevor King <wking@tremily.us>
Sun, 20 Jan 2013 16:07:50 +0000 (11:07 -0500)
committerW. Trevor King <wking@tremily.us>
Sun, 20 Jan 2013 18:05:19 +0000 (13:05 -0500)
commit53631c2e4e1f7e93ce2da03185678577ab2c62ab
tree76ee9350230f011d095d931ba24ae5db6c57c2d6
parent74ccf3e6fa1222621fc2220a4c50bc67ec7d72f0
CachedLDAPConnection: Don't cache fields we don't use

I had previously said:
> It would be nice to automatically detect and drop any non text
> fields, but that's probably not worth the trouble either...

Since the configuration ID is in the cached key, we already know
exactly which fields we need to save in the cache.  Only caching those
fields avoids binary fields and fields with large values (jpegPhoto is
both).  I made the cached fields configurable because the
CachedLDAPConnection instance doesn't know about the fields needed by
format_entry() and format_columns().  If you don't configure the
cached fields, a minimal default is setup in the main block.
mutt-ldap.py