Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package rubygem-activestorage-7.0 for
openSUSE:Factory checked in at 2022-10-12 18:25:01
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/rubygem-activestorage-7.0 (Old)
and /work/SRC/openSUSE:Factory/.rubygem-activestorage-7.0.new.2275 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "rubygem-activestorage-7.0"
Wed Oct 12 18:25:01 2022 rev:6 rq:1010046 version:7.0.4
Changes:
--------
---
/work/SRC/openSUSE:Factory/rubygem-activestorage-7.0/rubygem-activestorage-7.0.changes
2022-08-06 22:07:53.686610623 +0200
+++
/work/SRC/openSUSE:Factory/.rubygem-activestorage-7.0.new.2275/rubygem-activestorage-7.0.changes
2022-10-12 18:26:45.281962358 +0200
@@ -1,0 +2,21 @@
+Mon Oct 10 12:54:56 UTC 2022 - Stephan Kulow <co...@suse.com>
+
+updated to version 7.0.4
+ see installed CHANGELOG.md
+
+ ## Rails 7.0.4 (September 09, 2022) ##
+
+ * Fixes proxy downloads of files over 5MiB
+
+ Previously, trying to view and/or download files larger than 5mb stored
in
+ services like S3 via proxy mode could return corrupted files at around
+ 5.2mb or cause random halts in the download. Now,
+ `ActiveStorage::Blobs::ProxyController` correctly handles streaming these
+ larger files from the service to the client without any issues.
+
+ Fixes #44679
+
+ *Felipe Raul*
+
+
+-------------------------------------------------------------------
Old:
----
activestorage-7.0.3.1.gem
New:
----
activestorage-7.0.4.gem
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ rubygem-activestorage-7.0.spec ++++++
--- /var/tmp/diff_new_pack.qwlwHD/_old 2022-10-12 18:26:45.869963653 +0200
+++ /var/tmp/diff_new_pack.qwlwHD/_new 2022-10-12 18:26:45.873963661 +0200
@@ -24,7 +24,7 @@
#
Name: rubygem-activestorage-7.0
-Version: 7.0.3.1
+Version: 7.0.4
Release: 0
%define mod_name activestorage
%define mod_full_name %{mod_name}-%{version}
++++++ activestorage-7.0.3.1.gem -> activestorage-7.0.4.gem ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/CHANGELOG.md new/CHANGELOG.md
--- old/CHANGELOG.md 2022-07-12 19:30:32.000000000 +0200
+++ new/CHANGELOG.md 2022-09-09 20:42:22.000000000 +0200
@@ -1,3 +1,17 @@
+## Rails 7.0.4 (September 09, 2022) ##
+
+* Fixes proxy downloads of files over 5MiB
+
+ Previously, trying to view and/or download files larger than 5mb stored in
+ services like S3 via proxy mode could return corrupted files at around
+ 5.2mb or cause random halts in the download. Now,
+ `ActiveStorage::Blobs::ProxyController` correctly handles streaming these
+ larger files from the service to the client without any issues.
+
+ Fixes #44679
+
+ *Felipe Raul*
+
## Rails 7.0.3.1 (July 12, 2022) ##
* No changes.
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/app/controllers/concerns/active_storage/streaming.rb
new/app/controllers/concerns/active_storage/streaming.rb
--- old/app/controllers/concerns/active_storage/streaming.rb 2022-07-12
19:30:32.000000000 +0200
+++ new/app/controllers/concerns/active_storage/streaming.rb 2022-09-09
20:42:22.000000000 +0200
@@ -3,6 +3,7 @@
require "securerandom"
module ActiveStorage::Streaming
+ extend ActiveSupport::Concern
DEFAULT_BLOB_STREAMING_DISPOSITION = "inline"
include ActionController::DataStreaming
Binary files old/checksums.yaml.gz and new/checksums.yaml.gz differ
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/lib/active_storage/analyzer/image_analyzer/image_magick.rb
new/lib/active_storage/analyzer/image_analyzer/image_magick.rb
--- old/lib/active_storage/analyzer/image_analyzer/image_magick.rb
2022-07-12 19:30:32.000000000 +0200
+++ new/lib/active_storage/analyzer/image_analyzer/image_magick.rb
2022-09-09 20:42:22.000000000 +0200
@@ -10,9 +10,14 @@
private
def read_image
- download_blob_to_tempfile do |file|
+ begin
require "mini_magick"
+ rescue LoadError
+ logger.info "Skipping image analysis because the mini_magick gem
isn't installed"
+ return {}
+ end
+ download_blob_to_tempfile do |file|
image = instrument("mini_magick") do
MiniMagick::Image.new(file.path)
end
@@ -23,13 +28,10 @@
logger.info "Skipping image analysis because ImageMagick doesn't
support the file"
{}
end
+ rescue MiniMagick::Error => error
+ logger.error "Skipping image analysis due to an ImageMagick error:
#{error.message}"
+ {}
end
- rescue LoadError
- logger.info "Skipping image analysis because the mini_magick gem isn't
installed"
- {}
- rescue MiniMagick::Error => error
- logger.error "Skipping image analysis due to an ImageMagick error:
#{error.message}"
- {}
end
def rotated_image?(image)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/lib/active_storage/analyzer/image_analyzer/vips.rb
new/lib/active_storage/analyzer/image_analyzer/vips.rb
--- old/lib/active_storage/analyzer/image_analyzer/vips.rb 2022-07-12
19:30:32.000000000 +0200
+++ new/lib/active_storage/analyzer/image_analyzer/vips.rb 2022-09-09
20:42:22.000000000 +0200
@@ -10,9 +10,14 @@
private
def read_image
- download_blob_to_tempfile do |file|
+ begin
require "ruby-vips"
+ rescue LoadError
+ logger.info "Skipping image analysis because the ruby-vips gem isn't
installed"
+ return {}
+ end
+ download_blob_to_tempfile do |file|
image = instrument("vips") do
::Vips::Image.new_from_file(file.path, access: :sequential)
end
@@ -23,13 +28,10 @@
logger.info "Skipping image analysis because Vips doesn't support
the file"
{}
end
+ rescue ::Vips::Error => error
+ logger.error "Skipping image analysis due to an Vips error:
#{error.message}"
+ {}
end
- rescue LoadError
- logger.info "Skipping image analysis because the ruby-vips gem isn't
installed"
- {}
- rescue ::Vips::Error => error
- logger.error "Skipping image analysis due to an Vips error:
#{error.message}"
- {}
end
ROTATIONS = /Right-top|Left-bottom|Top-right|Bottom-left/
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/lib/active_storage/gem_version.rb
new/lib/active_storage/gem_version.rb
--- old/lib/active_storage/gem_version.rb 2022-07-12 19:30:32.000000000
+0200
+++ new/lib/active_storage/gem_version.rb 2022-09-09 20:42:22.000000000
+0200
@@ -9,8 +9,8 @@
module VERSION
MAJOR = 7
MINOR = 0
- TINY = 3
- PRE = "1"
+ TINY = 4
+ PRE = nil
STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".")
end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/metadata new/metadata
--- old/metadata 2022-07-12 19:30:32.000000000 +0200
+++ new/metadata 2022-09-09 20:42:22.000000000 +0200
@@ -1,14 +1,14 @@
--- !ruby/object:Gem::Specification
name: activestorage
version: !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
platform: ruby
authors:
- David Heinemeier Hansson
autorequire:
bindir: bin
cert_chain: []
-date: 2022-07-12 00:00:00.000000000 Z
+date: 2022-09-09 00:00:00.000000000 Z
dependencies:
- !ruby/object:Gem::Dependency
name: activesupport
@@ -16,56 +16,56 @@
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
type: :runtime
prerelease: false
version_requirements: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
- !ruby/object:Gem::Dependency
name: actionpack
requirement: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
type: :runtime
prerelease: false
version_requirements: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
- !ruby/object:Gem::Dependency
name: activejob
requirement: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
type: :runtime
prerelease: false
version_requirements: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
- !ruby/object:Gem::Dependency
name: activerecord
requirement: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
type: :runtime
prerelease: false
version_requirements: !ruby/object:Gem::Requirement
requirements:
- - '='
- !ruby/object:Gem::Version
- version: 7.0.3.1
+ version: 7.0.4
- !ruby/object:Gem::Dependency
name: marcel
requirement: !ruby/object:Gem::Requirement
@@ -198,10 +198,10 @@
- MIT
metadata:
bug_tracker_uri: https://github.com/rails/rails/issues
- changelog_uri:
https://github.com/rails/rails/blob/v7.0.3.1/activestorage/CHANGELOG.md
- documentation_uri: https://api.rubyonrails.org/v7.0.3.1/
+ changelog_uri:
https://github.com/rails/rails/blob/v7.0.4/activestorage/CHANGELOG.md
+ documentation_uri: https://api.rubyonrails.org/v7.0.4/
mailing_list_uri: https://discuss.rubyonrails.org/c/rubyonrails-talk
- source_code_uri: https://github.com/rails/rails/tree/v7.0.3.1/activestorage
+ source_code_uri: https://github.com/rails/rails/tree/v7.0.4/activestorage
rubygems_mfa_required: 'true'
post_install_message:
rdoc_options: []
