Commit graph

292 commits

Author SHA1 Message Date
Sergi Massaneda
56669c034c
[RAC] Persistent timeline fields fix (#110685)
* fix stringify circular ref crash and default columns on createTimeline

* rollback reset buton fix to split PR

* adding fields to the storage cleaning

* tests fixed

* test fix
2021-09-02 16:33:07 +02:00
Spencer
fecdba7eba
[eslint] add rule to prevent export* in plugin index files (#109357)
* [eslint] add rule to prevent export* in plugin index files

* deduplicate export names for types/instances with the same name

* attempt to auto-fix duplicate exports too

* capture exported enums too

* enforce no_export_all for core too

* disable rule by default, allow opting-in for help fixing

* update tests

* reduce yarn.lock duplication

* add rule but no fixes

* disable all existing violations

* update api docs with new line numbers

* revert unnecessary changes to yarn.lock which only had drawbacks

* remove unnecessary eslint-disable

* rework codegen to split type exports and use babel to generate valid code

* check for "export types" deeply

* improve test by using fixtures

* add comments to some helper functions

* disable fix for namespace exports including types

* label all eslint-disable comments with related team-specific issue

* ensure that child exports of `export type` are always tracked as types

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-09-01 18:05:45 -07:00
Angela Chuang
9dec97b5f4
[Security Solution] Enable repositionOnScroll for overflow popovers (#110584)
* enable repositionOnScroll

* unit test

* update test case

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-09-01 14:50:33 -04:00
Christos Nasikas
1f90712c47
[Timeline] Show cases create flyout with timeline open (#110811) 2021-09-01 20:06:55 +03:00
Sergi Massaneda
e9c20f9802
[RAC] Expand cell footer button changes to prevent overflow (#110506)
* fix hover actions tooltip footer buttons overflow

* reorder actions

* override eui styles to match popover actions design
2021-09-01 16:47:43 +02:00
Jonathan Buttner
31d335868e
Disable sync toggle in flyout (#110714) 2021-09-01 09:06:29 -04:00
Xavier Mouligneau
16af3e95cb
[RAC] Remove rbac on security solution side (#110472)
* wip to remove rbac

* Revert "[Cases] Include rule registry client for updating alert statuses (#108588)"

This reverts commit 1fd7038b34.

This leaves the rule registry mock changes

* remove rbac on Trend/Count alert

* update detection api for status

* remove @kbn-alerts packages

* fix leftover

* Switching cases to leverage update by query for alert status

* Adding missed files

* fix bad logic

* updating tests for use_alerts_privileges

* remove index alias/fields

* fix types

* fix plugin to get the right index names

* left over of alis on template

* forget to use current user for create/read route index

* updated alerts page to not show table when no privileges and updates to tests

* fix bug when switching between o11y and security solution

* updates tests and move to use privileges page when user tries to access alerts without proper access

* updating jest tests

* pairing with yara

* bring back kbn-alerts after discussion with the team

* fix types

* fix index field for o11y

* fix bug with updating index priv state

* fix i18n issue and update api docs

* fix refresh on alerts

* fix render view on alerts

* updating tests and checking for null in alerts page to not show no privileges page before load

* fix details rules

Co-authored-by: Jonathan Buttner <jonathan.buttner@elastic.co>
Co-authored-by: Yara Tercero <yara.tercero@elastic.co>
2021-09-01 04:23:44 -04:00
Steph Milovic
03d52e9920
[Security Solution] [Bugfix]Alerts table event rendered view timestamp (#110561) 2021-08-31 07:40:59 -06:00
Xavier Mouligneau
782f29a407
[RAC][o11y] fix o11y privileges when rule created in stack (#110167)
* fix o11y privileges when rule created in stack

* fix merge

* fix dsl alerts

* fix privileges on o11y

* after discussion with o11y, we agree to simplify logic of the count of alerts

* remove unused variable

* fix one more type error

Co-authored-by: mgiota <panagiota.mitsopoulou@elastic.co>
2021-08-31 07:31:38 -04:00
Sergi Massaneda
7ebed9321a
fix view all fields button when filtered (#110464) 2021-08-31 10:19:11 +02:00
Pablo Machado
3854d3a586
[RAC] EuiDataGrid pagination (#109269)
* Update T-Grid to use DataGrid pagination

* It also improves the Gtid loading state
* DataGrid pagination makes sure that we display the grid with the proper height.

* Add DataGrid height hack to t-grid

HUGE HACK!!!

DataGrtid height isn't properly calculated when the grid has horizontal scroll.
https://github.com/elastic/eui/issues/5030

In order to get around this bug we are calculating `DataGrid` height here and setting it as a prop.

Please revert this commit  and allow DataGrid to calculate its height when the bug is fixed.

* Apply DataGrid laoding and pagination changes to observability

* Fix cypress tests

* Fix t-grid page render bug on Observability

* some pagination fixes

* hide table when analyzer active

* isolate exported function

Co-authored-by: semd <sergi.massaneda@elastic.co>
2021-08-26 14:41:47 -04:00
Michael Olorunnisola
416d42a22a
[Security Solution][RAC] Refix expand (#110236) 2021-08-26 12:01:28 -04:00
Angela Chuang
602392e88d
[Security Solution] Host details fly out modal is not working in alerts table (#109942)
* fix expanded host and ip panel

* reuse existing links components

* rename

* add unit tests

* add unit tests

* update comment

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-26 11:40:30 -04:00
Steph Milovic
27af6ef068
[Security Solution] Bugfix for disable state of External Alert context menu (#109914) 2021-08-25 12:49:19 -06:00
Xavier Mouligneau
6a1a38b346
[RAC] [o11y] add permission in alerts table from kibana privilege/consumer (#109759)
* add alert permission in o11y

* review I

* review II

* fix selection all when checkbox disabled

* fix selected on bulk actions
2021-08-24 23:32:40 -04:00
Andrew Goldstein
a161c2b7d8
[RAC] [TGrid] Use EuiDataGridColumn schemas (for sorting) (#109983)
## Summary

Updates the `TGrid` to use `EuiDataGrid` [schemas](https://eui.elastic.co/#/tabular-content/data-grid-schemas-and-popovers/) as suggested by @snide in the following issue: <https://github.com/elastic/kibana/issues/108894>

## Desk testing

1) In the `Security Solution`, navigate to `Security > Rules` and enable multiple detection rules that have different `Risk Score`s

**Expected result**

- The Detection Engine generates alerts (when the rule's criteria is met) that have different risk scores

2) Navigate to the `Security > Alerts` page

**Expected results**

As shown in the screenshot below:

- The alerts table is sorted by `@timestamp` in descending (Z-A) order, "newest first"
- The `@timestamp` field in every row is newer than, or the same time as the row below it
- The alerts table shows a non-zero count of alerts, e.g. `20,600 alerts`

![alerts-table-at-page-load](https://user-images.githubusercontent.com/4459398/130700525-343d51af-7a3a-475c-b3b4-b429bc212adf.png)

_Above: At page load, the alerts table is sorted by `@timestamp` in descending (Z-A) order, "newest first"_

3) Observe the count of alerts shown in the header of the alerts table, e.g. `20,600 alerts`, and then change the global date picker in the KQL bar from `Today` to `Last 1 year`

**Expected results**

- The golbal date picker now reads `Last 1 year`
- The count of the alerts displayed in the alerts table has increased, e.g. from `20,600 alerts` to `118,709 alerts`
- The `@timestamp` field in every row is (still) newer than, or the same time as the row below it

4) Click on the `@timestamp` column, and choose `Sort A-Z` from the popover, to change the sorting to ascending, "oldest first", as shown in the screenshot below:

![click-sort-ascending](https://user-images.githubusercontent.com/4459398/130701250-3f229644-2a78-409e-80ff-f88588562190.png)

_Above: Click `Sort A-Z` to sort ascending, "oldest first"_

**Expected results**

As shown in the screenshot below:

- The alerts table is sorted by `@timestamp` in ascending (A-Z) order, "oldest first"
- The `@timestamp` field in every row is older than, or the same time as the row below it
- `@timestamp` is older than the previously shown value, e.g. `Aug 3` instead of `Aug 24`

![timestamp-ascending-oldest-first](https://user-images.githubusercontent.com/4459398/130702221-cc8cf84f-c044-4574-8a93-b9d35c14c890.png)

_Above: The alerts table is now sorted by `@timestamp` in ascending (A-Z) order, "oldest first"_

5) Click on the `Risk Score` column, and choose `Sort A-Z` from the popover, to add `Risk Score` as a secondary sort in descending (Z-A) "highest first" order, as shown in the screenshot below:

![sort-risk-score](https://user-images.githubusercontent.com/4459398/130702599-e4c0d74a-8775-435b-a263-5b6b278f6dfd.png)

_Above: Click `Sort A-Z` to add `Risk Score` as a secondary sort in descending (Z-A) "highest first" order_

**Expected results**

- The alerts table re-fetches data
- The alerts table shows `2 fields sorted`

6) Hover over the alerts table and click the `Inspect` magnifiing glass icon

**Expected result**

- The `Inspect` modal appaers, as shown in the screenshot below:

![inspect](https://user-images.githubusercontent.com/4459398/130702849-1189f32e-eb03-4d9d-b248-6c6f0b5665fa.png)

_Above: the `Inspect` modal_

7) Click the `Request` tab, and scroll to the `sort` section of the request

**Expected result**

Per the JSON shown below:

- The request is sorted first by `@timestamp` in ascending (A-Z) order, "oldest first"
- The request is sorted second by `signal.rule.risk_score` descending (Z-A) "highest first" order

```json
  "sort": [
    {
      "@timestamp": {
        "order": "asc",
        "unmapped_type": "date"
      }
    },
    {
      "signal.rule.risk_score": {
        "order": "desc",
        "unmapped_type": "number"
      }
    }
  ],
```

8) Click `Close` to close the `Inspect` modal

9) Click `2 fields sorted` to display the sort popover

10) Use the drag handles to, via drag-and-drop, update the sorting such that `Risk Score` is sorted **before** `@timestamp`, as shown in the screenshot below:

![sort-by-risk-score-first](https://user-images.githubusercontent.com/4459398/130704159-523effa2-21ef-4599-a939-964fc523f9ec.png)

_Above: Use the drag handles to, via drag-and-drop, update the sorting such that `Risk Score` is sorted **before** `@timestamp`_

**Expected results**

As shown in the screenshot below:

- The table is updated to be sorted first by the higest risk score, e.g. previously `47`, now `73`
- The alerts table is sorted second by `@timestamp` in ascending (A-Z) order, "oldest first", and *may* have changed, e.g. from `Aug 3` to `Aug 12`, depending on the sample data in your environment

![highest-risk-score](https://user-images.githubusercontent.com/4459398/130704878-163a2427-fc7a-4755-9adc-a06b0d7b8e43.png)

_Above: The alerts table is now sorted first by highest risk score_

11) Once again, hover over the alerts table and click the `Inspect` magnifiing glass icon

12) Once again, click the `Request` tab, and scroll to the `sort` section of the request

**Expected result**

Per the JSON shown below:

- The request is sorted first by `signal.rule.risk_score` in descending (Z-A) "highest first" order
- The request is sorted second by `@timestamp` in ascending (A-Z) order, "oldest first"

```json
  "sort": [
    {
      "signal.rule.risk_score": {
        "order": "desc",
        "unmapped_type": "number"
      }
    },
    {
      "@timestamp": {
        "order": "asc",
        "unmapped_type": "date"
      }
    }
  ],
```
2021-08-24 20:53:35 -06:00
Kerry Gallagher
09fda96254
[RAC] Fix scrolling on Obs alerts table (#109139)
* Fix scrolling on obs alerts table and default to 50 items per page
2021-08-24 18:21:23 +01:00
Steph Milovic
a75db0550b
[Security Solution] [Bugfix] Fixes broken alert actions (add to case, investigate in timeline) (#109339) 2021-08-24 10:44:56 -04:00
Michael Olorunnisola
9014e113f5
[Security Solution][RAC] - Hide hover actions overflow (#109693)
* add overflow

* fix types error
2021-08-24 05:44:27 -06:00
Angela Chuang
e4bb6a91c4
fix disabled dropdown in alerts table (#109724) 2021-08-24 08:59:15 +01:00
Steph Milovic
a0bd0affdd
[Security Solution] [Bugfix] Fix analyzer missing from alert table (#109183)
* fix n test heyyyy

* more dead code

* fix type things

* change field type to const
* type fix
2021-08-23 12:38:56 -06:00
Gloria Hornero
fe6e97f0f9
[Security Solution] Fixes the Alerts timeline failing test on master (#109644)
* fixes 'Alerts timeline' tests

* adds data test subj for the loading spinner
2021-08-23 20:13:38 +02:00
Sergi Massaneda
7a4a5d8ff6
typo fixed (#109407) 2021-08-23 12:34:18 -04:00
Sergi Massaneda
64dff78dce
[RAC] Actions popovers UI unification (#109221)
* popover padding size unified

* remove panels from all context menus

* action items order changed

* cases menu items test fixed

* translations and small changes

* remove components not used anywhere

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Angela Chuang <yi-chun.chuang@elastic.co>
2021-08-23 08:42:24 -04:00
Xavier Mouligneau
e8e53e36e5
[RAC] Get o11y alerts in alerts table (#109346)
* get back index names  in o11y

* testing and integration

* fix types

* Avoid using the rule data client for field list

* Remove left-over index argument

* no needs of alert consumer anymore

Co-authored-by: Felix Stürmer <stuermer@weltenwort.de>
2021-08-20 12:52:34 -04:00
Alejandro Fernández Gómez
5fd903b7fe
[RAC] Enable workflow status filtering (#108215)
Co-authored-by: Jason Rhodes <jason.matthew.rhodes@gmail.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-20 12:07:09 +02:00
Sergi Massaneda
60af98a1b7
fix unit prop and default it only in the tGrid body (#109252) 2021-08-20 11:29:03 +02:00
Pablo Machado
4274a2bf68
Preserve timeline data when navigating between tabs (#106716) 2021-08-20 09:45:47 +02:00
Angela Chuang
9d4c062e64
[Security Solution] TopN chart styling issue (#109007)
* fix topN style

* add unit tests for topN

* add unit tests

* review
2021-08-19 12:35:28 -04:00
Yara Tercero
9fa41d1aef
[Security Solution] - Updating UI to work with new kibana privileges abstraction on alerts (#108961)
## Summary

Holy moly. 

What is happening in this PR? 🤷🏽‍♀️ Let's break it down:
- Added a package `@kbn/alerts` - another one?! ...yes
  - This is meant to add shared hooks and components around alerts as data
  - `useGetUserAlertsPermissions` - accepts the Kibana capabilities object and returns whether the user has `read` and `crud` alerts privileges
  - `AlertsFeatureNoPermissions` - component displayed when user does not have alerts privileges
- UI changes for user with NO alerts privileges
  - `Alerts` tab hidden in security solution side navigation
  - `Alerts` tab hidden in rule details page  
- UI changes for user with alerts READ ONLY privileges
  - alerts checkboxes hidden in alerts table
  - alerts bulk actions hidden in alerts table
2021-08-18 14:48:31 -04:00
Sergi Massaneda
ff58841d89
added defaultColumns property to default security solutions timeline (#109086)
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-18 17:17:53 +02:00
Marshall Main
05d1e326a4
[RAC][Security Solution] Remove ALERT_RULE_ID in favor of ALERT_RULE_UUID (#108922)
* Remove ALERT_RULE_ID in favor of ALERT_RULE_UUID

* Update snapshot

* KEVINNN

* fix test

* Add back home.disableWelcomeScreen=true

* Only disable welcome screen in security solution cypress tests

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-18 10:37:14 -04:00
Xavier Mouligneau
e4bf6140fb
[SECURITY SOLUTION] [RAC] bug actions padding (#109029)
* wip

* match design for selecting grid view

* wip to integrate event rendered view

* wip

* integration of the event rendered

* fix perPage action on Euibasic table

* Add bulding block background color to EventRenderedView

* styling

* remove header

* fix types

* fix unit tests

* use memo for listProps

* fix styling + add feature flag

* review I

* fix merge

* change the gutter size

* fix bugs

* fix alert consumers

Co-authored-by: Pablo Neves Machado <pablo.nevesmachado@elastic.co>
Co-authored-by: Angela Chuang <yi-chun.chuang@elastic.co>
Co-authored-by: Michael Olorunnisola <michael.olorunnisola@elastic.co>
2021-08-18 07:37:40 -04:00
Davis Plumlee
fc1a2bbd1b
[Security Solution][Detection Alerts] Changes in-progress status to acknowledged (#107972) 2021-08-18 02:12:16 -04:00
Xavier Mouligneau
3013e10eda
[SECURITY SOLUTION] [RAC] Event rendered view (#108644)
* wip

* match design for selecting grid view

* wip to integrate event rendered view

* wip

* integration of the event rendered

* fix perPage action on Euibasic table

* Add bulding block background color to EventRenderedView

* styling

* remove header

* fix types

* fix unit tests

* use memo for listProps

* fix styling + add feature flag

* review I

* fix merge

* change the gutter size

Co-authored-by: Pablo Neves Machado <pablo.nevesmachado@elastic.co>
Co-authored-by: Angela Chuang <yi-chun.chuang@elastic.co>
2021-08-17 17:04:35 -04:00
Sergi Massaneda
b607f42fcf
[RAC][Observability] Add status update actions in row menu (#108698)
* use rac alerts bulk_update

* cleanup

* adds replace ALERT_STATUS with ALERT_WORKFLOW_STATUS and updates tests and adds logic for switching between signal.status and workflow status when updating alerts in .siem-signals

* allow object and string types in query param, fixed single update api to use WORKFLOW_STATUS instead of ALERT_STATUS

* adds additional integration test for when query is a DSL object in addtion to KQL string

* optionally use fields api in requests if _source does not contain authz properties

* integrate bulk update to all hook calls

* adds fields support, fixes bug where we were writing to 'signals.status' and not { signals: {status }} in alerts client

* clean up and fixes

* fix a bug where we were not waiting for updates to complete when using ids param in alerts bulk update. Adds integration tests for detection engine testing update alerts with new alerts as data client routes

* take index name from ecsData props

* pr suggestions

* some more type fixes

* refactor and type fixes

* snapshot updated

* add status update actions to row context menu

* refactor to use dispatch function in o11y actions

* comment removed

* bring alertConsumer back

* bring indexNames back

* check capabilities to show status update items

Co-authored-by: Devin Hurley <devin.hurley@elastic.co>
2021-08-17 16:03:35 -04:00
Katrin Freihofner
cac84d7cce
styling updates to alerts table and alignment of elements (#108507)
Co-authored-by: Kerry Gallagher <471693+Kerry350@users.noreply.github.com>
2021-08-17 14:35:21 -04:00
Kevin Qualters
87c93abf1d
[RAC] [Observability] [Security Solution] Use correct url to management app for observability cases, use normalized ids (#108775)
* Use correct url to management app for observability cases, use normalized ids in timelines

* Update failing test

* Load alert details data to render flyout in case detail view
2021-08-17 13:49:59 -04:00
Stacey Gammon
9258ba5147
Adding owners to kibana plugins (#108407)
* Adding owners to kibana plugins

* Fix ui actions enhanced owner

* Account for virtual RAC team owning a plugin

* Fix empty githubTeam for ui actions
2021-08-17 10:21:06 -04:00
Angela Chuang
04a8a3c199
[RAC] integrating rbac search strategy with alert flyout (#108748)
* add alert consumers for useTimelineEventDetails

* set entityType to events

* rename to AlertConsumers

* set entityType to alerts

* send entity type to search strategy

* fix import

* fix import

* fix import

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-17 14:20:17 +01:00
DeFazio
848085a8c3
Remove styling from euiPanel around Alerts tables (#108603)
* Add styling to euiPanel

* remove styling on Observability side in favor of separate PR

* Fix merge conflicts

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-17 09:18:36 -04:00
Liza Katz
9d1a8a87f8
[Data][Es Query] Use ES types in Kuery functions (#108367)
* es-query types

* jest and lint

* cc

* options

* type

* types for kuery FUNCTIONS

* doc

* sec fixes

* typey type

* test typescript

* test

* fixes

* test

* cr

* cleanup a bit more

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-17 11:49:16 +02:00
Steph Milovic
82af747532
[RAC][Security Solution] Alert table: Resolver and Cases icons to bulk action menu (#108420) 2021-08-16 21:09:08 -06:00
Tiago Costa
3ab852f213
Revert "[RAC] integrating rbac search strategy with alert flyout (#108375)"
This reverts commit c6c24e4392.
2021-08-16 15:15:19 +01:00
Angela Chuang
c6c24e4392
[RAC] integrating rbac search strategy with alert flyout (#108375)
* add alert consumers for useTimelineEventDetails

* set entityType to events

* rename to AlertConsumers

* set entityType to alerts

* send entity type to search strategy

* fix import

* fix import

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-16 08:58:27 -04:00
Angela Chuang
fe0322ac1f
[Security Solution] flyout UI adjustment (#108192)
* styling

* fix hover actions

* init overflow button for flyout

* init overflow button

* topN btn

* remove popover from topN

* fix tests

* fix unit test

* add use hover action items hook

* fix for code review

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-16 08:02:42 -04:00
Angela Chuang
fc3b3cc2bd
Fix expand hover action in data grid (#108616)
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-16 11:51:49 +01:00
Pablo Machado
c2d5d1b6c2
Improve security alerts t-grid loading and empty state (#108527) 2021-08-16 06:15:21 -04:00
Michael Olorunnisola
565276a90d
[Security Solution][RAC] - Enable tGrid by default (#108632) 2021-08-15 22:10:24 -04:00
Kevin Qualters
0828788b66
[RAC][Security Solution][Observability] Add the add to new case and add to existing case actions to observability alerts table (#108502)
* [RAC][Security Solution][Observability] Add the add to new case and add to existing case actions to observability alerts table

* Remove fake data and make features work with observability data format

* Remove console.log and unused translations

* Remove commented out code

* Remove unneeded copy pasta id, create initializeStore function in timelines

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
2021-08-15 13:27:28 -04:00