Repository navigation
cloudflare_logpush: expand set of supported fields and add data streams - #12782
Merged
Merged
Conversation
efd6
added
enhancement
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
February 13, 2025 22:31
e3ca826 to
7b5c0ea
Compare
Contributor
🚀 Benchmarks reportTo see the full report comment with |
efd6
force-pushed
the
9809-cloudflare_logpush
branch
3 times, most recently
from
March 5, 2025 23:36
b4e8594 to
3700282
Compare
efd6
changed the title
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
March 6, 2025 00:26
3700282 to
40b0641
Compare
efd6
marked this pull request as ready for review
March 6, 2025 01:25
|
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
March 6, 2025 10:06
40b0641 to
a92b51b
Compare
chemamartinez
left a comment
chemamartinez
left a comment
Contributor
There was a problem hiding this comment.
I think we still need to add some fields for the gateway_http data stream, the ones that are not linked in the table of the original issue.
Notice that the link to the documentation for the gateway_http data stream is pointing to the gateway_network dataset so that should be why you couldn't find them.
andrewkroh
added
the
Integration:entityanalytics_ad
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
March 13, 2025 22:31
7a1c24e to
546acaf
Compare
Contributor
Author
|
@chemamartinez I marked the missing (now corrected) fields in the table in the issue with a |
andrewkroh
removed
the
Integration:entityanalytics_ad
chemamartinez
approved these changes
Mar 14, 2025
chemamartinez
left a comment
chemamartinez
left a comment
Contributor
There was a problem hiding this comment.
One comment left. Apart from that, it looks good!
kcreddy
reviewed
Mar 14, 2025
61 of 66 tasks
kcreddy
approved these changes
Mar 17, 2025
chemamartinez
approved these changes
Mar 17, 2025
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
March 26, 2025 01:12
7928300 to
cce278d
Compare
andrewkroh
added
the
Integration:entityanalytics_ad
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
March 30, 2025 21:14
cce278d to
35543da
Compare
andrewkroh
removed
the
Integration:entityanalytics_ad
New data streams: * page_shield_events * dlp_forensic_copies * email_security_alerts Tests for new data streams were hand crafted from the API documentation.
efd6
force-pushed
the
9809-cloudflare_logpush
branch
from
April 2, 2025 08:46
35543da to
9cf3521
Compare
|
💚 Build Succeeded
History
cc @efd6 |
Contributor
|
Package cloudflare_logpush - 1.36.0 containing this change is available at https://epr.elastic.co/package/cloudflare_logpush/1.36.0/ |
flexitrev
pushed a commit
that referenced
this pull request
Apr 3, 2025
…ms (#12782) New data streams: * page_shield_events * dlp_forensic_copies * email_security_alerts Tests for new data streams were hand crafted from the API documentation.
4 of 5 tasks
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Proposed commit message
See title.
Warning
No test cases are available for the new fields and tests for the new data streams are constructed from documentation without any ground truth.
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots