Skip to content
This repository has been archived by the owner on Mar 22, 2024. It is now read-only.

Implement byte and flow time conversions for ASA - Address #22 #23

Conversation

sempervictus
Copy link

Convert fields relevant to flow time tracking and byte counts from
Cisco ASA 9.2 v9 to fields understood by the Kibana queries.

Testing:
Initial test runs on several ASAs, needs wider testing across
multiple versions, ASA/ASDM configs, etc.

Todo:
There's no pkt field in the ASA outputs, determine if metric
can be exposed and collected. Write conversions if data exists.

Convert fields relevant to flow time tracking and byte counts from
Cisco ASA 9.2 v9 to fields understood by the Kibana queries.

Testing:
  Initial test runs on several ASAs, needs wider testing across
multiple versions, ASA/ASDM configs, etc.

Todo:
  There's no *pkt* field in the ASA outputs, determine if metric
can be exposed and collected. Write conversions if data exists.
@sempervictus
Copy link
Author

This should address the critical issues in #22, but packet counters are still missing.

ASA netflow icludes cardinality for the bytes and packets seen in
flows, implement summations to generate the fields used in EF
Kibana queries.
@sempervictus
Copy link
Author

@robcowart: we've got this running in prod for a few days now, think it might be worthwhile to commit to master if you've no issues with the code since its one of the most widely used NF sources out there (revision tags work too if theres some concern about preserving prior behavior, though these changes havent affected our other collectors)

@robcowart
Copy link
Owner

The downside of simply adding these values together is that you loose the information related to how much traffic went in each direction. My tendency at the moment is to split such bidirectional flows into two unidirectional flows. I have in the past done this with the flow events from Packetbeat, and I find it to be a better path as the resulting data is more consistent across all sources. What are your thoughts?

@sempervictus
Copy link
Author

sempervictus commented Dec 3, 2017 via email

@DanSheps
Copy link

This is honestly working really well for me at the moment, not sure if it will be included in the current iteration, but it is working great.

@robcowart
Copy link
Owner

Compatible changes have been made in commit c6d01da

@robcowart robcowart closed this Dec 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants