fibered_mysql2 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 48308ffbe571baaa919c2f198d761657f9f77b95a87f67c3ef7493433a3e4a6a
-  data.tar.gz: c5893a679a730bfd7e1a088a06f86b82d65c95da9b9171fcb419f27298ca268c
+  metadata.gz: 03d7285011a1fd36ca7d030a233bdeb46900e44c512161abd0241c653b5cabd4
+  data.tar.gz: 5ec6253f493c773a11207664df054dab32f5cfaf996fbb4706bb76d89cd04b1c
 SHA512:
-  metadata.gz: af9979eb4a038db1fcbec58dc5a88d617a71271c1588ff0717a4792f5442b3c59d82e9bc8d76b59821179099c87c8387a72bbbcdb096d10991320e7001c4bda8
-  data.tar.gz: 05a912e4b5504aebfb6a0ad5e8a29e72cb8e739066279982ead7aa89ca97b9308f2144ed6142833cba84ad7939c8ee0d364e71065606221ebe52cf4ee35ce0e1
+  metadata.gz: 890bb36f3cbd96021d0afa0d3ce06b1d02765afc8be3d7335d5b6234ec6b6e11b58a24b4b499d0332a2a01c778757dd5c880af6291e7f535a740e2da9be2ef62
+  data.tar.gz: 0cc5c22298ca1bd4c6c443738b6aa1c306d412894141660f1ce49e4dc592474c172c014073b0a6e088506d3422c8983b21b00a9bd8bd018fd75a1b0904f33782

data/CHANGELOG.md

@@ -4,6 +4,10 @@ Inspired by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.1.2] - 2021-06-16
+### Fixed
+- Added checking to be certain that @owner is never overwritten with a non-Fiber by another mixin.
+
 ## [0.1.1] - 2021-02-12
 ### Fixed
 - Fixed bug with Rails 5+ adapter where connections that have `steal!` called on them were not having their owner updated to the current Fiber, which would then cause an exception when trying to expire the connection (this showed up with the Rails 5 `ConnectionPool::Reaper` that reaps unused connections)
@@ -19,5 +23,6 @@ Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0
 - Added TravisCI unit test pipeline.
 - Added coverage reports via Coveralls.
 
+[0.1.2]: https://github.com/Invoca/fibered_mysql2/compare/v0.1.1..v0.1.2
 [0.1.1]: https://github.com/Invoca/fibered_mysql2/compare/v0.1.0..v0.1.1
 [0.1.0]: https://github.com/Invoca/fibered_mysql2/tree/v0.1.0

data/Gemfile

@@ -8,6 +8,7 @@ gemspec
 gem 'appraisal'
 gem 'bundler',    '~> 1.8'
 gem 'coveralls', require: false
+gem 'mimemagic',  '~> 0.3',     git: 'git@github.com:Invoca/mimemagic',                      ref: 'b084ce8d50c080f5a312156498be21a541fe72a2'
 gem 'mysql2',     '0.4.5'
 gem 'pry',        '~> 0.13'
 gem 'pry-byebug', '~> 3.9'

data/Gemfile.lock

@@ -1,7 +1,14 @@
+GIT
+  remote: git@github.com:Invoca/mimemagic
+  revision: b084ce8d50c080f5a312156498be21a541fe72a2
+  ref: b084ce8d50c080f5a312156498be21a541fe72a2
+  specs:
+    mimemagic (0.3.5)
+
 PATH
   remote: .
   specs:
-    fibered_mysql2 (0.1.1)
+    fibered_mysql2 (0.1.2)
       em-synchrony (~> 1.0)
       rails (>= 4.2, < 7)
 
@@ -63,14 +70,14 @@ GEM
       minitest (~> 5.1)
       tzinfo (~> 1.1)
       zeitwerk (~> 2.2, >= 2.2.2)
-    appraisal (2.3.0)
+    appraisal (2.4.0)
       bundler
       rake
       thor (>= 0.14.0)
     builder (3.2.4)
     byebug (11.1.3)
     coderay (1.1.3)
-    concurrent-ruby (1.1.7)
+    concurrent-ruby (1.1.8)
     coveralls (0.8.23)
       json (>= 1.8, < 3)
       simplecov (~> 0.16.1)
@@ -82,14 +89,14 @@ GEM
     docile (1.3.2)
     em-synchrony (1.0.6)
       eventmachine (>= 1.0.0.beta.1)
