On May 12, 2015, at 12:23 PM, Paul Boehmer <pboehmer@seidata.com> wrote:
...And the first post is an epic fail.
Seems the code below works as advertised. As a part of troubleshooting, I had erroneously put a stray "startrad" in my sites-enabled/default without a test condition, thus executing the script regardless of Acct-Status-Type.
I was wondering how we overlooked such a major defect.
That being said, am I on the right track as far as best practices?
Well other than using exec at all... sure. It really is horrifically slow, and there's not much we can do about that. For 3.0.8 you might find the back ticks syntax is more convenient. update { Tmp-String-0 := `/my/shell/script --arg '%{arg}'` } Behaviour changes based on the operator and LHS expression. := will copy stdout to the attribute Tmp-String-0. <list>: += will look for attributes in the environment of the shell script and add them to the specified list. The reason why the exec module is sticking around, is because it allows you toggle more stuff, like not waiting for the script to complete before continuing. -Arran