Skip to content

[7.9] [Ingest Manager] Create default Error handler. Use on /setup & EPM routes (#74409)#76770

Merged
jfsiii merged 2 commits intoelastic:7.9from
jfsiii:backport/7.9/pr-74409
Sep 8, 2020
Merged

[7.9] [Ingest Manager] Create default Error handler. Use on /setup & EPM routes (#74409)#76770
jfsiii merged 2 commits intoelastic:7.9from
jfsiii:backport/7.9/pr-74409

Conversation

@jfsiii
Copy link
Copy Markdown
Contributor

@jfsiii jfsiii commented Sep 4, 2020

Backports the following commits to 7.9:

…utes (#74409)

* Add default Error handler. Try on /*setup & /epm/*

* Add return type. Rename interface

* Export error handler type & add comments

* use default error handler in installPackageHandler

* (re)-add comment
# Conflicts:
#	x-pack/plugins/ingest_manager/server/routes/epm/handlers.ts
@jfsiii jfsiii added the backport This PR is a backport of another PR label Sep 4, 2020
@jfsiii
Copy link
Copy Markdown
Contributor Author

jfsiii commented Sep 8, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

@jfsiii jfsiii merged commit 68d7c7c into elastic:7.9 Sep 8, 2020
dgieselaar pushed a commit to dgieselaar/kibana that referenced this pull request Sep 9, 2020
…utes (elastic#74409) (elastic#76770)

* Add default Error handler. Try on /*setup & /epm/*

* Add return type. Rename interface

* Export error handler type & add comments

* use default error handler in installPackageHandler

* (re)-add comment
# Conflicts:
#	x-pack/plugins/ingest_manager/server/routes/epm/handlers.ts

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
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