-
Notifications
You must be signed in to change notification settings - Fork 319
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 base image to mitigate CVE's #330
Conversation
@jtblin @ccarlfjord @walkafwalka @mariusv Any chance this can get reviewed/merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@jtblin @ccarlfjord @walkafwalka now that this is reviewed can it be merged and released |
Any chance this will be released soon? |
Also looking to know if this will be released soon? |
@jtblin @ccarlfjord @walkafwalka Thank you for getting this merged, however it isn't available in a released version |
The other maintainers do not have access to DockerHub. @jtblin is the only one. I would suggest either building and hosting the image yourself or using AWS' IAM for Service Accounts. |
Travis will push the image to docker hub if it has a tag, I'll try and look
into it tomorrow.
…On Sun, 6 Feb 2022 at 02:30, Aaron Walker ***@***.***> wrote:
@jtblin <https://github.com/jtblin> @ccarlfjord
<https://github.com/ccarlfjord> @walkafwalka
<https://github.com/walkafwalka> Thank you for getting this merged,
however it isn't available in a released version
The other maintainers do not have access to DockerHub. @jtblin
<https://github.com/jtblin> is the only one.
I would suggest either building and hosting the image yourself or using
AWS' IAM for Service Accounts.
—
Reply to this email directly, view it on GitHub
<#330 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKTDLSBOWVXQJCREGQ4LWTUZU7BRANCNFSM5IQPCHVA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@jtblin any update? |
Hi @jtblin , any updates on when this can be pushed to docker? Thanks! |
It's now pushed, sorry for the lag. |
What this PR does / why we need it:
The current base image 3.12 has multiple CVE's , to mitigate the issues we are updating the base image
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Special notes:
Checklist chart
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]