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

Address CVE-2014-3628 #571

Merged
merged 2 commits into from
Nov 3, 2015
Merged

Address CVE-2014-3628 #571

merged 2 commits into from
Nov 3, 2015

Conversation

pburkholder
Copy link
Contributor

@stevendanna
Copy link
Contributor

@pburkholder Thanks for this. A couple of todo's based on the process in CONTRIBUTING.md:

  1. Include a ChangeLog-Entry in your commit message
  2. Run this build through wilson.ci.opscode.us using chef-server-12-ad_hoc with your branch name. (Ping in chat if you need help).

Also, do you know if we are actively affected by that CVE or is this a "we are out of date and there are CVEs against the version we are on"-type update. Either way we should update but it does affect what we say about it in the release notes.

@irvingpop
Copy link

@irvingpop
Copy link

That build failed because the s3 cache entry was missing for Solr 4.10.4 ( bin/omnibus cache populate fix that).

@pburkholder I rebased this branch (heads up, I force pushed) and am sending a new build through wilson right now: http://wilson.ci.chef.co/job/chef-server-12-build/1516/

@irvingpop
Copy link

Validated this build under load (via ec-metal) - upgrade from an earlier build works, everything looks good.

paging @chef/lob for review

@stevendanna
Copy link
Contributor

I'm 👍 thanks for the testing @irvingpop

@sdelano
Copy link
Contributor

sdelano commented Nov 3, 2015

👍 - let's get the changelog updated too

irvingpop pushed a commit that referenced this pull request Nov 3, 2015
Changelog-Entry: Update Solr to 4.10.4 to address CVE-2014-3628
@irvingpop irvingpop merged commit a38ffda into master Nov 3, 2015
@irvingpop irvingpop deleted the pdb/CVE-2014-3628 branch November 3, 2015 00:29
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.

5 participants