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.

Sign up for GitHub

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

Jump to bottom

Use Boom.boomify over Boom.wrap #18451

Closed
elasticmachine opened this issue Oct 3, 2017 · 2 comments
Closed

Use Boom.boomify over Boom.wrap #18451

elasticmachine opened this issue Oct 3, 2017 · 2 comments
Labels
good first issue low hanging fruit Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc

Comments

@elasticmachine
Copy link
Contributor

Original comment by @archanid:

LINK REDACTED in x-pack-kibana where we're currently using Boom.wrap, which is deprecated in Boom 5.2.0.
@elasticmachine
Copy link
Contributor Author

Original comment by @pickypg:

Related: #14325

@elasticmachine elasticmachine added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc good first issue low hanging fruit labels Apr 25, 2018
@rhoboat rhoboat removed their assignment Apr 25, 2018
@pgayvallet
Copy link
Contributor

Closing. Outdated and Boom usages are to be avoided in Kibana Platform.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue low hanging fruit Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pgayvallet @rhoboat @elasticmachine