nonnative 1.37.0 → 1.38.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7bcf9cffec3971770b2593b373938a6c084ab34381228bb2d9e43f794fb87d4a
-  data.tar.gz: f1ef1bbb290b1a28d6036d06af5bf1230e32c22f4ea8e075c240480e807b212d
+  metadata.gz: 66174bae17db6e0fae12b8d0be37b1373c887687901a072e32914cb00bea1f08
+  data.tar.gz: a7430fe7c5d8bfd4a5a43fcf733fe80ef8fb1c8a25e78a2e22c5847518c547e7
 SHA512:
-  metadata.gz: 8efcbbbbccf80e445d95c94207e521c8525a835abf544dd0f47ac37dad2e9af02ba2bd1af86a0475bc0c9c3db2f8d7d1b40be7487cf33ec7dbcdd4661125af34
-  data.tar.gz: 1e69e4fd86e440c75b2ba4abf7eb1e929f3129e4d01b1a852ea9d0966e35e081ce6a4b40bb69ae3a984c1c36a743ae2b7c85f9af04a0fba5639afd451e636388
+  metadata.gz: 3b57cbc993f69dd8b051dcd51ccf2953dff7bede798fae49c75439e9d47b8dac8474f391d3050ec71af706bf75febfb681c65ddb247cbf7c87bc7d3d3f3e153f
+  data.tar.gz: 30f89fcd78a1a5eedb130f4707bb60246db7ad98df733cf13e6f39146a25c955d2664fb407e65ed318de28ab61b7f222a9e380abd50349c446ab1a7c39952953

data/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [1.38.0](https://github.com/alexfalkowski/nonnative/compare/v1.37.0...v1.38.0) (2021-04-22)
+
+
+### Features
+
+* terminate connections on change ([#74](https://github.com/alexfalkowski/nonnative/issues/74)) ([6c1cc8e](https://github.com/alexfalkowski/nonnative/commit/6c1cc8e1c3f15d11848889852897b523f5848794))
+
+## [1.37.0](https://github.com/alexfalkowski/nonnative/compare/v1.36.0...v1.37.0) (2021-04-22)
+
+
+### Features
+
+* simplify go exeutable ([#73](https://github.com/alexfalkowski/nonnative/issues/73)) ([95e4cf4](https://github.com/alexfalkowski/nonnative/commit/95e4cf44180ee84d32cc42ee058d6742732b57b9))
+
 ## [1.36.0](https://github.com/alexfalkowski/nonnative/compare/v1.35.2...v1.36.0) (2021-04-21)
 
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    nonnative (1.37.0)
+    nonnative (1.38.0)
       concurrent-ruby (~> 1.0, >= 1.0.5)
       cucumber (>= 5, < 6)
       grpc (>= 1, < 2)

data/lib/nonnative/close_all_socket_pair.rb

@@ -3,6 +3,7 @@
 module Nonnative
   class CloseAllSocketPair < SocketPair
     def connect(local_socket)
+    ensure
       local_socket.close
     end
   end

data/lib/nonnative/fault_injection_proxy.rb

@@ -48,14 +48,7 @@ module Nonnative
     def perform_start
       loop do
         thread = Thread.start(tcp_server.accept) do |local_socket|
-          id = Thread.current.object_id
-
-          logger.info "started connection for #{id} with socket #{local_socket.inspect}"
-
-          connect local_socket
-          connections.delete(id)
-
-          logger.info "finished connection for #{id} with socket #{local_socket.inspect}"
+          accept_connection local_socket
         end
 
         thread.report_on_exception = false
@@ -63,12 +56,32 @@ module Nonnative
       end
     end
 
+    def accept_connection(local_socket)
+      id = Thread.current.object_id
+      socket_info = local_socket.inspect
+
+      connect local_socket
+      connections.delete(id)
+    ensure
+      logger.info "handled connection for #{id} with socket #{socket_info}"
+    end
+
     def connect(local_socket)
-      SocketPairFactory.create(read_state, service.proxy, logger).connect(local_socket)
+      pair = SocketPairFactory.create(read_state, service.proxy)
+
+      pair.connect(local_socket)
+    end
+
+    def close_connections
+      connections.each_value(&:terminate)
+      connections.clear
     end
 
     def apply_state(state)
-      mutex.synchronize { @state = state }
+      mutex.synchronize do
+        @state = state
+        close_connections
+      end
     end
 
     def read_state

data/lib/nonnative/socket_pair.rb

@@ -2,9 +2,8 @@
 
 module Nonnative
   class SocketPair
-    def initialize(proxy, logger)
+    def initialize(proxy)
       @proxy = proxy
-      @logger = logger
     end
 
     def connect(local_socket)
@@ -16,8 +15,6 @@ module Nonnative
         break if pipe(ready, local_socket, remote_socket)
         break if pipe(ready, remote_socket, local_socket)
       end
-    rescue StandardError => e
-      logger.error e
     ensure
       local_socket.close
       remote_socket&.close
@@ -25,7 +22,7 @@ module Nonnative
 
     protected
 
-    attr_reader :proxy, :logger
+    attr_reader :proxy
 
     def create_remote_socket
       ::TCPSocket.new('0.0.0.0', proxy.port)

data/lib/nonnative/socket_pair_factory.rb

@@ -3,7 +3,7 @@
 module Nonnative
   class SocketPairFactory
     class << self
-      def create(type, proxy, logger)
+      def create(type, proxy)
         pair = case type
                when :close_all
                  CloseAllSocketPair
@@ -15,7 +15,7 @@ module Nonnative
                  SocketPair
                end
 
-        pair.new(proxy, logger)
+        pair.new(proxy)
       end
     end
   end

data/lib/nonnative/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Nonnative
-  VERSION = '1.37.0'
+  VERSION = '1.38.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nonnative
 version: !ruby/object:Gem::Version
-  version: 1.37.0
+  version: 1.38.0
 platform: ruby
 authors:
 - Alex Falkowski