Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Vulnerability in tornado dependency #6

Open
yadudoc opened this issue Feb 9, 2021 · 1 comment
Open

Vulnerability in tornado dependency #6

yadudoc opened this issue Feb 9, 2021 · 1 comment

Comments

@yadudoc
Copy link
Contributor

yadudoc commented Feb 9, 2021

There's an active vulnerability in the tornado 6.1 library that's being caught and reported by the vulnerability check in our github actions. The dependency comes through funcx->parsl and through nbsphinx. The first case is being fixed and won't be a dependency in parsl 1.1.0. nbsphinx is a test-dependency and shouldn't affect the package.

This test failure also blocks container builds necessary to test deployments. I believe suspending these tests for now, and addressing this over the next couple of weeks is the best course of action.

@BenGalewsky @joshbryan-globus please let me know what you think.

@joshbryan-globus
Copy link
Contributor

I think that is fine. In addition to the points you made, the vulnerability itself seems to require a proxy to be configured that keys on (or otherwise interprets) a query parameter, and we don't do that. We should resolve it eventually just to make the lights turn green, but it's certainly not critical.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants