Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filebeat Okta module #16362

Merged
merged 33 commits into from
Mar 18, 2020
Merged

Filebeat Okta module #16362

merged 33 commits into from
Mar 18, 2020

Conversation

alakahakai
Copy link

@alakahakai alakahakai commented Feb 16, 2020

Filebeat Okta module for Okta system logs.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/siem (Team:SIEM)

@andrewkroh andrewkroh changed the title Filebeat OKTA module Filebeat Okta module Feb 18, 2020
@adriansr adriansr self-requested a review February 18, 2020 17:14
Copy link
Contributor

@leehinman leehinman left a comment

Choose a reason for hiding this comment

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

Looks really good.

A couple of suggestions for mappings

  • add client.ip & source.ip to related.ip array
  • set event.kind to event
  • set event.category (are both iam and authentication appropriate?)
  • set event.type (allowed/denied for logins, probably user as well?)
  • map okta.event_type to event.action
  • map okta.outcome.result to event.outcome
  • map okta.uuid -> event.id
  • map okta.actor.display_name to user.full_name if okta.actor.type is User
  • map okta.actor.id to user.id if okta.actor.type is User

filebeat/docs/modules/okta.asciidoc Outdated Show resolved Hide resolved
filebeat/docs/modules/okta.asciidoc Outdated Show resolved Hide resolved
@andrewkroh
Copy link
Member

Please add an overview dashboard to accompany the module and highlight some of the key signals coming from the data (activity over time, geo data, failures, etc).

Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

There were some more renames of OKTA to Okta.

@alakahakai Is this ready for another look by @leehinman?

I think the only thing left is to add an overview dashboard. Maybe you want to add that in a follow up PR to get the testing going sooner?

x-pack/filebeat/module/okta/_meta/fields.yml Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/_meta/fields.yml Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/_meta/fields.yml Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/system/_meta/fields.yml Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/system/_meta/fields.yml Outdated Show resolved Hide resolved
x-pack/filebeat/modules.d/okta.yml.disabled Outdated Show resolved Hide resolved
x-pack/filebeat/modules.d/okta.yml.disabled Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/_meta/docs.asciidoc Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/_meta/config.yml Outdated Show resolved Hide resolved
x-pack/filebeat/module/okta/_meta/config.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@leehinman leehinman left a comment

Choose a reason for hiding this comment

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

Looking good. Just a few ECS changes.

Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

The dashboard looks nice

x-pack/filebeat/input/httpjson/input.go Outdated Show resolved Hide resolved
@alakahakai alakahakai merged commit ae43c2e into elastic:master Mar 18, 2020
alakahakai pushed a commit that referenced this pull request Mar 23, 2020
* Filebeat Okta module (#16362)

(cherry picked from commit ae43c2e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants