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

Update hapi related packages #54168

Closed
kobelb opened this issue Jan 7, 2020 · 10 comments · Fixed by #80468
Closed

Update hapi related packages #54168

kobelb opened this issue Jan 7, 2020 · 10 comments · Fixed by #80468
Assignees
Labels
chore Feature:Dependencies Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc

Comments

@kobelb
Copy link
Contributor

kobelb commented Jan 7, 2020

This was originally started in #48026 but the PR was closed because renovate kept messing up @spalger's work.

Some of the direct/transitive dependencies of Hapi are being falsely flagged by security vulnerability scanners.

/cc @elastic/kibana-security

@kobelb kobelb added chore Team:Operations Team label for Operations Team labels Jan 7, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@tylersmalley tylersmalley added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc and removed Feature:Dependencies labels Mar 26, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-platform (Team:Platform)

@tylersmalley tylersmalley added Feature:Dependencies Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc and removed Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Operations Team label for Operations Team labels Mar 26, 2020
@spalger spalger removed their assignment Mar 26, 2020
@spalger spalger closed this as completed Jun 11, 2020
@spalger spalger reopened this Jun 11, 2020
@watson
Copy link
Contributor

watson commented Oct 29, 2020

I'm upgrading a big bunch of them here in #80468. However, of the packages listed in #48026, this PR isn't upgrading @hapi/wreck, apollo-server-hapi, good-squeeze, or joi. I think however, that this issue could be considered closed once the PR lands and if we want to upgrade any of the 4 remaining packages, we should probably make separate issues for each of them.

Should I mark the PR so it closes this issue?

@legrego
Copy link
Member

legrego commented Oct 29, 2020

We can probably close this once #80468 is resolved, but I'd be interested in seeing what the new vulnerability report looks like after it merges. We will certainly close some existing findings, but we don't yet know what kind of new findings we'll see.

@watson
Copy link
Contributor

watson commented Oct 29, 2020

It would be nice if we could enable snyk to run on PRs so we can see if a PR introduces new vulnerabilities

@watson watson self-assigned this Oct 29, 2020
@legrego
Copy link
Member

legrego commented Oct 29, 2020

It would be nice if we could enable snyk to run on PRs so we can see if a PR introduces new vulnerabilities

I remember @jportner experimented with this a while back, and ran into some complications, but I don't know if they're are still issues, or if we just haven't had the time to revisit (I know I haven't...)

@jportner
Copy link
Contributor

I remember @jportner experimented with this a while back, and ran into some complications, but I don't know if they're are still issues, or if we just haven't had the time to revisit (I know I haven't...)

I didn't so much experiment with it as I accidentally enabled it 🙃 haven't taken a closer look than that!

@mshustov
Copy link
Contributor

mshustov commented Nov 2, 2020

@watson I created an issue for our team #82240
we need find owners for @hapi/wreck (Kibana operations?) & apollo-server-hapi version updates

@watson
Copy link
Contributor

watson commented Nov 2, 2020

Quick update regarding @hapi/wreck: I just checked again, and while I didn't intentionally upgrade wreck in the hapi upgrade PR, it is actually upgraded in the sense that we no longer depend on wreck - without the prefix - which we depended upon indirectly via hapi.

However, we still have a direct dependency on @hapi/wreck which is used here:

const promise = Wreck.request('GET', sourceUrl, reqOptions);

Now this direct dependency is just also used by hapi it self and we're currently locked on version 15.1.0, which is the latest v15.x version. And version 15.x is the one compatible with version 18 of hapi, so we're all good here 👍

@mshustov
Copy link
Contributor

mshustov commented Nov 2, 2020

@watson my point was that maybe we can use nodejs API to remove this dependency. WDYT @elastic/kibana-operations ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Feature:Dependencies Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants