Skip to content

Update ASIM Authentication Palo Alto PanOS and GlobalProtect parsers#14396

Merged
hunngu-ms merged 5 commits into
masterfrom
hunngu/UpdatePanOSParser
Jun 2, 2026
Merged

Update ASIM Authentication Palo Alto PanOS and GlobalProtect parsers#14396
hunngu-ms merged 5 commits into
masterfrom
hunngu/UpdatePanOSParser

Conversation

@hunngu-ms
Copy link
Copy Markdown
Contributor

@hunngu-ms hunngu-ms commented Jun 2, 2026

Assigning DvcIpAddr = DeviceAddress
Previously, DvcIpAddr is assigned to HostName, which is a bug

@hunngu-ms hunngu-ms requested review from a team as code owners June 2, 2026 20:36
@contentautomationbot
Copy link
Copy Markdown

ASIM parsers have been changed. ARM templates were regenerated from the updated KQL function YAML files.
To find the new ARM templates, pull your branch.


## Version 0.1.1

- (2026-06-02) Fixed incorrect DvcIpAddr mapping: replaced `Computer` (which contains a hostname) with `DeviceAddress` (the standard CEF device IP field). Removed redundant DvcHostname override that was clobbering the correct value from `_ASIM_ResolveDvcFQDN`.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Second part of changelog doesn't seem like it was done?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Bad changelog. We do not want to remove DvcHostName


## Version 0.1.1

- (2026-06-02) Fixed incorrect DvcIpAddr mapping: replaced `Computer` (which contains a hostname) with `DeviceAddress` (the standard CEF device IP field). Removed redundant DvcHostname override that was clobbering the correct value from `_ASIM_ResolveDvcFQDN`.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Second part of changelog doesn't seem like it was done?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Bad changelog. We do not want to remove DvcHostName

@hunngu-ms hunngu-ms merged commit 320357e into master Jun 2, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants