Skip to content

Conversation

@yummyblabla
Copy link
Collaborator

@yummyblabla yummyblabla commented Jan 7, 2026

Changes to Authentication OktaSSO parser:

  • Addition: EventResultDetails, SrcDeviceType
  • Remaps: EventSubType --> EventOriginalSubtype, TargetUserType
  • Extensions: ActorUserId, ActorUsername, etc. are extended from its Target counterparts.

EventSubType --> EventOriginalSubtype Change Rationale:

  • Subtype did not fit EventSubtype enumerations

TargetUserType mapping addition Rationale:

  • UserType did not fix UserType enumerations.

@yummyblabla yummyblabla requested review from a team as code owners January 7, 2026 21:10
Copy link
Contributor

@oshezaf oshezaf left a comment

Choose a reason for hiding this comment

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

The column_if_exists statements are very expensive as they imply calculated columns later on. There is a reason to why they are used (needed only for V1 tables. I can explain more). It certainly makes prefiltering ineffective. The way around it is to use a union and s datatable:

let empty = datatable (uni_s:string) [];
(Syslog | union empty)
| where uni_s == ""

@yummyblabla yummyblabla changed the title [ASIM] Make changes to Authentication ASIM OktaSSO parser [ASIM] Make changes to Authentication ASIM OktaSSO and OktaV2 parser Jan 8, 2026
Copy link
Contributor

@oshezaf oshezaf left a comment

Choose a reason for hiding this comment

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

Thanks! looks good. Two small comments.

| where not(disabled)
| lookup OutcomeReasonLookup on outcome_reason_s
| extend EventResultDetails = iif(outcome_result_s in (OktaFailedOutcome), coalesce(EventResultDetails, "Other"), "")
| extend
Copy link
Contributor

Choose a reason for hiding this comment

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

Split between extend for calculated fields and project-rename for just mapping.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Agreed. Updated in latest iteration

EventOriginalSubType=legacyEventType_s,
EventMessage=displayMessage_s,
EventOriginalResultDetails=outcome_reason_s,
EventOriginalUid=coalesce(uuid_g, ""),
Copy link
Contributor

Choose a reason for hiding this comment

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

No need for coalesce I think, so can be project-renamed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

By product of initial testing. Removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants