In any case I have added your suggested changes to sql.conf in the mainline cvs. However can you please double check your suggestions and compare with Arthur's patch for postgres. His patch does some checks that yours does not.. If you come up with a revised patch feel free to mail it to me directly and I will commit it.
we use now the following query with freeradius 0.93, an Oracle database and Cisco NASes:
accounting_update_query = "UPDATE ${acct_table1} \
SET FramedIPAddress = NULLIF('%{Framed-IP-Address}', ''), \
AcctSessionTime = '%{Acct-Session-Time}', \
AcctInputOctets = '%{Acct-Input-Octets}', \
AcctOutputOctets = '%{Acct-Output-Octets}' \
WHERE \
AcctSessionId = '%{Acct-Session-Id}' AND \
UserName = '%{SQL-User-Name}' AND \
NASIPAddress= '%{NAS-IP-Address}' AND \
AcctStopTime IS NULL"It works just fine since a few weeks.
Didi
-- --------------------------------- Didi Rieder [EMAIL PROTECTED] PGPKey ID: 3431D0B0 ---------------------------------
p7s00000.p7s
Description: S/MIME cryptographic signature
