Skip to content

[8.5] [Security Solution][Fix]-Issue with disabled dataProvider (#140735)#141424

Merged
kibanamachine merged 2 commits intoelastic:8.5from
kibanamachine:backport/8.5/pr-140735
Sep 22, 2022
Merged

[8.5] [Security Solution][Fix]-Issue with disabled dataProvider (#140735)#141424
kibanamachine merged 2 commits intoelastic:8.5from
kibanamachine:backport/8.5/pr-140735

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.5:

Questions ?

Please refer to the Backport tool documentation

…0735)

Fixes : elastic#129958

When the data provider was disabled, the final query getting created is not syntactically correct and throws a syntax error as show in the screenshot below:

![](https://user-images.githubusercontent.com/2946766/162839613-88320f35-ec0d-4df3-aa66-167593ef4955.png)

(cherry picked from commit ec1fe0a)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Sep 22, 2022
@kibanamachine kibanamachine enabled auto-merge (squash) September 22, 2022 13:09
@spalger
Copy link
Copy Markdown
Contributor

spalger commented Sep 22, 2022

@elasticmachine merge upstream

(to resolve ci issues caused by Github outage)

@kibanamachine kibanamachine merged commit ba9f3a0 into elastic:8.5 Sep 22, 2022
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
timelines 344 346 +2

Async chunks

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

id before after diff
securitySolution 6.6MB 6.6MB -970.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
timelines 268.9KB 268.7KB -180.0B
Unknown metric groups

API count

id before after diff
timelines 450 452 +2

History

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

cc @logeekal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants