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

Update jetty:9.2 image to 9.2.12.v20150709 #892

Merged
merged 1 commit into from
Jul 13, 2015
Merged

Update jetty:9.2 image to 9.2.12.v20150709 #892

merged 1 commit into from
Jul 13, 2015

Conversation

md5
Copy link
Contributor

@md5 md5 commented Jul 13, 2015

@yosifkit
Copy link
Member

Why no 9.3 release too? 😮

@md5
Copy link
Contributor Author

md5 commented Jul 13, 2015

The Jetty Project only did a release of 9.2 this time (cf. http://dev.eclipse.org/mhonarc/lists/jetty-announce/msg00081.html).

Thanks for checking though! 👍

@yosifkit
Copy link
Member

Oh, it was in the changeset and I got confused and delved too deeply. 🐑 🚧🐏

LGTM, Build test of #892; 8029efc (jetty):

$ url="https://raw.githubusercontent.com/docker-library/official-images/8029efc055363b66694c8571401d36f76be4a47b/library/jetty"
$ bashbrew build "$url"
Fetching jetty (git://github.com/appropriate/docker-jetty) ...
Processing jetty:9.3.0 ...
Processing jetty:9.3 ...
Processing jetty:9 ...
Processing jetty:9.3.0-jre8 ...
Processing jetty:9.3-jre8 ...
Processing jetty:9-jre8 ...
Processing jetty:latest ...
Processing jetty:jre8 ...
Processing jetty:9.2.12 ...
Processing jetty:9.2 ...
Processing jetty:9.2.12-jre8 ...
Processing jetty:9.2-jre8 ...
Processing jetty:9.2.12-jre7 ...
Processing jetty:9.2-jre7 ...
Processing jetty:9-jre7 ...
Processing jetty:jre7 ...
$ bashbrew list --uniq "$url" | xargs test/run.sh
testing jetty:9.3.0
    'utc' [1/3]...passed
    'cve-2014--shellshock' [2/3]...passed
    'no-hard-coded-passwords' [3/3]...passed
testing jetty:9.2.12
    'utc' [1/3]...passed
    'cve-2014--shellshock' [2/3]...passed
    'no-hard-coded-passwords' [3/3]...passed
testing jetty:9.2.12-jre7
    'utc' [1/3]...passed
    'cve-2014--shellshock' [2/3]...passed
    'no-hard-coded-passwords' [3/3]...passed

@tianon
Copy link
Member

tianon commented Jul 13, 2015

LGTM

tianon added a commit that referenced this pull request Jul 13, 2015
Update jetty:9.2 image to 9.2.12.v20150709
@tianon tianon merged commit 30e5187 into docker-library:master Jul 13, 2015
@md5
Copy link
Contributor Author

md5 commented Jul 13, 2015

@yosifkit Got it. The jetty:9.3 image was already coming from a more recent SHA than 7946910.

@md5 md5 deleted the jetty-9.2.12 branch July 13, 2015 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants