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

[Security solutions] Fix execution context page #152816

Merged

Conversation

machadoum
Copy link
Member

@machadoum machadoum commented Mar 7, 2023

Summary

Event Based Telemetry sends context data for each event, for that it uses Kibana ExecutionContext. Before this PR events sent from Security Solutions had context.page value undefined.

How to test it

  1. Open the browser console and filter for EBT
  2. Navigate between pages
  3. Check if the click event has the right context.page value.

@machadoum machadoum force-pushed the siem-explore-145276-fix-execution_context branch from edf0d29 to 8447bf2 Compare March 7, 2023 14:15
@machadoum machadoum changed the title Siem explore 145276 fix execution context [Security solutions] Fix execution context page Mar 7, 2023
@machadoum machadoum self-assigned this Mar 8, 2023
@machadoum machadoum added Team:Threat Hunting Security Solution Threat Hunting Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Explore v8.8.0 telemetry Issues related to the addition of telemetry to a feature release_note:skip Skip the PR/issue when compiling release notes labels Mar 8, 2023
@machadoum machadoum marked this pull request as ready for review March 8, 2023 08:46
@machadoum machadoum requested a review from a team as a code owner March 8, 2023 08:46
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@machadoum machadoum requested review from angorayc and afharo March 8, 2023 08:48
Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

Core review only. LGTM.

Just added a NIT 😇

@machadoum machadoum force-pushed the siem-explore-145276-fix-execution_context branch from 8447bf2 to 641346c Compare March 9, 2023 14:23
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 3751 3752 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 15.7MB 15.7MB +3.0KB
Unknown metric groups

ESLint disabled line counts

id before after diff
securitySolution 431 433 +2

Total ESLint disabled count

id before after diff
securitySolution 508 510 +2

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @machadoum

@machadoum machadoum merged commit 0a6458b into elastic:main Mar 10, 2023
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Mar 10, 2023
bmorelli25 pushed a commit to bmorelli25/kibana that referenced this pull request Mar 10, 2023
## Summary

Event Based Telemetry sends context data for each event, for that it
uses Kibana `ExecutionContext`. Before this PR events sent from Security
Solutions had `context.page` value undefined.



### How to test it
1. Open the browser console and filter for EBT
2. Navigate between pages 
3. Check if the click event has the right `context.page` value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Explore Team:Threat Hunting Security Solution Threat Hunting Team telemetry Issues related to the addition of telemetry to a feature v8.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants