On Tue, Nov 17, 2009 at 02:51:29PM +0100, Alan DeKok wrote:
John Morrissey wrote:
Any comments on this patch, Alan? Don't mean to be pushy, but it's been a couple weeks and wanted to make sure this didn't get dropped on the floor.
I'm still hesitant. Part of the reason is that there is *already* code to handle this in src/main/xlat.c. The reason you're proposing new code is that the xlat functionality isn't integrated into rlm_perl. It *is* integrated into the rest of the server...
It's not integrated into rlm_python, either. Or anything else that wants this value and isn't explicitly filtering values through the xlat code. john -- John Morrissey _o /\ ---- __o jwm@horde.net _-< \_ / \ ---- < \, www.horde.net/ __(_)/_(_)________/ \_______(_) /_(_)__