> -----Message d'origine-----
> De : ramoore [mailto:[EMAIL PROTECTED]]
> Objet : Re: Multiple reply attributes of the same type
> 
> To set more than one value for a single reply attribute, you 
> have to use 
> the += operator rather than the default == operator.  With 
> the default SQL 
> schema this is not possible.  You'll have to update your SQL 
> schema and 
> queries to include the "op" field.
> 
> The list archives includes some discussion of doing this.  
> Look for one of 
> my old messages with the subject "RE: Simultaneous-Use in SQL 
> table? works & minor patch"

  Thanks for the pointer. I had already seen these messages without 
paying much attention.

  I have switched to v0.4, modified my schema and queries, added the 
"+=" op on appropiate rows, but it didn't change anything...

  Even if your advice hasn't resolved my problem, it led me to the 
source code part that is responsible for that behaviour ! :)
  It seems that before checking the "op" column, FreeRADIUS checks 
whether the attribute has already been set before. It ignores the newest 
pair if it's a group-defined AV pair, on any attribute set before 
(group- or user-defined). So, group attributes don't override user 
specific attributes (the latter are set before the former), but that 
algorithm doesn't allow multiple group attributes...

  (see function sql_userparse() in ./src/modules/rlm_sql/sql.c)

   When I comment out the test, I have the behaviour I want... So I'm 
going to try and find a more elegant way to do it than commenting out...
   Shouldn't I switch to the developpement list to go further on that 
topic ?

   Anyway thanks Randy.

Thomas FAVIER
Accelance
Tel: +33 (0)4 37 43 12 22
Fax: +33 (0)4 37 43 12 20
www.accelance.fr


- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to