Hello community, here is the log from the commit of package rubygem-actionpack-4_2 for openSUSE:Factory checked in at 2015-07-05 18:02:27 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/rubygem-actionpack-4_2 (Old) and /work/SRC/openSUSE:Factory/.rubygem-actionpack-4_2.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "rubygem-actionpack-4_2" Changes: -------- --- /work/SRC/openSUSE:Factory/rubygem-actionpack-4_2/rubygem-actionpack-4_2.changes 2015-03-27 09:40:00.000000000 +0100 +++ /work/SRC/openSUSE:Factory/.rubygem-actionpack-4_2.new/rubygem-actionpack-4_2.changes 2015-07-05 18:02:30.000000000 +0200 @@ -1,0 +2,57 @@ +Fri Jun 26 04:29:34 UTC 2015 - co...@suse.com + +- updated to version 4.2.3 + see installed CHANGELOG.md + + ## Rails 4.2.3 (June 25, 2015) ## + + * Fix rake routes not showing the right format when + nesting multiple routes. + + See #18373. + + *Ravil Bayramgalin* + + * Fix regression where a gzip file response would have a Content-type, + even when it was a 304 status code. + + See #19271. + + *Kohei Suzuki* + + * Fix handling of empty X_FORWARDED_HOST header in raw_host_with_port + + Previously, an empty X_FORWARDED_HOST header would cause + Actiondispatch::Http:URL.raw_host_with_port to return nil, causing + Actiondispatch::Http:URL.host to raise a NoMethodError. + + *Adam Forsyth* + + * Fallback to `ENV['RAILS_RELATIVE_URL_ROOT']` in `url_for`. + + Fixed an issue where the `RAILS_RELATIVE_URL_ROOT` environment variable is not + prepended to the path when `url_for` is called. If `SCRIPT_NAME` (used by Rack) + is set, it takes precedence. + + Fixes #5122. + + *Yasyf Mohamedali* + + * Fix regression in functional tests. Responses should have default headers + assigned. + + See #18423. + + *Jeremy Kemper*, *Yves Senn* + +------------------------------------------------------------------- +Wed Jun 17 04:30:01 UTC 2015 - co...@suse.com + +- updated to version 4.2.2 + see installed CHANGELOG.md + + ## Rails 4.2.2 (June 16, 2015) ## + + * No Changes * + +------------------------------------------------------------------- Old: ---- actionpack-4.2.1.gem New: ---- actionpack-4.2.3.gem ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ rubygem-actionpack-4_2.spec ++++++ --- /var/tmp/diff_new_pack.tUQwqF/_old 2015-07-05 18:02:31.000000000 +0200 +++ /var/tmp/diff_new_pack.tUQwqF/_new 2015-07-05 18:02:31.000000000 +0200 @@ -24,7 +24,7 @@ # Name: rubygem-actionpack-4_2 -Version: 4.2.1 +Version: 4.2.3 Release: 0 %define mod_name actionpack %define mod_full_name %{mod_name}-%{version} ++++++ actionpack-4.2.1.gem -> actionpack-4.2.3.gem ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/CHANGELOG.md new/CHANGELOG.md --- old/CHANGELOG.md 2015-03-19 17:41:11.000000000 +0100 +++ new/CHANGELOG.md 2015-06-25 23:29:44.000000000 +0200 @@ -1,4 +1,51 @@ -## Rails 4.2.1 (March 19, 2014) ## +## Rails 4.2.3 (June 25, 2015) ## + +* Fix rake routes not showing the right format when + nesting multiple routes. + + See #18373. + + *Ravil Bayramgalin* + +* Fix regression where a gzip file response would have a Content-type, + even when it was a 304 status code. + + See #19271. + + *Kohei Suzuki* + +* Fix handling of empty X_FORWARDED_HOST header in raw_host_with_port + + Previously, an empty X_FORWARDED_HOST header would cause + Actiondispatch::Http:URL.raw_host_with_port to return nil, causing + Actiondispatch::Http:URL.host to raise a NoMethodError. + + *Adam Forsyth* + +* Fallback to `ENV['RAILS_RELATIVE_URL_ROOT']` in `url_for`. + + Fixed an issue where the `RAILS_RELATIVE_URL_ROOT` environment variable is not + prepended to the path when `url_for` is called. If `SCRIPT_NAME` (used by Rack) + is set, it takes precedence. + + Fixes #5122. + + *Yasyf Mohamedali* + +* Fix regression in functional tests. Responses should have default headers + assigned. + + See #18423. + + *Jeremy Kemper*, *Yves Senn* + + +## Rails 4.2.2 (June 16, 2015) ## + +* No Changes * + + +## Rails 4.2.1 (March 19, 2015) ## * Non-string authenticity tokens do not raise NoMethodError when decoding the masked token. Files old/checksums.yaml.gz and new/checksums.yaml.gz differ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_controller/metal/strong_parameters.rb new/lib/action_controller/metal/strong_parameters.rb --- old/lib/action_controller/metal/strong_parameters.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_controller/metal/strong_parameters.rb 2015-06-25 23:29:45.000000000 +0200 @@ -24,11 +24,13 @@ end end - # Raised when a supplied parameter is not expected. + # Raised when a supplied parameter is not expected and + # ActionController::Parameters.action_on_unpermitted_parameters + # is set to <tt>:raise</tt>. # # params = ActionController::Parameters.new(a: "123", b: "456") # params.permit(:c) - # # => ActionController::UnpermittedParameters: found unexpected keys: a, b + # # => ActionController::UnpermittedParameters: found unpermitted parameters: a, b class UnpermittedParameters < IndexError attr_reader :params # :nodoc: diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/http/response.rb new/lib/action_dispatch/http/response.rb --- old/lib/action_dispatch/http/response.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/http/response.rb 2015-06-25 23:29:45.000000000 +0200 @@ -115,10 +115,11 @@ # The underlying body, as a streamable object. attr_reader :stream - def initialize(status = 200, header = {}, body = []) + def initialize(status = 200, header = {}, body = [], options = {}) super() - header = merge_default_headers(header, self.class.default_headers) + default_headers = options.fetch(:default_headers, self.class.default_headers) + header = merge_default_headers(header, default_headers) self.body, self.header, self.status = body, header, status @@ -324,9 +325,7 @@ end def merge_default_headers(original, default) - return original unless default.respond_to?(:merge) - - default.merge(original) + default.respond_to?(:merge) ? default.merge(original) : original end def build_buffer(response, body) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/http/url.rb new/lib/action_dispatch/http/url.rb --- old/lib/action_dispatch/http/url.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/http/url.rb 2015-06-25 23:29:45.000000000 +0200 @@ -184,7 +184,7 @@ # Returns the \host for this request, such as "example.com". def raw_host_with_port - if forwarded = env["HTTP_X_FORWARDED_HOST"] + if forwarded = env["HTTP_X_FORWARDED_HOST"].presence forwarded.split(/,\s?/).last else env['HTTP_HOST'] || "#{env['SERVER_NAME'] || env['SERVER_ADDR']}:#{env['SERVER_PORT']}" diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/journey/formatter.rb new/lib/action_dispatch/journey/formatter.rb --- old/lib/action_dispatch/journey/formatter.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/journey/formatter.rb 2015-06-25 23:29:45.000000000 +0200 @@ -40,7 +40,7 @@ return [route.format(parameterized_parts), params] end - message = "No route matches #{Hash[constraints.sort].inspect}" + message = "No route matches #{Hash[constraints.sort_by{|k,v| k.to_s}].inspect}" message << " missing required keys: #{missing_keys.sort.inspect}" unless missing_keys.empty? raise ActionController::UrlGenerationError, message diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/journey/route.rb new/lib/action_dispatch/journey/route.rb --- old/lib/action_dispatch/journey/route.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/journey/route.rb 2015-06-25 23:29:45.000000000 +0200 @@ -36,7 +36,7 @@ def requirements # :nodoc: # needed for rails `rake routes` - path.requirements.merge(@defaults).delete_if { |_,v| + @defaults.merge(path.requirements).delete_if { |_,v| /.+?/ == v } end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/journey/router.rb new/lib/action_dispatch/journey/router.rb --- old/lib/action_dispatch/journey/router.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/journey/router.rb 2015-06-25 23:29:45.000000000 +0200 @@ -121,7 +121,6 @@ end def match_head_routes(routes, req) - routes.delete_if { |route| route.verb == // } head_routes = match_routes(routes, req) if head_routes.empty? diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/journey/routes.rb new/lib/action_dispatch/journey/routes.rb --- old/lib/action_dispatch/journey/routes.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/journey/routes.rb 2015-06-25 23:29:45.000000000 +0200 @@ -15,6 +15,10 @@ @simulator = nil end + def empty? + routes.empty? + end + def length routes.length end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/middleware/cookies.rb new/lib/action_dispatch/middleware/cookies.rb --- old/lib/action_dispatch/middleware/cookies.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/middleware/cookies.rb 2015-06-25 23:29:45.000000000 +0200 @@ -181,7 +181,7 @@ # to the Message{Encryptor,Verifier} allows us to handle the # (de)serialization step within the cookie jar, which gives us the # opportunity to detect and migrate legacy cookies. - module VerifyAndUpgradeLegacySignedMessage + module VerifyAndUpgradeLegacySignedMessage # :nodoc: def initialize(*args) super @legacy_verifier = ActiveSupport::MessageVerifier.new(@options[:secret_token], serializer: ActiveSupport::MessageEncryptor::NullSerializer) @@ -392,7 +392,7 @@ end end - class JsonSerializer + class JsonSerializer # :nodoc: def self.load(value) ActiveSupport::JSON.decode(value) end @@ -402,7 +402,7 @@ end end - module SerializedCookieJars + module SerializedCookieJars # :nodoc: MARSHAL_SIGNATURE = "\x04\x08".freeze protected diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/middleware/static.rb new/lib/action_dispatch/middleware/static.rb --- old/lib/action_dispatch/middleware/static.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/middleware/static.rb 2015-06-25 23:29:45.000000000 +0200 @@ -29,7 +29,7 @@ } if match = paths.detect { |p| - path = File.join(@root, p) + path = File.join(@root, p.force_encoding('UTF-8')) begin File.file?(path) && File.readable?(path) rescue SystemCallError @@ -48,6 +48,9 @@ if gzip_path && gzip_encoding_accepted?(env) env['PATH_INFO'] = gzip_path status, headers, body = @file_server.call(env) + if status == 304 + return [status, headers, body] + end headers['Content-Encoding'] = 'gzip' headers['Content-Type'] = content_type(path) else diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/railtie.rb new/lib/action_dispatch/railtie.rb --- old/lib/action_dispatch/railtie.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/railtie.rb 2015-06-25 23:29:45.000000000 +0200 @@ -40,6 +40,8 @@ ActionDispatch::Cookies::CookieJar.always_write_cookie = config.action_dispatch.always_write_cookie ActionDispatch.test_app = app + + ActionDispatch::Routing::RouteSet.relative_url_root = app.config.relative_url_root end end end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/routing/mapper.rb new/lib/action_dispatch/routing/mapper.rb --- old/lib/action_dispatch/routing/mapper.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/routing/mapper.rb 2015-06-25 23:29:45.000000000 +0200 @@ -291,7 +291,7 @@ when String ActiveSupport::Deprecation.warn(<<-MSG.squish) Defining a route where `to` is a controller without an action is deprecated. - Please change `to: :#{to}` to `controller: :#{to}`. + Please change `to: '#{to}'` to `controller: '#{to}'`. MSG [to, nil] diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/routing/route_set.rb new/lib/action_dispatch/routing/route_set.rb --- old/lib/action_dispatch/routing/route_set.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/routing/route_set.rb 2015-06-25 23:29:45.000000000 +0200 @@ -21,6 +21,8 @@ # alias inspect to to_s. alias inspect to_s + mattr_accessor :relative_url_root + class Dispatcher < Routing::Endpoint def initialize(defaults) @defaults = defaults @@ -252,7 +254,7 @@ end def raise_generation_error(args, missing_keys) - constraints = Hash[@route.requirements.merge(args).sort] + constraints = Hash[@route.requirements.merge(args).sort_by{|k,v| k.to_s}] message = "No route matches #{constraints.inspect}" message << " missing required keys: #{missing_keys.sort.inspect}" @@ -768,7 +770,7 @@ end def find_script_name(options) - options.delete(:script_name) || '' + options.delete(:script_name) || relative_url_root || '' end def path_for(options, route_name = nil) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/testing/assertions.rb new/lib/action_dispatch/testing/assertions.rb --- old/lib/action_dispatch/testing/assertions.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/testing/assertions.rb 2015-06-25 23:29:45.000000000 +0200 @@ -12,7 +12,7 @@ include Rails::Dom::Testing::Assertions def html_document - @html_document ||= if @response.content_type =~ /xml$/ + @html_document ||= if @response.content_type === Mime::XML Nokogiri::XML::Document.parse(@response.body) else Nokogiri::HTML::Document.parse(@response.body) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/testing/integration.rb new/lib/action_dispatch/testing/integration.rb --- old/lib/action_dispatch/testing/integration.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/testing/integration.rb 2015-06-25 23:29:45.000000000 +0200 @@ -299,7 +299,7 @@ @request_count += 1 @request = ActionDispatch::Request.new(session.last_request.env) response = _mock_session.last_response - @response = ActionDispatch::TestResponse.new(response.status, response.headers, response.body) + @response = ActionDispatch::TestResponse.from_response(response) @html_document = nil @url_options = nil diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_dispatch/testing/test_response.rb new/lib/action_dispatch/testing/test_response.rb --- old/lib/action_dispatch/testing/test_response.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_dispatch/testing/test_response.rb 2015-06-25 23:29:45.000000000 +0200 @@ -7,11 +7,7 @@ # See Response for more information on controller response objects. class TestResponse < Response def self.from_response(response) - new.tap do |resp| - resp.status = response.status - resp.headers = response.headers - resp.body = response.body - end + new response.status, response.headers, response.body, default_headers: nil end # Was the response successful? @@ -25,12 +21,5 @@ # Was there a server-side error? alias_method :error?, :server_error? - - def merge_default_headers(original, *args) - # Default headers are already applied, no need to merge them a second time. - # This makes sure that default headers, removed in controller actions, will - # not be reapplied to the test response. - original - end end end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/action_pack/gem_version.rb new/lib/action_pack/gem_version.rb --- old/lib/action_pack/gem_version.rb 2015-03-19 17:41:11.000000000 +0100 +++ new/lib/action_pack/gem_version.rb 2015-06-25 23:29:45.000000000 +0200 @@ -7,7 +7,7 @@ module VERSION MAJOR = 4 MINOR = 2 - TINY = 1 + TINY = 3 PRE = nil STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".") diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/metadata new/metadata --- old/metadata 2015-03-19 17:41:11.000000000 +0100 +++ new/metadata 2015-06-25 23:29:44.000000000 +0200 @@ -1,14 +1,14 @@ --- !ruby/object:Gem::Specification name: actionpack version: !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 platform: ruby authors: - David Heinemeier Hansson autorequire: bindir: bin cert_chain: [] -date: 2015-03-19 00:00:00.000000000 Z +date: 2015-06-25 00:00:00.000000000 Z dependencies: - !ruby/object:Gem::Dependency name: activesupport @@ -16,14 +16,14 @@ requirements: - - '=' - !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 type: :runtime prerelease: false version_requirements: !ruby/object:Gem::Requirement requirements: - - '=' - !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 - !ruby/object:Gem::Dependency name: rack requirement: !ruby/object:Gem::Requirement @@ -61,7 +61,7 @@ version: '1.0' - - ">=" - !ruby/object:Gem::Version - version: 1.0.1 + version: 1.0.2 type: :runtime prerelease: false version_requirements: !ruby/object:Gem::Requirement @@ -71,7 +71,7 @@ version: '1.0' - - ">=" - !ruby/object:Gem::Version - version: 1.0.1 + version: 1.0.2 - !ruby/object:Gem::Dependency name: rails-dom-testing requirement: !ruby/object:Gem::Requirement @@ -98,28 +98,28 @@ requirements: - - '=' - !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 type: :runtime prerelease: false version_requirements: !ruby/object:Gem::Requirement requirements: - - '=' - !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 - !ruby/object:Gem::Dependency name: activemodel requirement: !ruby/object:Gem::Requirement requirements: - - '=' - !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 type: :development prerelease: false version_requirements: !ruby/object:Gem::Requirement requirements: - - '=' - !ruby/object:Gem::Version - version: 4.2.1 + version: 4.2.3 description: Web apps on Rails. Simple, battle-tested conventions for building and testing MVC web applications. Works with any Rack-compatible server. email: da...@loudthinking.com