-    erubi (1.9.0)
+    erubi (1.10.0)
     eventmachine (1.2.7)
     globalid (0.4.2)
       activesupport (>= 4.2.0)
-    i18n (1.8.5)
+    i18n (1.8.7)
       concurrent-ruby (~> 1.0)
     json (2.3.1)
-    loofah (2.7.0)
+    loofah (2.8.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
     mail (2.7.1)
@@ -97,10 +104,9 @@ GEM
     marcel (0.3.3)
       mimemagic (~> 0.3.2)
     method_source (1.0.0)
-    mimemagic (0.3.5)
     mini_mime (1.0.2)
     mini_portile2 (2.4.0)
-    minitest (5.14.2)
+    minitest (5.14.3)
     mysql2 (0.4.5)
     nio4r (2.5.4)
     nokogiri (1.10.10)
@@ -173,12 +179,12 @@ GEM
     thread_safe (0.3.6)
     tins (1.25.0)
       sync
-    tzinfo (1.2.7)
+    tzinfo (1.2.9)
       thread_safe (~> 0.1)
     websocket-driver (0.7.3)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
-    zeitwerk (2.4.0)
+    zeitwerk (2.4.1)
 
 PLATFORMS
   ruby
@@ -188,6 +194,7 @@ DEPENDENCIES
   bundler (~> 1.8)
   coveralls
   fibered_mysql2!
+  mimemagic (~> 0.3)!
   mysql2 (= 0.4.5)
   pry (~> 0.13)
   pry-byebug (~> 3.9)
@@ -195,4 +202,4 @@ DEPENDENCIES
   rspec (~> 3.0)
 
 BUNDLED WITH
-   1.17.2
+   1.17.3

data/lib/active_record/connection_adapters/fibered_mysql2_adapter.rb

@@ -20,10 +20,10 @@ module FiberedMysql2
     def lease
       if in_use?
         msg = "Cannot lease connection, ".dup
-        if @owner == Fiber.current
+        if owner_fiber == Fiber.current
           msg << "it is already leased by the current fiber."
         else
-          msg << "it is already in use by a different fiber: #{@owner}. " \
+          msg << "it is already in use by a different fiber: #{owner_fiber}. " \
                   "Current fiber: #{Fiber.current}."
         end
         raise ::ActiveRecord::ActiveRecordError, msg
@@ -37,9 +37,9 @@ module FiberedMysql2
         # Because we are actively releasing connections from dead fibers, we only want
         # to enforce that we're expiring the current fibers connection, iff the owner
         # of the connection is still alive.
-        if @owner.alive? && @owner != Fiber.current
+        if owner_fiber.alive? && owner_fiber != Fiber.current
           raise ::ActiveRecord::ActiveRecordError, "Cannot expire connection, " \
-            "it is owned by a different fiber: #{@owner}. " \
+            "it is owned by a different fiber: #{owner_fiber}. " \
             "Current fiber: #{Fiber.current}."
         end
 
@@ -52,8 +52,8 @@ module FiberedMysql2
 
     def steal!
       if in_use?
-        if @owner != Fiber.current
-          pool.send :remove_connection_from_thread_cache, self, @owner
+        if owner_fiber != Fiber.current
+          pool.send :remove_connection_from_thread_cache, self, owner_fiber
 
           @owner = Fiber.current
         end
@@ -61,6 +61,13 @@ module FiberedMysql2
         raise ::ActiveRecord::ActiveRecordError, "Cannot steal connection, it is not currently leased."
       end
     end
+
+    private
+
+    def owner_fiber
+      @owner.is_a?(Fiber) or raise "@owner must be a Fiber! Found #{@owner.inspect}"
+      @owner
+    end
   end
 
   class FiberedMysql2Adapter < ::ActiveRecord::ConnectionAdapters::EMMysql2Adapter

data/lib/fibered_mysql2/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FiberedMysql2
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fibered_mysql2
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Invoca Development
-autorequire:
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-02-12 00:00:00.000000000 Z
+date: 2021-06-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: em-synchrony
@@ -44,7 +44,7 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '7'
-description:
+description: 
 email:
 - development@invoca.com
 executables: []
@@ -77,7 +77,7 @@ licenses: []
 metadata:
   allowed_push_host: https://rubygems.org
   homepage_uri: https://github.com/Invoca/fibered_mysql2
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -93,7 +93,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.1
-signing_key:
+signing_key: 
 specification_version: 4
 summary: An adapter for fibered mysql2
 test_files: []