<div dir="ltr">Hi Fajar<br><br>Yes we REALLY tested it :)<br><br>"A quick glance says the OK block will always<br>
be used" is what we wanted! Because this block is critical.<br><br>What was happening earlier was since we didnt have any return code for the sql_update_**** ,any failure at that level was making the accounting module return a false and thus the session disconnection. In our case the last two statements are not so critical that the sessions be dropped.<br>
So now with this approach - mission accomplished!<br><br>Regards<br>Shweta<br><br><br><div class="gmail_quote">On Tue, Dec 13, 2011 at 10:59 AM, Fajar A. Nugraha <span dir="ltr"><<a href="mailto:list@fajar.net">list@fajar.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Dec 13, 2011 at 1:54 PM, Shweta Khadse <<a href="mailto:shwetgk@gmail.com">shwetgk@gmail.com</a>> wrote:<br>

> Thankyou Alan!<br>
><br>
> You gave us an idea about how we can set "ok" to return a default ok for the<br>
> section.<br>
><br>
> We tested with our second approach as below and thrilled that it works/<br>
> This ensures radius returns ok even if the sql_update_**** modules fail.<br>
><br>
>             redundant{<br>
>                      ok<br>
>                  }<br>
>             sql_update_xxxx {<br>
>                         FAIL=1<br>
>                 }<br>
>              sql_update_yyyy {<br>
>                      FAIL=1<br>
>                 }<br>
><br>
<br>
</div>DId you REALLY test that? A quick glance says the OK block will always<br>
be used, thus making sql_update* pretty much useless. Why not use<br>
Alan's example?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Fajar<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
-<br>
List info/subscribe/unsubscribe? See <a href="http://www.freeradius.org/list/users.html" target="_blank">http://www.freeradius.org/list/users.html</a><br>
</div></div></blockquote></div><br></div>