-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
x-pack/filebeat/input/entityanalytics/provider/okta/internal/okta: new package #35478
Conversation
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
@taylor-swanson would you mind reviewing this one? Dan has added support for Okta users to the input, and Crest hoping to start on the package itself once we get this PR reviewed & merged. |
@jamiehynds This is the first half, the actual input will build on this. |
40f618b
to
757f94b
Compare
cc0a21d
to
6818ea8
Compare
This pull request is now in conflicts. Could you fix it? 🙏
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks really great! Aside from a minor comment and fixing the linter issues, LGTM.
What does this PR do?
This adds a new package for accessing the Okta user API.
Why is it important?
It is necessary for adding the Okta entity analytics input.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Obtain an Okta dev account and set up an auth token, then you can run the tests in x-pack/filebeat/input/entityanalytics/provider/okta/internal/okta with
Related issues
Use cases
Screenshots
Logs