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

Security vulnerabilities #132

Closed
danieljwilson opened this issue Dec 9, 2020 · 10 comments
Closed

Security vulnerabilities #132

danieljwilson opened this issue Dec 9, 2020 · 10 comments

Comments

@danieljwilson
Copy link

I noticed that there are some high level vulnerabilities on the latest image.

Is this cause for concern? I am about to submit a project proposal to a Research Ethics Board and felt like I should know what these vulnerabilities actually mean in terms of data security. Is it a matter of updating the packages?

@vsoch
Copy link
Member

vsoch commented Dec 9, 2020

We could definitely do a PR with suggested updates, the builder-base Dockerfile is here https://github.com/expfactory/expfactory/tree/master/expfactory/templates/build/docker/builder-base and then that is bootstrapped by https://github.com/expfactory/expfactory/blob/master/expfactory/templates/build/docker/builder/Dockerfile.

Let me give it a quick shot and push a test image, and then we can see if the analysis improves. If not, I'll share the changes and you can take a shot, and if it does, then I can open a PR and see if anything is broken (and you can further test too). Sound good?

@vsoch
Copy link
Member

vsoch commented Dec 9, 2020

Also, I'm going to transfer this issue to expfactory/expfactory - this expfactory-docker repository is for the Poldracklab deployed version of expfactory.org.

@danieljwilson
Copy link
Author

Great, thanks for the quick response!

@vsoch vsoch transferred this issue from expfactory/expfactory-docker Dec 9, 2020
@vsoch
Copy link
Member

vsoch commented Dec 9, 2020

okay, I've pushed a test image, the same repository with tag test-12-9-2020 (at the top) and it looks like the security scan is green! Do you want to take it for a spin to see if it works as you'd expect? If it looks good to you, I'll open a PR with the changes.

@vsoch
Copy link
Member

vsoch commented Dec 9, 2020

Oops one second, I didn't build the image from the branch that is now for PR. Give me a second to build again!

@vsoch
Copy link
Member

vsoch commented Dec 9, 2020

okay, all set.

@danieljwilson
Copy link
Author

Great, have a meeting and will test afterwards!

@danieljwilson
Copy link
Author

Seems like it worked (though keep in mind this is my second day using expfactory)...

@vsoch
Copy link
Member

vsoch commented Dec 9, 2020

No worries! We have basic tests in the CI (and they have passed) and I'm of the opinion that it's better to release often, and we can address any issues that might arise.

@vsoch
Copy link
Member

vsoch commented Dec 10, 2020

Issues fixed, closing issue.

@vsoch vsoch closed this as completed Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants