Last call for 3.0.8

[Stefan Winter]

Hi,

> I had four such "empty" statements in one of my modules, but only got
> three warnings like:
> 
> My also empty "authorize_repply_query" went through the config check
> unnoticed.

Ah, it's actually something else. My config erroneously contains *two*
such config items, and one of them is empty:

        authorize_reply_query = \
"SELECT * FROM reply_aai_firstname WHERE username='%{SQL-User-Name}' \
 UNION ALL \
 SELECT * FROM reply_aai_lastname WHERE username='%{SQL-User-Name}' \
 UNION ALL \
 SELECT * FROM reply_aai_mail WHERE username='%{SQL-User-Name}' \
 UNION ALL \
 SELECT * FROM reply_aai_eduPersonAffiliation WHERE
username='%{SQL-User-Name}' \
 UNION ALL \
 SELECT * FROM reply_aai_fullname WHERE username='%{SQL-User-Name}' \
 UNION ALL \
 SELECT * FROM reply_aai_commonname WHERE username='%{SQL-User-Name}' \
 UNION ALL \
 SELECT id, username, 'RESTENA-AAI-Attribute',
CONCAT('urn:oid:1.3.6.1.4.1.23735.100.3=',id), '+=' FROM check_aai_ssha1
WHERE username='%{SQL-User-Name}' \
"

        authorize_reply_query = ""

I guess the server would spit out the warning if the latter entry was
the only one.

But this makes me think: would it be nice to introduce a config WARNING
level output that an entry is duplicate?

And I always thought later entries overwrite previous ones. But funny
enough, I do get all those attributes. Oh well.

Stefan


-- 
Stefan WINTER
Ingenieur de Recherche
Fondation RESTENA - Réseau Téléinformatique de l'Education Nationale et
de la Recherche
6, rue Richard Coudenhove-Kalergi
L-1359 Luxembourg

Tel: +352 424409 1
Fax: +352 422473

PGP key updated to 4096 Bit RSA - I will encrypt all mails if the
recipient's key is known to me

http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xC0DE6A358A39DC66
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freeradius.org/pipermail/freeradius-devel/attachments/20150401/1310e447/attachment.sig>