issue with dialup.conf

[Tyller D]

Sorry, the values I gave before were on a test db after I tested out the
stop queries in the dialup.conf fille manually, so the values were
incorrect. Here is the correct info

mysql> select sum(acctinputoctets + acctoutputoctets ) from radacct where
username='scotty';
+------------------------------------------+
| sum(acctinputoctets + acctoutputoctets ) |
+------------------------------------------+
|                               1840263628 |
+------------------------------------------+
1 row in set (0.00 sec)



mysql> select * from radcheck where username='scotty';
+------+----------+--------------------+----+------------+
| id   | username | attribute          | op | value      |
+------+----------+--------------------+----+------------+
| 5192 | Scotty   | Auth-Type          | := | Perl       |
| 5191 | Scotty   | databank           | := | -302340151 |
| 5190 | Scotty   | Cleartext-Password | := | DALNIC     |
+------+----------+--------------------+----+------------+
3 rows in set (0.00 sec)


original databank value = 262141750

as you can see the amount of traffic used, far exceeds what was "assigned"
originally. I understand the negative value, Its something I can deal with
to be a certain amount off but this is really bad. When I ran the queries
manually I noticed that without quoting the radcheck.value in the
accounting_stop_query_alt querie, it would not update the radcheck.value for
databank. but once it is quoted seems, to work. I am going to keep an eye on
it to see if it resolves itself.

Im not too clued up on freeradius, so just to clear my mind to only way for
a session to end is on a stop request(correct?) and even if interim updates
are send, on the stop request it will show all data used in the session (not
just just since the last update)



On Mon, Feb 7, 2011 at 11:13 AM, Brian Candler <B.Candler at pobox.com> wrote:

> On Mon, Feb 07, 2011 at 08:48:27AM +0200, Tyller D wrote:
> >    if ( $device =~ /^nomadix/i ) {
> >    if ($DATABANK != '') {
> >                    if ( $DATABANK le 0 ) {
> >                                               $RAD_REPLY{'Reply-Message'}
> >    = "You have no more Data Left";
> >                                            return RLM_MODULE_REJECT;
> >                    }else {
> >                                            return RLM_MODULE_REJECT;
> >
> >    $RAD_REPLY{'Nomadix-MaxBytesDown'} = "$DATABANK";
> >                    }
>
> That logic returns RLM_MODULE_REJECT in both branches of the if statement,
> so I imagine you haven't copy-pasted it correctly. But I think I see what
> you're getting at.
>

You are correct, it was a copy error

>
> >    exctract from radcheck:
> >    +------+----------+--------------------+----+----------+
> >    | id   | username | attribute          | op | value    |
> >    +------+----------+--------------------+----+----------+
> >    | 3069 | Joe  | databank           | := | 52428800 |
> >    | 3068 | Joe  | Cleartext-Password | := | Joe123   |
> >    | 3070 | Joe  | Auth-Type          | := | Perl     |
> >    +------+----------+--------------------+----+----------+
>
> OK, so you're using radcheck as a convenient place to store the user's
> remaining quota. I guess that should work.
>
> >    but it not always exectuting stop request correctly because
> >    mysql> select sum(acctinputoctets + acctoutputoctets) from radacct
> >    where username='scotty';
> >    +-----------------------------------------+
> >    | sum(acctinputoctets + acctoutputoctets) |
> >    +-----------------------------------------+
> >    |                              1840263628 |
> >    +-----------------------------------------+
> >    mysql> select value from radcheck where username='scotty' and
> >    attribute='databank';
> >    +------------+
> >    | value      |
> >    +------------+
> >    | -302340151 |
> >    +------------+
>
> Well, those are two different things. The databank value will be whatever
> original value you put in databank (which you haven't told us), minus the
> bytes in and bytes out. So that would be correct if the initial value was
> 1537923477
>
> I have to say this looks like a pretty fragile way of doing accounting,
> because you are relying 100% on Stop packets. This means:
>
> (1) A lost stop packet will not update databank
>
> (2) If a user stays online solidly for months, they won't get their quota
> updated in the database
>
> Doing accounting based on interim-update packets is more robust. However,
> they are cumulative (each interim-update packet shows the *total* used so
> far for that session, just as if it were a Stop packet), so you can't just
> subtract them from an accumulator.  You need to add together the last
> values
> seen for each session.
> -
> List info/subscribe/unsubscribe? See
> http://www.freeradius.org/list/users.html
>
This is something that I have been wanting to look at but I need to get this
atleast to work as it should and I can update the process when I have some
time. Thanks for the advice.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freeradius.org/pipermail/freeradius-users/attachments/20110207/a26a35c2/attachment.html>