Skip to content

Conversation

@chemamartinez
Copy link
Contributor

Proposed commit message

Modified the processing of user name fields including email addresses to align with the rest of the integrations. Changes applied include the following:

  • When the user name is an email, this field is dissected into <user.name>@<user.domain>.
  • Also the user.email field is populated with the full email address.
  • Append user.name and user.email to related.user.

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.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related issues

@chemamartinez chemamartinez self-assigned this May 30, 2025
@chemamartinez chemamartinez added enhancement New feature or request Integration:auth0 Auth0 Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels May 30, 2025
@chemamartinez chemamartinez marked this pull request as ready for review May 30, 2025 15:56
@chemamartinez chemamartinez requested a review from a team as a code owner May 30, 2025 15:56
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

copy_from: auth0.logs.data.user_name
if: 'ctx?.auth0?.logs?.data?.user_name != null'
tag: copy_user_email
if: ctx.auth0?.logs?.data?.user_name != null && ctx.auth0.logs.data.user_name.contains("@")
Copy link
Contributor

Choose a reason for hiding this comment

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

ctx.auth0.logs.data.user_name.indexOf("@") > 0?

@chemamartinez chemamartinez requested a review from efd6 June 2, 2025 10:25
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

Package auth0 👍(0) 💚(0) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
logs 5128.21 2421.31 -2706.9 (-52.78%) 💔

To see the full report comment with /test benchmark fullreport

@elastic-sonarqube
Copy link

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @chemamartinez

Copy link
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

Suggested commit message body reformat:

Modified the processing of user name fields including email addresses to
align with the rest of the integrations. Changes applied include the
following:

- When the user name is an email, this field is dissected into
  <user.name>@<user.domain>.
- Also the user.email field is populated with the full email address.
- Append user.name and user.email to related.user.

@chemamartinez chemamartinez merged commit 1f851bd into elastic:main Jun 3, 2025
8 checks passed
@chemamartinez chemamartinez deleted the 24531-username-processing-auth0 branch June 3, 2025 09:24
@elastic-vault-github-plugin-prod

Package auth0 - 1.22.0 containing this change is available at https://epr.elastic.co/package/auth0/1.22.0/

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:auth0 Auth0 Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants