[Bro-Dev] [Bro-Commits] [git/bro] topic/policy-scripts-new: Extract packet data for the dpd.log (7c168e0)

Robin Sommer robin at icir.org
Thu Apr 21 10:29:48 PDT 2011


On Thu, Apr 21, 2011 at 12:34 -0400, you wrote:

> I initially thought of an option, but then decided it might be better
> as a separate script since the field would still be included if I left
> it as an option and I didn't want the field to show up at all.

Ok, I see that argument. That may actually also be a good rule of
thumb for @load vs redef-an-option: if not setting the option would
still be visible to the user in some non-pleasant form, a script may
indeed be the better way (you may have such rules in your head already
anyway; but I'm still learning them in the new model :-). 

Robin

-- 
Robin Sommer * Phone +1 (510) 722-6541 * robin at icir.org
ICSI/LBNL    * Fax   +1 (510) 666-2956 *   www.icir.org


More information about the bro-dev mailing list