Skip to content

[8.9] [Enterprise Search]Add 404 error handling for mappings and documents endpoints (#161203)#161234

Merged
kibanamachine merged 1 commit intoelastic:8.9from
kibanamachine:backport/8.9/pr-161203
Jul 5, 2023
Merged

[8.9] [Enterprise Search]Add 404 error handling for mappings and documents endpoints (#161203)#161234
kibanamachine merged 1 commit intoelastic:8.9from
kibanamachine:backport/8.9/pr-161203

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.9:

Questions ?

Please refer to the Backport tool documentation

…endpoints (elastic#161203)

## Summary

Add 404 handler for the Document and mappings endpoints. They were
default handlers before and returning 502. Error message is much more
meaningful at the moment.
![Screenshot 2023-07-04 at 18 16
53](https://github.com/elastic/kibana/assets/1410658/f595391b-2889-4370-907f-7e5c0d331f2c)
![Screenshot 2023-07-04 at 18 17
01](https://github.com/elastic/kibana/assets/1410658/018d34d1-273a-4d85-9f5a-78bbf15cf526)

### Checklist

Delete any items that are not applicable to this PR.

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit e654c8e)
@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
enterpriseSearch 10 9 -1

Async chunks

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

id before after diff
enterpriseSearch 2.5MB 2.5MB -36.0B

Page load bundle

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

id before after diff
enterpriseSearch 38.0KB 38.0KB +58.0B
Unknown metric groups

API count

id before after diff
enterpriseSearch 10 9 -1

ESLint disabled line counts

id before after diff
enterpriseSearch 13 15 +2
securitySolution 415 419 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 14 16 +2
securitySolution 496 500 +4
total +6

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

cc @efegurkan

@kibanamachine kibanamachine merged commit baf50a1 into elastic:8.9 Jul 5, 2023
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.

3 participants