exceptional_synchrony 1.4.0.pre.2 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7337c98f6e124b7e27327707ad7996bea893fe5c005ae57627e855ec35c77ff4
-  data.tar.gz: 2603dc05990ab4c3f257409382f8fc6453f8e22446a82a1ad9841564b509994e
+  metadata.gz: f2f574bdd917d5a7cab99f7844581ad87a86666f29678aa125aa68fdfb709d35
+  data.tar.gz: 68d9c838bcf933a4edb2c51dd0701e36d24dd99a2faf5aa526ce038541b0163b
 SHA512:
-  metadata.gz: bd0b2894a70161f7162c465aacf315cada6748267d88590bb943d3c68ad382f0ed1b910224738f54b1a1ada219f894b9fd64bcb2f06c7f3b95f0c9b3efb1d218
-  data.tar.gz: b9b7c5fde46b71d32651108b855f4396517e32a73915b1e3bc09e258a5bcee678d3946ca3fc6c768eef1deebc0edaa66e4e41898b3a08a6d95e993c0bc12339e
+  metadata.gz: 6b963e314d96e4e0ed269f6c532db13ae599a9f9478858a5087cea2cfc2fa85b011344cc4051da7ccdbb1a1142a627cb3c776a72c8cbacd4af99feede18a32e3
+  data.tar.gz: 51316500edb4ce824d3c19df94c81e0f5f2411411110f70d80b9ac31447b46aaea61c0efb60034aaac7f9ab4f58c689f83060346952deaee7a8b96476f12fc85

data/CHANGELOG.md

@@ -6,10 +6,18 @@ Note: This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0
 
 All notable changes to this project will be documented in this file.
 
-## [1.4.0] - UNRELEASED
+## [1.4.2] - 2021-03-10
+### Fixed
+- Additional symbol reference fixes in FaradayAdapterPatch_v1
+
+## [1.4.1] - 2021-03-09
+### Fixed
+- FaradayAdapterPatch_v1 name typo
+
+## [1.4.0] - 2021-03-08
 ### Added
-- Added `:faraday_adapter` option to `EventMachineProxy#run` with a default of `:em_synchrony`
-  so that the reactor does not get blocked when using `Faraday` connections
+- Added use of Thread local variable to indicate when Eventmachine is running using EM::Synchrony
+- Added faraday gem monkey patch to use the new Thread local variable to choose the adapter to use
 
 ## [1.3.0] - 2021-02-04
 ### Added
@@ -29,6 +37,9 @@ All notable changes to this project will be documented in this file.
 ## [1.1.1] - 2020-05-03
 - Replace hobo_support with invoca_utils
 
+[1.4.2]: https://github.com/Invoca/exceptional_synchrony/compare/v1.4.1...v1.4.2
+[1.4.1]: https://github.com/Invoca/exceptional_synchrony/compare/v1.4.0...v1.4.1
+[1.4.0]: https://github.com/Invoca/exceptional_synchrony/compare/v1.3.0...v1.4.0
 [1.3.0]: https://github.com/Invoca/exceptional_synchrony/compare/v1.2.0...v1.3.0
 [1.2.0]: https://github.com/Invoca/exceptional_synchrony/compare/v1.1.1...v1.2.0
 [1.1.1]: https://github.com/Invoca/exceptional_synchrony/compare/v1.1.0...v1.1.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    exceptional_synchrony (1.4.0.pre.2)
+    exceptional_synchrony (1.4.2)
       em-http-request
       em-synchrony
       eventmachine
@@ -11,30 +11,30 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    actionmailer (6.1.1)
-      actionpack (= 6.1.1)
-      actionview (= 6.1.1)
-      activejob (= 6.1.1)
-      activesupport (= 6.1.1)
+    actionmailer (6.1.3)
+      actionpack (= 6.1.3)
+      actionview (= 6.1.3)
+      activejob (= 6.1.3)
+      activesupport (= 6.1.3)
       mail (~> 2.5, >= 2.5.4)
       rails-dom-testing (~> 2.0)
-    actionpack (6.1.1)
-      actionview (= 6.1.1)
-      activesupport (= 6.1.1)
+    actionpack (6.1.3)
+      actionview (= 6.1.3)
+      activesupport (= 6.1.3)
       rack (~> 2.0, >= 2.0.9)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actionview (6.1.1)
-      activesupport (= 6.1.1)
+    actionview (6.1.3)
+      activesupport (= 6.1.3)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activejob (6.1.1)
-      activesupport (= 6.1.1)
+    activejob (6.1.3)
+      activesupport (= 6.1.3)
       globalid (>= 0.3.6)
