> That's really what multiple module are for. Have the sqlcounter > module return a special return code when the user is over the limit, > and use module failover to switch on that return code, and run another > module.
Ok, any idea on the return code to be used? > > > - char *counter_name; /* Daily-Session-Time */ > > - char *check_name; /* Max-Daily-Session */ > > - char *key_name; /* User-Name */ > > + char *counter_name; /* Daily-Session-Time */ > > Why? Gratuitous whitespace changes don't belong in a patch. My bad, sorry > > > - if ((reply_item = pairfind(request->reply->vps, > > PW_SESSION_TIMEOUT)) != NULL) { > > + /* if ((reply_item = pairfind(request->reply->vps, > > PW_SESSION_TIMEOUT)) != NULL) { */ > > If you're changing code PLEASE do not leave the old code in > comments. It's confusing, and pointless. CVS will tell you what the > old code was. This practice is just unnecessary. This was for internal purposes only. I'll submit the patch in the form you want it to bugs.freeradius.org. I hope its compatible with the CVS code. J. - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html