[Bro-Dev] #960: topic/dnthayer/cleanup

Bro Tracker bro at tracker.bro.org
Fri Mar 8 14:12:04 PST 2013


#960: topic/dnthayer/cleanup
---------------------------+------------------------
 Reporter:  dnthayer       |      Owner:
     Type:  Merge Request  |     Status:  new
 Priority:  Low            |  Milestone:  Bro2.2
Component:  BroControl     |    Version:  git/master
 Keywords:                 |
---------------------------+------------------------
 This branch fixes various issues with broctl. Most of the fixes
 are related to the broctl plugin API.  Here are the one-line
 summaries of all commits in this branch:

 Check for plugins with same prefix
 Prevent capstats from being run with invalid args
 Fix plugin inconsistency for certain broctl commands
 Document the broctl user option KeepLogs
 Add a note in documentation about editing crontab
 Fix broctl plugin option names to be case-insensitive
 Remove reserved word "cluster" from node args
 Fix documentation of broctl commands
 Add calls to plugin cmd_restart_pre/post methods
 Fix instructions for adding plugin directories
 Fix the broctl check command to report results
 Fix handling of cmd_diag_pre for diag command
 Changed return value of plugin API "execute" method
 Add return value to some cmd_<cmd>_pre methods
 Add a check for state variables in broctl.cfg
 Changed "hosts" method to return list of hosts
 Call "done" method from plugin API
 Call hostStatusChanged with correct arg type
 Fix the parseNodes method in plugin API
 Fix the "error" method in broctl plugin API
 Fixed tab-completion of commands with node args
 Fix broctl plugin API documentation errors
 Fix typos in TestPlugin output messages
 Add cron "--no-watch" option to broctl "help" output
 Fix the "execute" method of the Plugin class
 Fix various bugs and remove some unused code

-- 
Ticket URL: <http://tracker.bro.org/bro/ticket/960>
Bro Tracker <http://tracker.bro.org/bro>
Bro Issue Tracker



More information about the bro-dev mailing list