Increase the ZT event coverage for Cloudflare Logpush #6132
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
chemamartinez added enhancement 
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsExpand to view the GitHub comments
To re-run your PR in the CI, just comment with:
|
🌐 Coverage report
|
chemamartinez force-pushed the 5867-cloudflare-new-datastreams branch 3 times, most recently from 773a94a to 7022ff9 Compare 
chemamartinez force-pushed the 5867-cloudflare-new-datastreams branch 6 times, most recently from 8d1d28d to df3a937 Compare 
chemamartinez force-pushed the 5867-cloudflare-new-datastreams branch from 9ca1a87 to ff9abcc Compare 
P1llus reviewed May 25, 2023
Member
P1llus left a comment
P1llus left a comment There was a problem hiding this comment.
We still have a few datastreams left, but this is a partial review.
...ages/cloudflare_logpush/data_stream/access_request/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
...ages/cloudflare_logpush/data_stream/access_request/elasticsearch/ingest_pipeline/default.yml Show resolved Hide resolved
...ages/cloudflare_logpush/data_stream/access_request/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
...ages/cloudflare_logpush/data_stream/access_request/elasticsearch/ingest_pipeline/default.yml Show resolved Hide resolved
packages/cloudflare_logpush/data_stream/casb/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
packages/cloudflare_logpush/data_stream/casb/elasticsearch/ingest_pipeline/default.yml Show resolved Hide resolved
...ages/cloudflare_logpush/data_stream/device_posture/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
packages/cloudflare_logpush/data_stream/gateway_dns/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
packages/cloudflare_logpush/data_stream/gateway_dns/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
chemamartinez changed the title P1llus approved these changes May 29, 2023
...s/cloudflare_logpush/data_stream/network_analytics/elasticsearch/ingest_pipeline/default.yml Outdated Show resolved Hide resolved
Member
| We also want to add some more dashboards before merging, will do a second review for them once they are available. |
| Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
💚 Build Succeeded
History |
| Package cloudflare_logpush - 1.5.0 containing this change is available at https://epr.elastic.co/search?package=cloudflare_logpush |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Add new data streams to the Cloudflare Logpush integration to cover all the possible event types provided by Cloudflare.
Here is the list of new supported events:
For now, the list above covers the Zero Trust events.
Checklist
changelog.ymlfile.How to test this PR locally
Related issues
Screenshots
Integration main page
Zero Trust Overview
Access Requests
Audit
CASB Findings
Device Posture Results
Gateway DNS
Gateway HTTP
Gateway Network
Network Sessions