Update Sentinel and Splunk analytics integrations - #33059
Open
ngayerie wants to merge 5 commits into
Open
Conversation
ngayerie
requested review from
a team,
rianvdm and
soheiokamoto
as code owners
August 27, 2026 09:35
Contributor
Review
👉 Fix in your agent 👈Fix the following review findings in PR #33059 (https://github.com/cloudflare/cloudflare-docs/pull/33059).
Before making changes, review each finding and present a brief summary table:
- For each finding, state whether you agree, disagree, or need clarification
- If you disagree (e.g. the fix requires disproportionate effort for minimal benefit,
or the finding is factually incorrect), explain why
- If you need clarification before deciding, ask those questions
- Then share your plan for which issues to tackle and in what order
After triaging, follow this order:
1. Post a comment on this PR for any findings you are skipping, with the finding ID and your reasoning.
2. Then commit the fixes for the legitimate findings.
The comment must come before the commit — the bot reads PR comments when a new
push triggers a review, so skip comments posted after the push will be missed.
---
## Code Review
### Warnings (3)
#### CR-3138e225a15c · Wrong Splunk CIM field name for referrer
- **File:** `src/content/docs/analytics/analytics-integrations/splunk.mdx` line 221
- **Issue:** The HTTP requests table maps `ClientRequestReferer` to `http_referer`, but the Splunk CIM Web data model normalizes this value as `http_referrer` (two r's).
- **Fix:** Change the Splunk CIM field from `http_referer` to `http_referrer` so the mapping aligns with the standard CIM Web field.
#### CR-1025419467c2 · Semantic mismatch for DNS query size mapping
- **File:** `src/content/docs/analytics/analytics-integrations/splunk.mdx` line 252
- **Issue:** The DNS logs table maps `QuerySize` to `query_count`. In the Cloudflare gateway_dns schema `QuerySize` is the DNS request size in bytes, while Splunk CIM DNS uses `query_count` to mean the number of queries in a transaction.
- **Fix:** Remove or remap `QuerySize` to a field that represents payload size, and avoid conflating byte size with query count.
#### CR-15e6a491a88d · Wrong Splunk CIM field name for session identifier
- **File:** `src/content/docs/analytics/analytics-integrations/splunk.mdx` line 306
- **Issue:** The Zero Trust Gateway Network table maps `SessionID` to `ssid`. In Splunk CIM Network Traffic, the session identifier field is `session_id`; `ssid` is the wireless service-set identifier field.
- **Fix:** Change the CIM mapping for `SessionID` from `ssid` to `session_id`.
### Suggestions (1)
#### CR-a506e8058cfe · Ambiguous DNS dataset heading
- **File:** `src/content/docs/analytics/analytics-integrations/splunk.mdx` line 244
- **Issue:** The 'DNS logs' mapping lists fields (`DstIP`, `DstPort`, `Protocol`, `QuerySize`, `QueryTypeName`, `RCode`, `SrcPort`) that match the account-scoped `gateway_dns` dataset instead of the zone-scoped `dns_logs` dataset documented elsewhere, which lacks those fields and uses `ResponseCode` rather than `RCode`.
- **Fix:** Rename the section to 'Gateway DNS' or otherwise disambiguate which Cloudflare Logpush dataset the DNS mapping applies to, and verify the field list matches that dataset.
---
## Style Guide Review
### Warnings (1)
#### SG-9389419bc184 · Avoid directional words
- **File:** `src/content/docs/analytics/analytics-integrations/sentinel.mdx` line 164
- **Issue:** Line contains "the tables below"
- **Fix:** Replace "the tables below" with a direct reference such as "the following tables" or "the tables in this section"
Code ReviewThis code review is in beta and may not always be helpful — use your judgment. Warnings (3)
Suggestions (1)
ConventionsNo convention issues found. Style Guide ReviewWarnings (1)
CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
Collaborator
Author
|
[AI] Triage of the code-review bot findings on
Applying fixes now. |
ngayerie
marked this pull request as ready for review
August 27, 2026 10:53
Collaborator
Author
|
[AI] Triage of the flue review on
Applying now. |
Expand the Sentinel page around the recommended CCF connector: detailed prerequisites (ADLS Gen2, RBAC roles, Event Grid resource provider, DCR/DCE, network access), refreshed Content hub and connector fields walkthrough, a Log Analytics verification query, and a troubleshooting entry for the CreateDataFlowResources ARM template error. Add a deprecation notice for the Azure Function-based solution. Extend the Splunk page with HEC Global Settings guidance for enabling SSL, dataset-to-sourcetype mapping for the Cloudflare App for Splunk, and a confirmation-events note when validating a new Logpush job. DEE-3779
Tie the Function-based connector deprecation notice to the public changelog entry and to Microsoft's Azure Monitor HTTP Data Collector API end-of-life on 2026-09-14. DEE-3779
- Reword Sentinel CCF connector identity references to point to the Cloudflare CCF connector application and its service principal (CR-e7252f2c3d02). - Drop directional 'below' references (SG-2d8ad00db9da, SG-6d5acda3ec8a). - Replace 'set to Enabled' / 'Only enable' toggle jargon with the style-guide 'turned on' / 'turn on' phrasing (SG-e13d4b76765b, SG-5a06a4ee4266). - Use 'Select' instead of 'Click' for the '+Add new' step (SG-c30ba167399b). DEE-3779
Move the exact 2026-09-14 end-of-support date out of the page body and admonition title into the linked changelog entry, per the style guide on time-sensitive dates outside the changelog (SG-e900072a2c83, SG-e053572c78cc). DEE-3779
Splunk: add a Splunk CIM field mappings section documenting the Cloudflare-to-CIM field mapping the Cloudflare App for Splunk applies per Logpush dataset — HTTP requests, CASB findings, DNS logs, Audit logs, Access requests, Zero Trust Gateway HTTP, Zero Trust Gateway Network. Sentinel: replace the parser/workbook/analytic-rules/hunting-queries field lists with the ASIM-normalized names the CCF connector actually emits (SrcIpAddr, HttpRequestMethod, HttpStatusCode, DvcAction, etc.), so KQL queries against the connector output match the documented names. DEE-3779
ngayerie
force-pushed
the
dee-3779-sentinel-splunk-updates
branch
from
August 28, 2026 09:25
aaeacf4 to
4b64248
Compare
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.
Summary
What: Expand
analytics/analytics-integrations/sentinelandanalytics/analytics-integrations/splunkwith the setup and operations details customers have been asking for.Sentinel:
InvalidTemplate: CreateDataFlowResourcesARM error.Splunk:
cloudflare:json,cloudflare:dns,cloudflare:audit,cloudflare:access).Why: Customers deploying these integrations were missing prerequisites, connector-field detail, and the troubleshooting path for the most common ARM template failure. No screenshots are added; steps link out to the relevant Microsoft and Splunk docs.
DEE-3779
Documentation checklist