[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [PATCH] KDM Photo, LDAP patch
From:       Robin Rosenberg <robin.rosenberg () dewire ! com>
Date:       2005-08-22 20:21:32
Message-ID: 200508222221.33035.robin.rosenberg () dewire ! com
[Download RAW message or body]

måndagen den 22 augusti 2005 07.18 skrev Oswald Buddenhagen:
> On Mon, Aug 22, 2005 at 01:50:29AM +0200, Robin Rosenberg wrote:
> > Here is a patch for retrieving the face i KDM from LDAP.
>
> 1) using the jpeg should be only a fallback, as scaling down is sloooow
> (how many users does your installation have?) and sometimes makes the
> image hardly recognizable.
Do large sites use/need the photo feature? I have a dozen users. To me it's 
more lika a cute featurefor small installations. Since I use KDE in a small 
installation and I had the photos in LDAP already, I decided it was a little 
cool to get them from LDAP directly, getting a reason to dig into KDE at the 
same time. However if a large installation would like the photo they may well 
want the LDAP feature.. hmm.

Network traffic is what I would expect to be a larger source of slowness in a 
large installation as I see it. Also, scaling photos could be an option, as 
would be the size etc. etc. 

As for hard-to-see images,I'm not sure what to do about that. I added scaling
since varying images sizes looked horrible.

> 2) one could/should drag things further and get the realname, etc. from
> ldap, too. also, i'm not sure whether hardcoding the scheme without even
> an option to turn it off is really a good idea.
KDM already uses the Gecos field, which usually often contains the Full Name. 
An option to turn if off; Ok. As for the schema the jpegPhoto is a standard. 
Caching is a possibility (where, how often..)

> 3) i'm wondering, whether there really is no api for reading the ldap
> config? somehow it looks unclean the current way ...
It is indeed very unlean, but I know of no library so please tell me. The 
patch also does not handle the URL syntax in ldap.conf yet and is also bound 
to the format nss_ldap uses.  How many version of doing this exists?

pam + local images
pam + ldap
pam + some other LDAP than the one PAM uses
KDM plugins somehow, probably not using LDAP (no idea how that
works but there seems to be something there). 

Maybe that shouldn't go to LDAP right away. Maybe the LDAP config
looks different on other systems, so the patches should probably only
be enabled for known-to-work configurations (i.e. nss_ldap+linux+pam)?

-- robin
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

[prev in list] [next in list] [prev in thread] [next in thread]