-    activesupport (6.1.1)
+    activesupport (6.1.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -64,19 +64,21 @@ GEM
     em-synchrony (1.0.6)
       eventmachine (>= 1.0.0.beta.1)
     erubi (1.10.0)
+    escalate (0.3.0)
     eventmachine (1.2.7)
-    exception_handling (2.8.1)
+    exception_handling (2.9.0)
       actionmailer (>= 4.2, < 7.0)
       actionpack (>= 4.2, < 7.0)
       activesupport (>= 4.2, < 7.0)
       contextual_logger (~> 0.7)
+      escalate (~> 0.2)
       eventmachine (~> 1.0)
       invoca-utils (~> 0.3)
     globalid (0.4.2)
       activesupport (>= 4.2.0)
     hashdiff (1.0.1)
     http_parser.rb (0.6.0)
-    i18n (1.8.8)
+    i18n (1.8.9)
       concurrent-ruby (~> 1.0)
     invoca-utils (0.4.1)
     json (2.5.1)

data/lib/exceptional_synchrony/event_machine_proxy.rb

@@ -3,6 +3,7 @@
 require 'eventmachine'
 require 'em-http'
 require 'em-synchrony/em-http'
+require_relative 'faraday_monkey_patch'
 
 module ExceptionalSynchrony
   # It is important for this exception to be inherited from Exception so that
@@ -62,6 +63,7 @@ module ExceptionalSynchrony
     def stop
       @proxy_class.stop
       @proxy_class.next_tick { } #Fake out EventMachine's epoll mechanism so we don't block until timers fire
+      Thread.current.thread_variable_set(:em_synchrony_reactor_thread, false)
     end
 
     def defers_finished?
@@ -76,12 +78,7 @@ module ExceptionalSynchrony
     # The on_error option has these possible values:
     #   :log   - log any rescued StandardError exceptions and continue
     #   :raise - raise FatalRunError for any rescued StandardError exceptions
-    # The faraday_adapter option has these possible values that only apply if Faraday connections are in use:
-    #   :em_synchrony - for use when EM::Synchrony is being used in all threads of the given process
-    #   :net_http     - for use when EM::Synchrony is being used in only some threads of the given process;
-    #                   in this case requests made over the Faraday connection will block the reactor
-    def run(on_error: :log, faraday_adapter: :em_synchrony, &block)
-      configure_faraday(faraday_adapter)
+    def run(on_error: :log, &block)
       case on_error
       when :log   then run_with_error_logging(&block)
       when :raise then run_with_error_raising(&block)
@@ -139,16 +136,10 @@ module ExceptionalSynchrony
 
     private
 
-    def configure_faraday(adapter)
-      if defined?(Faraday)
-        adapter.in?([:em_synchrony, :net_http]) or raise ArgumentError, "Invalid faraday_adapter: #{adapter.inspect}"
-        Faraday.default_adapter = adapter
-      end
-    end
-
     def run_with_error_logging(&block)
       ensure_completely_safe("run_with_error_logging") do
         if @proxy_class.respond_to?(:synchrony)
+          Thread.current.thread_variable_set(:em_synchrony_reactor_thread, true)
           @proxy_class.synchrony(&block)
         else
           @proxy_class.run(&block)
@@ -161,6 +152,7 @@ module ExceptionalSynchrony
 
       rescue_exceptions_and_ensure_exit("run_with_error_raising") do
         if @proxy_class.respond_to?(:synchrony)
+          Thread.current.thread_variable_set(:em_synchrony_reactor_thread, true)
           @proxy_class.synchrony(&run_block)
         else
           @proxy_class.run(&run_block)

data/lib/exceptional_synchrony/faraday_monkey_patch.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+# Monkey patch for the Faraday method that creates the adapter used for a connection.
+# If the thread local variable :em_synchrony_reactor_thread is true, it overrides this method
+# in order to force use of the :em_synchrony adapter rather than the :net_http adapter.
+# This ensures that the Eventmachine reactor does not get blocked by connection i/o.
+begin
+  require 'faraday'
+
+  module ExceptionalSynchrony
+    # Patch built relative to faraday v0.17.3
+    module FaradayAdapterPatch_v0
+      def adapter(key, *args, &block)
+
+        # BEGIN PATCH
+        if key == :net_http && Thread.current.thread_variable_get(:em_synchrony_reactor_thread)
+          key = :em_synchrony
+        end
+        # END PATCH
+
+        use_symbol(Faraday::Adapter, key, *args, &block)
+      end
+    end
+
+    # Patch built relative to faraday v1.3.0 although the ruby2_keywords prefix
+    # was dropped from the adapter method definition to simplify this code
+    module FaradayAdapterPatch_v1
+      def adapter(klass = Faraday::RackBuilder::NO_ARGUMENT, *args, &block)
+        return @adapter if klass == Faraday::RackBuilder::NO_ARGUMENT
+
+        klass = Faraday::Adapter.lookup_middleware(klass) if klass.is_a?(Symbol)
+
+        # BEGIN PATCH
+        if klass == Faraday::Adapter::NetHttp && Thread.current.thread_variable_get(:em_synchrony_reactor_thread)
+          klass = Faraday::Adapter::EMSynchrony
+        end
+        # END PATCH
+
+        @adapter = Faraday::RackBuilder::Handler.new(klass, *args, &block)
+      end
+    end
+  end
+
+  if Faraday::VERSION.start_with?("0")
+    Faraday::RackBuilder.prepend ExceptionalSynchrony::FaradayAdapterPatch_v0
+  else
+    Faraday::RackBuilder.prepend ExceptionalSynchrony::FaradayAdapterPatch_v1
+  end
+
+rescue LoadError
+  # Monkey patch is not needed if faraday is not available
+end

data/lib/exceptional_synchrony/version.rb

@@ -1,3 +1,3 @@
 module ExceptionalSynchrony
-  VERSION = '1.4.0.pre.2'
+  VERSION = '1.4.2'
 end

data/test/unit/event_machine_proxy_test.rb

@@ -64,6 +64,14 @@ describe ExceptionalSynchrony::EventMachineProxy do
     @em.stop
   end
 
+  it "should set thread variable :em_synchrony_reactor_thread running to false when stop" do
+    @em.run do
+      assert_equal true, Thread.current.thread_variable_get(:em_synchrony_reactor_thread)
+      @em.stop
+      assert_equal false, Thread.current.thread_variable_get(:em_synchrony_reactor_thread)
+    end
+  end
+
   it "should proxy connect" do
     ServerClass = Class.new
     mock(EventMachine).connect(ServerClass, 8080, :handler, :extra_arg).yields(:called)
@@ -158,42 +166,6 @@ describe ExceptionalSynchrony::EventMachineProxy do
     end
   end
 
-  describe "run with faraday" do
-    before do
-      class Faraday
-        class << self
-          attr_reader :default_adapter
-
-          def default_adapter=(adapter)
-            @default_adapter = adapter
-          end
-        end
-
-        self.default_adapter = :net_http
-      end
-
-      assert_equal :net_http, Faraday.default_adapter
-    end
-
-    it "sets Faraday default_adapter to :em_synchrony by default if Faraday is defined" do
-      mock(@em).run_with_error_logging
-      @em.run
-      assert_equal :em_synchrony, Faraday.default_adapter
-    end
-
-    it "sets Faraday default_adapter to :net_http if Faraday is defined" do
-      mock(@em).run_with_error_logging
-      @em.run(faraday_adapter: :net_http)
-      assert_equal :net_http, Faraday.default_adapter
-    end
-
-    it "raise ArgumentError if the specified faraday_adapter is invalid" do
-      assert_raises(ArgumentError, "Invalid faraday_adapter: :bogus") do
-        @em.run(faraday_adapter: :bogus)
-      end
-    end
-  end
-
   { synchrony: SynchronyProxyMock, run: RunProxyMock }.each do |method, proxy_mock|
     describe "run" do
       before do
@@ -207,6 +179,10 @@ describe ExceptionalSynchrony::EventMachineProxy do
       end
 
       describe "without error" do
+        before do
+          Thread.current.thread_variable_set(:em_synchrony_reactor_thread, nil)
+        end
+
         [:log, :raise].each do |on_error|
           describe "when using #{method} and on_error = #{on_error}" do
             it "should dispatch to the proxy's synchrony method instead of run iff synchrony" do
@@ -214,6 +190,15 @@ describe ExceptionalSynchrony::EventMachineProxy do
               assert_equal method, (@proxy.run(on_error: on_error) { dispatched = true })
               assert_equal true, dispatched
             end
+
+            if method == :synchrony
+              it "should set thread variable :em_synchrony_reactor_thread running to true" do
+                assert_nil Thread.current.thread_variable_get(:em_synchrony_reactor_thread)
+                @proxy.run(on_error: on_error) do
+                  assert_equal true, Thread.current.thread_variable_get(:em_synchrony_reactor_thread)
+                end
+              end
+            end
           end
         end
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: exceptional_synchrony
 version: !ruby/object:Gem::Version
-  version: 1.4.0.pre.2
+  version: 1.4.2
 platform: ruby
 authors:
 - Invoca
@@ -101,6 +101,7 @@ files:
 - lib/exceptional_synchrony.rb
 - lib/exceptional_synchrony/callback_exceptions.rb
 - lib/exceptional_synchrony/event_machine_proxy.rb
+- lib/exceptional_synchrony/faraday_monkey_patch.rb
 - lib/exceptional_synchrony/limited_work_queue.rb
 - lib/exceptional_synchrony/parallel_sync.rb
 - lib/exceptional_synchrony/version.rb
@@ -126,9 +127,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.0.3
 signing_key: