Skip to content

Add support for querying Bro conn.log via ELSA's cli.pl#1

Open
dougburks wants to merge 7 commits intoint13h:masterfrom
Security-Onion:master
Open

Add support for querying Bro conn.log via ELSA's cli.pl#1
dougburks wants to merge 7 commits intoint13h:masterfrom
Security-Onion:master

Conversation

@dougburks
Copy link
Collaborator

Hey Paul,

Per our email discussion, take a look at these changes and let me know what you think.

Thanks!

Doug

@int13h
Copy link
Owner

int13h commented Jul 4, 2013

On Thu, Jul 4, 2013 at 12:53 PM, dougburks notifications@github.com wrote:

Hey Paul,

Per our email discussion, take a look at these changes and let me know what
you think.

Thanks!

Doug

The only thing you should do is initialize $sensor just before line
38. If !elsa then $sensor will not exist and it is referenced in the
array below.

So just put: $sensor = ""; on line 37

Good work.


You can merge this Pull Request by running

git pull https://github.com/Security-Onion/capme master

Or view, comment on, or merge it at:

#1

Commit Summary

Add support for querying Bro conn.log via ELSA's cli.pl

File Changes

M .inc/callback.php (78)
M .js/capme.js (2)
M index.php (10)

Patch Links:

https://github.com/int13h/capme/pull/1.patch
https://github.com/int13h/capme/pull/1.diff

Paul Halliday
http://www.pintumbler.org/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants