-
Notifications
You must be signed in to change notification settings - Fork 44
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 concerns with using the '--ssh' option #238
Comments
Thank you for reporting this @slagelwa ! Appreciate you for digging into it.
This is correct. dsub just picks up the SSH image provided by the Lifesciences team to run an SSH server in that container. The intent, just as with the pipelines tool is make a bit of inspection possible, along with being able to inspect logs in real-time. I've contacted the Lifesciences team and will update here accordingly. |
Curious if there's been any feedback on this issue from the Lifesciences team? |
Not yet. I have pinged again for an update. The repo for pipelines tools is here. It may be worthwhile to file an issue there, though as you mentioned it hasn't been updated in a few years. |
Hi @slagelwa, The team that supports the Lifesciences API responded and agreed that the image needs updating. I've also filed googlegenomics/pipelines-tools#108 to track in that repo. |
I'm following up on an item reported by a colleague about and issue using
dsub
reporting an ssh error:I'm looking at the dsub code and see that the image it appears to use is gcr.io/cloud-genomics-pipelines/tools. It looks like that image was last updated in February of 2019. I pulled the docker image myself and ran an open source vulnerability scanning tool, trivy, on it:
A fair number of the critical CVE's are related to ssh. Am I right in understanding that this is the image that is used to provide ssh services to the VM when running an dsub job? And if so isn't there a concern that users might not have their (default or other) network setup properly and they could be giving these VM's external IP addresses?
And I almost wonder if this might be related to #233
The text was updated successfully, but these errors were encountered: