-
Notifications
You must be signed in to change notification settings - Fork 511
cisco_umbrella: add support for log schema version v13 #15791
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
cisco_umbrella: add support for log schema version v13 #15791
Conversation
d9c73be to
e6778c0
Compare
🚀 Benchmarks reportTo see the full report comment with |
| }, | ||
| "network": { | ||
| "application": "Dropbox", | ||
| "application": "dropbox", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normalizing network.application field to lowercase as per ECS recommendation.
|
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you provide links to the specific log examples that you used?
packages/cisco_umbrella/data_stream/log/_dev/test/pipeline/test-umbrella-dnslogs.log
Show resolved
Hide resolved
packages/cisco_umbrella/data_stream/log/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
...co_umbrella/data_stream/log/_dev/test/pipeline/test-umbrella-intrusionlogs.log-expected.json
Outdated
Show resolved
Hide resolved
💚 Build Succeeded
History
|
|
Package cisco_umbrella - 1.33.0 containing this change is available at https://epr.elastic.co/package/cisco_umbrella/1.33.0/ |
Proposed commit message
Note
Documentation for log schema version v13: https://docs.umbrella.com/umbrella-user-guide/docs/log-format-and-versioning
The DNS logs appear to be ingesting as expected. We couldn't test the other logs due to the unavailability of logs in the instance. I'll raise an issue to expand tests coverage for the v13 logs to keep a note of this.
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Pipeline test:
Related issues
Screenshots