Skip to content

[ASIM] WebSessionNative - Fix mismatched field types - #14942

Open
Remco Hofman (SpeedyFireCyclone) wants to merge 4 commits into
Azure:masterfrom
SpeedyFireCyclone:ASIM/WebSessionNative
Open

[ASIM] WebSessionNative - Fix mismatched field types#14942
Remco Hofman (SpeedyFireCyclone) wants to merge 4 commits into
Azure:masterfrom
SpeedyFireCyclone:ASIM/WebSessionNative

Conversation

@SpeedyFireCyclone

Copy link
Copy Markdown
Contributor

Change(s):

  • Cast fields to longs for schema compliance

Reason for Change(s):

  • The current table defines several fields as real where the schema expects longs.

Version Updated:

  • Yes

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • Yes

@v-atulyadav
v-atulyadav requested a lite review from Copilot August 19, 2026 11:08
@v-atulyadav v-atulyadav added ASIM SafeToRun This is used only for ASim parsers Fork PR Pipeline run. labels Aug 19, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the ASIM WebSession Native parsers to align field data types with the schema by casting several size/packet counters to long.

Changes:

  • Bumped parser and schema versions and updated LastUpdated metadata.
  • Extended the schema-fix step to cast byte/packet-related fields to long in both YAML parser definitions.
  • Mirrored the same KQL changes in the corresponding ARM template JSON query payloads.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
Parsers/ASimWebSession/Parsers/vimWebSessionNative.yaml Updates metadata and casts multiple byte/packet fields to long for schema compliance.
Parsers/ASimWebSession/Parsers/ASimWebSessionNative.yaml Updates metadata and casts multiple byte/packet fields to long for schema compliance.
Parsers/ASimWebSession/ARM/vimWebSessionNative/vimWebSessionNative.json Updates embedded KQL query to cast the same fields to long.
Parsers/ASimWebSession/ARM/ASimWebSessionNative/ASimWebSessionNative.json Updates embedded KQL query to cast the same fields to long.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@v-atulyadav v-atulyadav added SafeToRun This is used only for ASim parsers Fork PR Pipeline run. and removed SafeToRun This is used only for ASim parsers Fork PR Pipeline run. labels Aug 19, 2026
@v-atulyadav v-atulyadav added SafeToRun This is used only for ASim parsers Fork PR Pipeline run. and removed SafeToRun This is used only for ASim parsers Fork PR Pipeline run. labels Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ASIM SafeToRun This is used only for ASim parsers Fork PR Pipeline run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants