chef 11.12.0-x86-mingw32 → 11.12.2-x86-mingw32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e1715fff9e95389b926a226b9c8f26b908f0fafb
-  data.tar.gz: 2e579c47c42d4c7bfe53ef675823038383f4d349
+  metadata.gz: e8569627190d2efc4f966d9a016c1816bc865826
+  data.tar.gz: 158cc0a37657509979c45c548fc62554a9819444
 SHA512:
-  metadata.gz: 928b5de97c2254799f0abf4ac170f4ebe233b36d756d7ad8c7f5281e9911524113aa0329810e3373cc887ba64310d548e7c488f027692c7b00e789fd0c94db43
-  data.tar.gz: 6fe410164d43b2acaa99e0079286a89609768bdcd9433f6ddc8b8b00ca323c534f24b8d607f51643a7300a360ee0406df1950a461765f025a11383207e80e762
+  metadata.gz: ca943559b89c38f60aec250a527ce38f7d2aac0f76e4c1dab2de78986b7612660aa614d7f3cc4f09803446520d7f2698cd4aa7a705708e7585c8f2c77c71b13c
+  data.tar.gz: 00b109ada95a45d2782b105fd6c596aab4597638e8ae04c556c34b22a12c63630377ee80389841581b3aa8f90defff897f0e6d97c8166058d552af4cecb54c0f

data/lib/chef/cookbook/syntax_check.rb

@@ -247,7 +247,12 @@ class Chef
         tmp_stderr = $stderr = StringIO.new
         abs_path = File.expand_path(ruby_file)
         file_content = IO.read(abs_path)
-        RubyVM::InstructionSequence.new(file_content, ruby_file, abs_path, 0)
+        # We have to wrap this in a block so the user code evaluates in a
+        # similar context as what Chef does normally. Otherwise RubyVM
+        # will reject some common idioms, like using `return` to end evaluation
+        # of a recipe. See also CHEF-5199
+        wrapped_content = "Object.new.instance_eval do\n#{file_content}\nend\n"
+        RubyVM::InstructionSequence.new(wrapped_content, ruby_file, abs_path, 0)
         true
       rescue SyntaxError
         $stderr = old_stderr

data/lib/chef/http.rb

@@ -50,7 +50,10 @@ class Chef
       end
 
       def handle_chunk(next_chunk)
-        @stream_handlers.inject(next_chunk) do |chunk, handler|
+        # stream handlers handle responses so must be applied in reverse order
+        # (same as #apply_stream_complete_middleware or #apply_response_midddleware)
+        @stream_handlers.reverse.inject(next_chunk) do |chunk, handler|
+          Chef::Log.debug("Chef::HTTP::StreamHandler calling #{handler.class}#handle_chunk")
           handler.handle_chunk(chunk)
         end
       end
@@ -210,18 +213,21 @@ class Chef
 
     def apply_request_middleware(method, url, headers, data)
       middlewares.inject([method, url, headers, data]) do |req_data, middleware|
+        Chef::Log.debug("Chef::HTTP calling #{middleware.class}#handle_request")
         middleware.handle_request(*req_data)
       end
     end
 
     def apply_response_middleware(response, rest_request, return_value)
       middlewares.reverse.inject([response, rest_request, return_value]) do |res_data, middleware|
+        Chef::Log.debug("Chef::HTTP calling #{middleware.class}#handle_response")
         middleware.handle_response(*res_data)
       end
     end
 
     def apply_stream_complete_middleware(response, rest_request, return_value)
       middlewares.reverse.inject([response, rest_request, return_value]) do |res_data, middleware|
+        Chef::Log.debug("Chef::HTTP calling #{middleware.class}#handle_stream_complete")
         middleware.handle_stream_complete(*res_data)
       end
     end
@@ -253,14 +259,6 @@ class Chef
         end
         @last_response = response
 
-        Chef::Log.debug("---- HTTP Status and Header Data: ----")
-        Chef::Log.debug("HTTP #{response.http_version} #{response.code} #{response.msg}")
-
-        response.each do |header, value|
-          Chef::Log.debug("#{header}: #{value}")
-        end
-        Chef::Log.debug("---- End HTTP Status/Header Data ----")
-
         if response.kind_of?(Net::HTTPSuccess)
           [response, request, return_value]
         elsif response.kind_of?(Net::HTTPNotModified) # Must be tested before Net::HTTPRedirection because it's subclass.

data/lib/chef/http/basic_client.rb

@@ -61,7 +61,16 @@ class Chef
         base_headers.each do |name, value|
           Chef::Log.debug("#{name}: #{value}")
         end
+        Chef::Log.debug("---- End HTTP Request Header Data ----")
         http_client.request(http_request) do |response|
+          Chef::Log.debug("---- HTTP Status and Header Data: ----")
+          Chef::Log.debug("HTTP #{response.http_version} #{response.code} #{response.msg}")
+
+          response.each do |header, value|
+            Chef::Log.debug("#{header}: #{value}")
+          end
+          Chef::Log.debug("---- End HTTP Status/Header Data ----")
+
           yield response if block_given?
           # http_client.request may not have the return signature we want, so
           # force the issue:

data/lib/chef/version.rb

@@ -17,7 +17,7 @@
 
 class Chef
   CHEF_ROOT = File.dirname(File.expand_path(File.dirname(__FILE__)))
-  VERSION = '11.12.0'
+  VERSION = '11.12.2'
 end
 
 # NOTE: the Chef::Version class is defined in version_class.rb

data/spec/data/cookbooks/openldap/recipes/default.rb

@@ -1,3 +1,4 @@
+
 cat "blanket" do
   pretty_kitty true
 end

data/spec/data/cookbooks/openldap/recipes/return.rb

@@ -0,0 +1,2 @@
+# CHEF-5199 regression test.
+return nil

data/spec/unit/cookbook/syntax_check_spec.rb

@@ -24,7 +24,7 @@ describe Chef::Cookbook::SyntaxCheck do
   let(:cookbook_path) { File.join(CHEF_SPEC_DATA, 'cookbooks', 'openldap') }
   let(:syntax_check) { Chef::Cookbook::SyntaxCheck.new(cookbook_path) }
 
-  let(:open_ldap_cookbook_files) {
+  let(:open_ldap_cookbook_files) do
     %w{ attributes/default.rb 
         attributes/smokey.rb
         definitions/client.rb
@@ -32,8 +32,9 @@ describe Chef::Cookbook::SyntaxCheck do
         metadata.rb
         recipes/default.rb
         recipes/gigantor.rb
-        recipes/one.rb }.map{ |f| File.join(cookbook_path, f) }
-}
+        recipes/one.rb
+        recipes/return.rb }.map{ |f| File.join(cookbook_path, f) }
+  end
 
   before do
     Chef::Log.logger = Logger.new(StringIO.new)
@@ -41,7 +42,7 @@ describe Chef::Cookbook::SyntaxCheck do
 
     @attr_files = %w{default.rb smokey.rb}.map { |f| File.join(cookbook_path, 'attributes', f) }
     @defn_files = %w{client.rb server.rb}.map { |f| File.join(cookbook_path, 'definitions', f)}
-    @recipes = %w{default.rb gigantor.rb one.rb}.map { |f| File.join(cookbook_path, 'recipes', f) }
+    @recipes = %w{default.rb gigantor.rb one.rb return.rb}.map { |f| File.join(cookbook_path, 'recipes', f) }
     @ruby_files = @attr_files + @defn_files + @recipes + [File.join(cookbook_path, "metadata.rb")]
     basenames = %w{ helpers_via_partial_test.erb
                     helper_test.erb

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chef
 version: !ruby/object:Gem::Version
-  version: 11.12.0
+  version: 11.12.2
 platform: x86-mingw32
 authors:
 - Adam Jacob
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-08 00:00:00.000000000 Z
+date: 2014-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-config
@@ -1401,6 +1401,7 @@ files:
 - spec/data/cookbooks/openldap/recipes/default.rb
 - spec/data/cookbooks/openldap/recipes/gigantor.rb
 - spec/data/cookbooks/openldap/recipes/one.rb
+- spec/data/cookbooks/openldap/recipes/return.rb
 - spec/data/cookbooks/openldap/templates/default/all_windows_line_endings.erb
 - spec/data/cookbooks/openldap/templates/default/helper_test.erb
 - spec/data/cookbooks/openldap/templates/default/helpers_via_partial_test.erb