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

[SIEM] When no siem:defaultIndex is configured, 'SIEM job fetch failure' error shouldn't be displayed #49753

Closed
spong opened this issue Oct 30, 2019 · 2 comments
Labels
bug Fixes for quality problems that affect the customer experience Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:SIEM v7.5.0

Comments

@spong
Copy link
Member

spong commented Oct 30, 2019

When no siem:defaultIndex is configured and we're showing the placeholder overview for each page, we shouldn't show the SIEM job fetch failure error as it is not helpful to the user, and adding indices as the page suggest will resolve the issue.

E.g.
image

@spong spong added bug Fixes for quality problems that affect the customer experience Team:SIEM v7.5.0 labels Oct 30, 2019
@spong spong self-assigned this Oct 30, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/siem (Team:SIEM)

@spong spong removed their assignment Jun 25, 2020
@spong
Copy link
Member Author

spong commented Jun 25, 2020

@MadameSheema this behavior is still present on 7.9.0-snapshot, albeit a little different... 😅

FrankHassanabad added a commit that referenced this issue Jul 31, 2020
… patterns can cause UI toaster errors (#73754)

## Summary

#49753

When you have no data you get a toaster error when we don't want a toaster error.

Before with the toaster error:
![error](https://user-images.githubusercontent.com/1151048/88860918-0e2a5900-d1ba-11ea-95e7-5ed7324fc831.png)

After:
You don't get an error toaster because I catch any regular expression errors and do not report them up to the UI.

### Checklist

- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
FrankHassanabad added a commit to FrankHassanabad/kibana that referenced this issue Jul 31, 2020
… patterns can cause UI toaster errors (elastic#73754)

## Summary

elastic#49753

When you have no data you get a toaster error when we don't want a toaster error.

Before with the toaster error:
![error](https://user-images.githubusercontent.com/1151048/88860918-0e2a5900-d1ba-11ea-95e7-5ed7324fc831.png)

After:
You don't get an error toaster because I catch any regular expression errors and do not report them up to the UI.

### Checklist

- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
FrankHassanabad added a commit that referenced this issue Jul 31, 2020
… patterns can cause UI toaster errors (#73754) (#73952)

## Summary

#49753

When you have no data you get a toaster error when we don't want a toaster error.

Before with the toaster error:
![error](https://user-images.githubusercontent.com/1151048/88860918-0e2a5900-d1ba-11ea-95e7-5ed7324fc831.png)

After:
You don't get an error toaster because I catch any regular expression errors and do not report them up to the UI.

### Checklist

- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
FrankHassanabad added a commit that referenced this issue Jul 31, 2020
…ty spaces (#73991)

## Summary

Fixes fully this issue: #49753

If you go to advanced settings and configure siemDefaultIndex to be an empty string or have empty spaces:

<img width="1291" alt="Screen Shot 2020-07-31 at 12 52 00 PM" src="https://user-images.githubusercontent.com/1151048/89067511-a9434000-d32c-11ea-9106-e2079a5db317.png">

You shouldn't get any toaster errors when going to any of the pages such as overview, detections, etc...

This fixes that and adds both unit and integration tests around those areas. The fix is to add a filter which will filter all the patterns out that are either empty strings or have the _all within them rather than just looking for a single value to exist.

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
FrankHassanabad added a commit to FrankHassanabad/kibana that referenced this issue Jul 31, 2020
…ty spaces (elastic#73991)

## Summary

Fixes fully this issue: elastic#49753

If you go to advanced settings and configure siemDefaultIndex to be an empty string or have empty spaces:

<img width="1291" alt="Screen Shot 2020-07-31 at 12 52 00 PM" src="https://user-images.githubusercontent.com/1151048/89067511-a9434000-d32c-11ea-9106-e2079a5db317.png">

You shouldn't get any toaster errors when going to any of the pages such as overview, detections, etc...

This fixes that and adds both unit and integration tests around those areas. The fix is to add a filter which will filter all the patterns out that are either empty strings or have the _all within them rather than just looking for a single value to exist.

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
FrankHassanabad added a commit that referenced this issue Jul 31, 2020
…ty spaces (#73991) (#74020)

## Summary

Fixes fully this issue: #49753

If you go to advanced settings and configure siemDefaultIndex to be an empty string or have empty spaces:

<img width="1291" alt="Screen Shot 2020-07-31 at 12 52 00 PM" src="https://user-images.githubusercontent.com/1151048/89067511-a9434000-d32c-11ea-9106-e2079a5db317.png">

You shouldn't get any toaster errors when going to any of the pages such as overview, detections, etc...

This fixes that and adds both unit and integration tests around those areas. The fix is to add a filter which will filter all the patterns out that are either empty strings or have the _all within them rather than just looking for a single value to exist.

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios
@MindyRS MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Oct 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:SIEM v7.5.0
Projects
None yet
Development

No branches or pull requests

4 participants