activerecord 6.0.6 → 6.0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c6ce12f937513b9efe3578fdbe01cdeee38dbb3d16d38058099f2ab1e1c7e71a
-  data.tar.gz: ec06c07afcbdd62043bce7bb57c0e3fca32658404088b706a2b4188e919e5bf7
+  metadata.gz: 62b3dcd186d829531c34dec576e3fd66b4cccb2406fe1e0f8acad98a98e60ba6
+  data.tar.gz: 36d39bff0326c50ab47c2f2b36626bce5d670a38c8e6a01a50176ff9f04f74b0
 SHA512:
-  metadata.gz: d77444fd1cafdcf149d0e64da878e9f756b5a836720fa49145f93520168ed709c52e6e03d7081c2a7916b11cd3f9a4973859257b3f02765c9e504253dbb85783
-  data.tar.gz: 1ef9d00fc16e11d028a464e437f285a3f3d4300295c6d947ef0c9b0e51f377f2015e60166ae4c0ef719f8593f3af151c275cc1377e982405df9dd978736c43cd
+  metadata.gz: 9c6294d1f7ac84e1c763ba79d91877f64af76daad98a44c820690d4b325efa87a6de87e257c2a9491a5ffcd0b0ce0e4f9ac453df28bdb11b389c4005e73e9139
+  data.tar.gz: b0977e39fb7156f7d3cb9940da3036987fc37d7d813d2c17ec465097dab254231236f5c2fb4854991320a0b6bd624ce860faf2db42dfc45fed20dfb743f0b038

data/CHANGELOG.md

@@ -1,3 +1,22 @@
+## Rails 6.0.6.1 (January 17, 2023) ##
+
+*   Make `sanitize_as_sql_comment` more strict
+
+    Though this method was likely never meant to take user input, it was
+    attempting sanitization. That sanitization could be bypassed with
+    carefully crafted input.
+
+    This commit makes the sanitization more robust by replacing any
+    occurrances of "/*" or "*/" with "/ *" or "* /". It also performs a
+    first pass to remove one surrounding comment to avoid compatibility
+    issues for users relying on the existing removal.
+
+    This also clarifies in the documentation of annotate that it should not
+    be provided user input.
+
+    [CVE-2023-22794]
+
+
 ## Rails 6.0.6 (September 09, 2022) ##
 
 *   Symbol is allowed by default for YAML columns

data/lib/active_record/connection_adapters/abstract/quoting.rb

@@ -139,7 +139,16 @@ module ActiveRecord
       end
 
       def sanitize_as_sql_comment(value) # :nodoc:
-        value.to_s.gsub(%r{ (/ (?: | \g<1>) \*) \+? \s* | \s* (\* (?: | \g<2>) /) }x, "")
+        # Sanitize a string to appear within a SQL comment
+        # For compatibility, this also surrounding "/*+", "/*", and "*/"
+        # charcacters, possibly with single surrounding space.
+        # Then follows that by replacing any internal "*/" or "/ *" with
+        # "* /" or "/ *"
+        comment = value.to_s.dup
+        comment.gsub!(%r{\A\s*/\*\+?\s?|\s?\*/\s*\Z}, "")
+        comment.gsub!("*/", "* /")
+        comment.gsub!("/*", "/ *")
+        comment
       end
 
       def column_name_matcher # :nodoc:

data/lib/active_record/gem_version.rb

@@ -10,7 +10,7 @@ module ActiveRecord
     MAJOR = 6
     MINOR = 0
     TINY  = 6
-    PRE   = nil
+    PRE   = "1"
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".")
   end

data/lib/active_record/relation/query_methods.rb

@@ -1000,6 +1000,8 @@ module ActiveRecord
     #   # SELECT "users"."name" FROM "users" /* selecting */ /* user */ /* names */
     #
     # The SQL block comment delimiters, "/*" and "*/", will be added automatically.
+    #
+    # Some escaping is performed, however untrusted user input should not be used.
     def annotate(*args)
       check_if_method_has_arguments!(:annotate, args)
       spawn.annotate!(*args)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord
 version: !ruby/object:Gem::Version
-  version: 6.0.6
+  version: 6.0.6.1
 platform: ruby
 authors:
 - David Heinemeier Hansson
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-09 00:00:00.000000000 Z
+date: 2023-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 6.0.6
+        version: 6.0.6.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 6.0.6
+        version: 6.0.6.1
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 6.0.6
+        version: 6.0.6.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 6.0.6
+        version: 6.0.6.1
 description: Databases on Rails. Build a persistent domain model by mapping database
   tables to Ruby classes. Strong conventions for associations, validations, aggregations,
   migrations, and testing come baked-in.
@@ -391,10 +391,10 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/rails/rails/issues
-  changelog_uri: https://github.com/rails/rails/blob/v6.0.6/activerecord/CHANGELOG.md
-  documentation_uri: https://api.rubyonrails.org/v6.0.6/
+  changelog_uri: https://github.com/rails/rails/blob/v6.0.6.1/activerecord/CHANGELOG.md
+  documentation_uri: https://api.rubyonrails.org/v6.0.6.1/
   mailing_list_uri: https://discuss.rubyonrails.org/c/rubyonrails-talk
-  source_code_uri: https://github.com/rails/rails/tree/v6.0.6/activerecord
+  source_code_uri: https://github.com/rails/rails/tree/v6.0.6.1/activerecord
   rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options:
@@ -413,7 +413,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
+rubygems_version: 3.4.3
 signing_key:
 specification_version: 4
 summary: Object-relational mapper framework (part of Rails).