-
Notifications
You must be signed in to change notification settings - Fork 474
checkpoint: test and support R81 logs #5440
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🌐 Coverage report
|
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
This adds minimal representors of a corpus of ~3300 events from an R81.20 appliance. Some additional field definitions are added and missing type conversions for icmp_code and icmp_type are added. In addition to this there was one case of a log line containing more than one time field. This line does not appear to be corrupted and has an outzonlags which may explain the presence of two times. Additional processing is added to support multiple times. Finally error handling in script and kv processors is enhanced.
Thanks for working through this one @efd6. We'll also need to adjust the compatibility section of our docs to indicate support for R81. We can drop all mention of R77.30 as it's end of life. |
R81 is listed there, I'll remove R77. |
packages/checkpoint/data_stream/firewall/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
packages/checkpoint/data_stream/firewall/elasticsearch/ingest_pipeline/default.yml
Show resolved
Hide resolved
packages/checkpoint/data_stream/firewall/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
@efd6 I only see a reference to R80 - "This module has been tested against Check Point Log Exporter on R80.X but should also work with R77.30." |
Package checkpoint - 1.16.0 containing this change is available at https://epr.elastic.co/search?package=checkpoint |
3 similar comments
Package checkpoint - 1.16.0 containing this change is available at https://epr.elastic.co/search?package=checkpoint |
Package checkpoint - 1.16.0 containing this change is available at https://epr.elastic.co/search?package=checkpoint |
Package checkpoint - 1.16.0 containing this change is available at https://epr.elastic.co/search?package=checkpoint |
This adds minimal representors of a corpus of ~3300 events from an R81.20 appliance. Some additional field definitions are added and missing type conversions for icmp_code and icmp_type are added. In addition to this there was one case of a log line containing more than one time field. This line does not appear to be corrupted and has an outzonlags which may explain the presence of two times. Additional processing is added to support multiple times. Finally error handling in script and kv processors is enhanced.
This adds minimal representors of a corpus of ~3300 events from an R81.20 appliance. Some additional field definitions are added and missing type conversions for icmp_code and icmp_type are added. In addition to this there was one case of a log line containing more than one time field. This line does not appear to be corrupted and has an outzonlags which may explain the presence of two times. Additional processing is added to support multiple times. Finally error handling in script and kv processors is enhanced.
What does this PR do?
This adds minimal representors of a corpus of ~3300 events from an R81.20 appliance. Some additional field definitions are added and missing type conversions for icmp_code and icmp_type are added.
In addition to this there was one case of a log line containing more than one time field. This line does not appear to be corrupted and has an outzonlags which may explain the presence of two times. Additional processing is added to support multiple times.
Finally error handling in script and kv processors is enhanced.
Corpus is available on request.
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots