> First, excuse me, but the link not worked properly. I > can't download the file.
The ftp only allowed active connections, this has been fixed, passive connection should now work. > Second, I appreciate your try to create sql version of > ippool module (btw, why > it not process accounting on/off packets - recieving this > packet RADIUS server > should mark all IPs as deallocated?). It does process accounting packets, stop record: unallocate the IP address, checkpoint record: increases lease of current IP address by 'lease-duration'. > Why not using rlm_sql? I think any tries to implement > rlm_sql module/driver > specially for this module will be ugly. If it not have > transactions implemented, > then we have reason to implement it rlm_sql in the > interface calls. We tried using the rlm_sql module the same way the sqlcounter module does but because we used transactions it did not work. Yes maybe the rlm_sql module sould support transactions, but it currently does not. If you are going to start rewriting rlm_sql it might be a good idea to make it a database access module only, and therefore remove the sql accounting, authenication etc and put them in separate modules. > Anyway, if it's not currently possible we can create > rlm_sqlippool without transactions. With the design we chose, we needed transactions. We wrote this module because we had a need for it. We are only releasing it because others might want to use it. > I have my own idea about DB's scheema for this module. We based ours from the current ippool module. Regards Allister Maguire - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
