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

Alpine Openjdk 8u141 #138

Closed
wants to merge 1 commit into from
Closed

Conversation

jpramondon
Copy link

Our Twistlock has recently reported a couple of security issues labelled as "high", which breaks our build. Most of the reported problems come from Openjdk version 8u131. Hence, this updated Dockerfile moves Openjdk to 8u141.
Additional apk repositories have been added in order to resolve all the required dependencies.

Our Twistlock has recently reported a couple of security issues labelled as "high", which in turn breaks the build. Most of the reported problems come from Openjdk version 8u131. Hence, this updated Dockerfile moves Openjdk to 8u141.
Additional apk repositories have been added in order to resolve all the required dependencies.
@tianon
Copy link
Member

tianon commented Aug 18, 2017

We've had a lot of trouble in the past trying to mix released Alpine versions with packages from Edge, so I'd be pretty strongly opposed to this change.

If there are security vulnerabilities in the existing package, it shouldn't be hard to convince the Alpine package maintainers to do an update in Alpine 3.6 directly (they've done so in the past for exactly that reason). 👍

@joelbirchler
Copy link

I agree this should be fixed upstream by moving openjdk8-141 to Alpine v3.6 or an edge docker image could be built.

CVE: https://nvd.nist.gov/vuln/detail/CVE-2017-10102
Alpine package: https://pkgs.alpinelinux.org/package/edge/community/x86/openjdk8

@tianon
Copy link
Member

tianon commented Jan 3, 2018

The OpenJDK 8 Alpine variants have since moved to Alpine 3.7, and thus 8u151 and this no longer applies. 👍

@tianon tianon closed this Jan 3, 2018
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

Successfully merging this pull request may close these issues.

None yet

3 participants