Cloudflare One Web Session Parser - #14982
Open
EndrigaInno wants to merge 24 commits into
Open
Conversation
EndrigaInno
marked this pull request as ready for review
August 26, 2026 12:56
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds Cloudflare One Web Session ASIM parsers (both filtering vim* and normalization ASim*) and wires them into the top-level WebSession aggregators and ARM deployment artifacts.
Changes:
- Introduced
vimWebSessionCloudflareOne(filtering) andASimWebSessionCloudflareOne(normalization) parsers forCloudflareV2_CL. - Updated
imWebSessionandASimWebSessionaggregators (and ARM templates) to include the new Cloudflare One parsers. - Added schema/data validation CSV outputs and changelog entries for the new parsers and version bumps.
Reviewed changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| Parsers/ASimWebSession/Tests/Cloudflare_CloudflareOne_vimWebSession_SchemaTest.csv | Adds expected schema validation output for the filtering parser. |
| Parsers/ASimWebSession/Tests/Cloudflare_CloudflareOne_vimWebSession_DataTest.csv | Adds expected data-quality validation output for the filtering parser. |
| Parsers/ASimWebSession/Tests/Cloudflare_CloudflareOne_ASimWebSession_SchemaTest.csv | Adds expected schema validation output for the normalization parser. |
| Parsers/ASimWebSession/Tests/Cloudflare_CloudflareOne_ASimWebSession_DataTest.csv | Adds expected data-quality validation output for the normalization parser. |
| Parsers/ASimWebSession/Parsers/vimWebSessionCloudflareOne.yaml | Implements the Cloudflare One WebSession filtering parser. |
| Parsers/ASimWebSession/Parsers/imWebSession.yaml | Registers vimWebSessionCloudflareOne in the top-level filtering parser. |
| Parsers/ASimWebSession/Parsers/ASimWebSessionCloudflareOne.yaml | Implements the Cloudflare One WebSession normalization parser. |
| Parsers/ASimWebSession/Parsers/ASimWebSession.yaml | Registers ASimWebSessionCloudflareOne in the top-level normalization parser. |
| Parsers/ASimWebSession/CHANGELOG/vimWebSessionCloudflareOne.md | Adds changelog for the new filtering parser. |
| Parsers/ASimWebSession/CHANGELOG/imWebSession.md | Bumps imWebSession changelog version for Cloudflare One addition. |
| Parsers/ASimWebSession/CHANGELOG/ASimWebSessionCloudflareOne.md | Adds changelog for the new normalization parser. |
| Parsers/ASimWebSession/CHANGELOG/ASimWebSession.md | Bumps ASimWebSession changelog version for Cloudflare One addition. |
| Parsers/ASimWebSession/ARM/vimWebSessionCloudflareOne/vimWebSessionCloudflareOne.json | Adds ARM template to deploy the new filtering function. |
| Parsers/ASimWebSession/ARM/vimWebSessionCloudflareOne/README.md | Adds deployment README for the new filtering function. |
| Parsers/ASimWebSession/ARM/imWebSession/imWebSession.json | Updates ARM template for imWebSession to include Cloudflare One. |
| Parsers/ASimWebSession/ARM/FullDeploymentWebSession.json | Links new ARM deployments for Cloudflare One parsers in full deployment template. |
| Parsers/ASimWebSession/ARM/ASimWebSessionCloudflareOne/README.md | Adds deployment README for the new normalization function. |
| Parsers/ASimWebSession/ARM/ASimWebSessionCloudflareOne/ASimWebSessionCloudflareOne.json | Adds ARM template to deploy the new normalization function. |
| Parsers/ASimWebSession/ARM/ASimWebSession/ASimWebSession.json | Updates ARM template for ASimWebSession to include Cloudflare One. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Collaborator
|
Hi EndrigaInno, |
EndrigaInno
marked this pull request as draft
August 28, 2026 07:53
EndrigaInno
marked this pull request as ready for review
August 28, 2026 08:07
EndrigaInno
marked this pull request as draft
August 28, 2026 08:09
EndrigaInno
marked this pull request as ready for review
August 28, 2026 09:13
EndrigaInno
marked this pull request as draft
September 1, 2026 06:06
EndrigaInno
marked this pull request as ready for review
September 1, 2026 06:08
EndrigaInno
marked this pull request as draft
September 1, 2026 06:44
EndrigaInno
marked this pull request as ready for review
September 1, 2026 06:45
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.
Required items, please complete
Change(s):
Reason for Change(s):
Version Updated:
Top level Alert Event ASIM parsers updated to 0.1.4
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: