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

Integration for Auth0 #2152

Merged
merged 45 commits into from
Jan 25, 2022
Merged

Integration for Auth0 #2152

merged 45 commits into from
Jan 25, 2022

Conversation

r00tu53r
Copy link
Contributor

@r00tu53r r00tu53r commented Nov 11, 2021

What does this PR do?

This PR adds new a new package to enable Auth0 log events integration.

Public documentation for event schema -

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • If I'm introducing a new feature, I have modified the Kibana version constraint in my package's manifest.yml file to point to the latest Elastic stack release (e.g. ^7.13.0).

Author's Checklist

  • Check dashboard graphs
  • Check ECS for fields

How to test this PR locally

  • Run pipeline tests
elastic-package test pipeline

Related issues

Screenshots

Auth0-Dashboard-v2

* Create login group (auth0.data.login)
  - for fields from details of type (s)
* Update pipeline to set initiatedTime
* Create separate test for each type
@r00tu53r r00tu53r self-assigned this Nov 11, 2021
@elasticmachine
Copy link

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@elasticmachine
Copy link

elasticmachine commented Nov 11, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-01-24T16:20:22.681+0000

  • Duration: 17 min 44 sec

  • Commit: b5ffa92

Test stats 🧪

Test Results
Failed 0
Passed 17
Skipped 0
Total 17

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

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.

This will need some system tests to exercise the HTTP + TLS + authentication aspect of the integration.

If you used any public documentation for the event schema please link it in the PR description.

@r00tu53r
Copy link
Contributor Author

r00tu53r commented Dec 9, 2021

This will need some system tests to exercise the HTTP + TLS + authentication aspect of the integration.

If you used any public documentation for the event schema please link it in the PR description.

Public documentation for event schema. Added them to the description too -

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 pipeline is looking good. I added a few comments.

This will need the system tests before we can merge it.

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.

LGTM 👍

@r00tu53r r00tu53r merged commit 1639d33 into elastic:main Jan 25, 2022
eyalkraft pushed a commit to build-security/integrations that referenced this pull request Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.1-candidate enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auth0
3 participants