<div>Boina,<br><br>That works fine for me (patching against a clean 1.1.2 tree) I've only tested == and := operators but they seem fine.<br><br>Only
one point to note; if you do not include an element in the hash with
the same name as the attribute (
e.g. due to a typo or just a misconfiguration), the server hangs
completely the first time something gets passed through the perl module
and needs a kill -9 to stop it. I know you can't protect people against
their own configuration errors, but perhaps it would be nicer to log an
error (or just ignore the attribute).
<br><br>Anyway, thanks very much for the patch!<br><br>Thanks,<br></div><span class="sg"><br></span><br><br><div><span class="gmail_quote">On 15/08/06, <b class="gmail_sendername">Boian Jordanov</b> <<a href="mailto:bjordanov@orbitel.bg">
bjordanov@orbitel.bg</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">On Monday 14 August 2006 21:27, Alex French wrote:
<br>> Boian,<br>><br>> Thanks, if you have a patch that actually implements the hash for the<br>> operator etc, that would be great (in fact, why not just submit it as a<br>> feature). If it's just to change the operator hardcoded in rlm_perl.c,
<br>> that's fine, I have that recompiled and installed at the moment,<br><br>Yep, i have the patch that implements the operator with hash ref.<br>Test it and if you like it i will submit it in CVS HEAD.<br><br>For example to change Operator for Framed-MTU
<br><br>$hash{'Framed-MTU'} = "100";<br>$hash{'Operator'} = "==";<br>$RAD_REPLY{'Framed-MTU'} = \%hash;<br><br><br>--<br>Best Regards,<br>Boian Jordanov<br>SNE<br>Orbitel - Next Generation Telecom<br>tel. +359 2 4004 723
<br>tel. +359 2 4004 002<br><br><br>-<br>List info/subscribe/unsubscribe? See <a href="http://www.freeradius.org/list/users.html">http://www.freeradius.org/list/users.html</a><br><br><br></blockquote></div><br>