Hello Joost -

On Fri, 05 Nov 1999, Joost Stegeman wrote:
> Hugh,
> 
> I'm not sure your suggestion to define multiple CheckAttr and ReplyAttr fields
> in the LDAP config works in Radiator 2.14.1. It doesn't work in 2.11 for sure.
> Maybe you mean that multiple atribute values in the LDAP directory are
> concatenated with commas?
> 
> from AuthLDAP.pm, line 274:
> 
>             elsif ($attr eq lc $self->{CheckAttr})
>             {
>                 # This is the attribute with check items in it
>                 $user->get_check->parse(join ',', @vals);
>             }
>             elsif ($attr eq lc $self->{ReplyAttr})
>             {
>                 # This is the attribute with reply items in it
>                 $user->get_reply->parse(join ',', @vals);
>             }
> 
> Here I see the $attr's from the LDAP query being checked against a single value
>  in the configuration tree. This, and my tests with 2.11 lead me to believe
> your suggestion is not correct. I really like the idea from Steven Ames, who
> build som patches which make the configuration much alike the SQL
> configuration.
> 

Yes, so does Mike - he is rolling it into the next release.

> Any other thoughts, I am wrong?

No - *you* aren't wrong - I am! That's what I get for reading the manual and
not the source!! No, I read the manual entry to mean what I thought it meant,
not what it really meant.

** SIGH **

thanks Joost!!

Hugh


--
Radiator: the most portable, flexible and configurable RADIUS server
anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald,
Platypus, Freeside, TACACS+, PAM, external, etc etc on Unix, Win95/8,
NT, Rhapsody

===
Archive at http://www.thesite.com.au/~radiator/
To unsubscribe, email '[EMAIL PROTECTED]' with
'unsubscribe radiator' in the body of the message.

Reply via email to