Skip to content

[Security Solution] Display multiple copyable fields for process.args in resolver node detail panel#93280

Merged
kqualters-elastic merged 2 commits intoelastic:masterfrom
kqualters-elastic:resolver-show-all-array-values-node-detail
Mar 4, 2021
Merged

[Security Solution] Display multiple copyable fields for process.args in resolver node detail panel#93280
kqualters-elastic merged 2 commits intoelastic:masterfrom
kqualters-elastic:resolver-show-all-array-values-node-detail

Conversation

@kqualters-elastic
Copy link
Contributor

Summary

Previously, the node detail panel in resolver would only display the first value the process.args array, this pr changes the panel to show all values in the same style as other panel views.
multiple_process_args

Checklist

@kqualters-elastic kqualters-elastic added release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team v7.13.0 labels Mar 2, 2021
@kqualters-elastic kqualters-elastic requested a review from a team as a code owner March 2, 2021 18:08
@elasticmachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

💛 Build succeeded, but was flaky


Test Failures

Kibana Pipeline / general / Closes and opens alerts.Closing alerts Closes and opens alerts

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has not failed recently on tracked branches

AssertionError: Timed out retrying after 60000ms: expected '<span.euiBadge.euiBadge--hollow.euiBadge--iconLeft>' to have text '397', but the text was '497'
    at Context.eval (http://localhost:6121/__cypress/tests?p=cypress/integration/detection_alerts/closing.spec.ts:20202:43)

Metrics [docs]

Async chunks

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

id before after diff
securitySolution 7.8MB 7.8MB +331.0B

History

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

@kqualters-elastic kqualters-elastic merged commit 8937349 into elastic:master Mar 4, 2021
@kqualters-elastic kqualters-elastic deleted the resolver-show-all-array-values-node-detail branch March 4, 2021 05:25
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 4, 2021
* master: (48 commits)
  Fix wrong import in data plugin causing 100kB bundle increase (elastic#93448)
  [Fleet] Correctly track install status of an integration (elastic#93464)
  Reviews data frame analytics UI text (elastic#93033)
  Display multiple copyable fields for process.args in resolver node detail panel (elastic#93280)
  [Security Solution][Detections] ML Popover overflow fix (elastic#93525)
  chore(NA): do not use execa on bazel workspace status update script (elastic#93532)
  Bump dependencies (elastic#93511)
  [dev/build_ts_refs] support disabling the ts-refs build completely (elastic#93529)
  [Security Solution] fix data provider cypress test (elastic#93465)
  Fix service map for All environment single service (elastic#93517)
  [Fleet] Fix package version comparaison in the UI (elastic#93498)
  [alerting] adds doc on JSON-expanded action variables and task manager max_workers (elastic#92720)
  [dev/build_ts_refs] ignore type checking failures when building ts refs (elastic#93473)
  [core-new-docs] Adds a dev-doc for core documentation (elastic#92976)
  remove opacity from maps legacy style (elastic#93456)
  [Security Solution][Lists] Escape quotes in list ids and quote the id in KQL query (elastic#93176)
  Revert "Make tests deterministic by providing unique timestamps (elastic#93350)"
  [Discover] Fix link from dashboard saved search to Discover (elastic#92937)
  update public api docs
  App Search - Polishing Analytics Views (elastic#92939)
  ...
kqualters-elastic added a commit to kqualters-elastic/kibana that referenced this pull request Mar 5, 2021
kqualters-elastic added a commit to kqualters-elastic/kibana that referenced this pull request Mar 5, 2021
kqualters-elastic added a commit to kqualters-elastic/kibana that referenced this pull request Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team v7.13.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants