dagger 2.0.0 → 2.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 447d77e66ec69445f5040231d19c038279d4ec7de624461deda078c544423ca9
-  data.tar.gz: c5eb814e71fd6a37b3b4272f6f747654eeb157655dec65ae6ec6f0cf86d4e5ff
+  metadata.gz: 5942c147bb4bcf52ad8d1aa04cb5a44010eb15a6c85877a71b68872d745710c2
+  data.tar.gz: 14654c7a309a5719f0b23d136f62f67a6e995b9ac0d6dff0e697b628acf57f35
 SHA512:
-  metadata.gz: 31108057fbc3cd1d2e0c319d4c64244c596cbbb90d8ae6067945caedb2f8f7ff5c0fd144b16be55ac24cd36bbd466f08659b995b47ecd67425d26224bd505ede
-  data.tar.gz: d996eaefa87c30682eb127d9f2f28cbb28363fcf2a2f758e32610d553937c05a35750316f043a6f4d2a9eb6727aad8f0351ca761ee62b326313b46e902764fd2
+  metadata.gz: 4f71ee8900cc198365cb0ec64176b0861ebe9e8c295a407b048d6c712806f80453770f336e0b518a3f87338b34be4a110963bfb978fdaafec80b54a885d79466
+  data.tar.gz: ec8bc6f7f374466b4d73fdb5cca061b2cab17918818d72e28c7b03b728cdfe5328e04a79584c483a223dbec2ec767dc10a9e0e562152d561f55c24db3b08bd0e

data/dagger.gemspec

@@ -19,8 +19,8 @@ Gem::Specification.new do |s|
   s.add_development_dependency "rspec-mocks"
   s.add_development_dependency "rspec-expectations"
 
-  s.add_runtime_dependency "oj", "~> 2.1"
-  s.add_runtime_dependency "ox", "~> 2.4"
+  s.add_runtime_dependency "oj", ">= 2.1"
+  s.add_runtime_dependency "ox", ">= 2.4"
 
   s.files        = `git ls-files`.split("\n")
   s.executables  = `git ls-files`.split("\n").map{|f| f =~ /^bin\/(.*)/ ? $1 : nil}.compact

data/lib/dagger/connection_manager.rb

@@ -3,7 +3,7 @@ module Dagger
   class ConnectionManager
 
     def initialize(opts = {})
-      @opts = {}
+      @opts = opts
       @active_connections = {}
       @mutex = Mutex.new
     end

data/lib/dagger/version.rb

@@ -1,7 +1,7 @@
 module Dagger
   MAJOR = 2
-  MINOR = 0
-  PATCH = 0
+  MINOR = 1
+  PATCH = 1
 
   VERSION = [MAJOR, MINOR, PATCH].join('.')
 end

data/lib/dagger.rb

@@ -40,7 +40,7 @@ module Dagger
     end
 
     def self.resolve_uri(uri, host = nil, query = nil)
-      uri = host + uri if uri[0] == '/' && host
+      uri = host + uri if uri.to_s[0] == '/' && host
       uri = parse_uri(uri.to_s)
       uri.path.sub!(/\?.*|$/, '?' + to_query_string(query)) if query and query.any?
       uri
@@ -91,7 +91,7 @@ module Dagger
     def self.init_connection(uri, opts = {})
       http = Net::HTTP.new(opts[:ip] || uri.host, uri.port)
 
-      if uri.port == 443
+      if uri.port == 443 || uri.scheme == 'https'
         http.use_ssl = true if http.respond_to?(:use_ssl=) # persistent does it automatically
         http.verify_mode = opts[:verify_ssl] === false ? OpenSSL::SSL::VERIFY_NONE : OpenSSL::SSL::VERIFY_PEER
       end

data/spec/retries_spec.rb

@@ -0,0 +1,61 @@
+require './lib/dagger'
+
+require 'rspec/mocks'
+require 'rspec/expectations'
+
+describe 'Retries' do
+
+  def send_request
+    Dagger.get('http://foobar.com/test', opts)
+  end
+
+  let(:fake_http) { double('Net::HTTP', started?: true, "keep_alive_timeout=": true, "open_timeout=": true, "read_timeout=": true) }
+  let(:fake_resp) { double('Net::HTTPResponse', code: 200, body: 'foo') }
+
+  before do
+    allow(Net::HTTP).to receive(:new).at_least(:once).and_return(fake_http)
+    allow(fake_http).to receive(:verify_mode=).and_return(true)
+  end
+
+  describe 'on ECONNREFUSED' do
+
+    context 'if no retries option passed' do
+
+      let(:opts) { {} }
+
+      it 'does not retry request, and raises error' do
+        expect(fake_http).to receive(:request).once.and_raise(Errno::ECONNREFUSED)
+        expect { send_request }.to raise_error(Errno::ECONNREFUSED)
+      end
+
+    end
+
+    context 'if retries is 1' do
+
+      let(:opts) { { retries: 1, retry_wait: 1 } }
+
+      context 'and it still fails' do
+
+        it 'sends a second request, and raises error' do
+          expect(fake_http).to receive(:request).twice.and_raise(Errno::ECONNREFUSED)
+          expect { send_request }.to raise_error(Errno::ECONNREFUSED)
+        end
+
+      end
+
+      context 'and then it works' do
+
+        it 'sends a second request, and does not raise error' do
+          expect(fake_http).to receive(:request).once.and_raise(Errno::ECONNREFUSED)
+          expect(fake_http).to receive(:request).once.and_return(fake_resp)
+          allow(fake_resp).to receive(:[]).with('Content-Type').and_return('text/plain')
+
+          expect(send_request.body).to eq('foo')
+        end
+
+      end
+
+    end
+
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dagger
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.1
 platform: ruby
 authors:
 - Tomás Pollak
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-24 00:00:00.000000000 Z
+date: 2023-04-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -70,28 +70,28 @@ dependencies:
   name: oj
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '2.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '2.1'
 - !ruby/object:Gem::Dependency
   name: ox
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '2.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '2.4'
 description: Dagger.post(url, params).body
@@ -120,6 +120,7 @@ files:
 - spec/ip_connect_spec.rb
 - spec/parsers_spec.rb
 - spec/persistent_spec.rb
+- spec/retries_spec.rb
 - spec/sending_data_spec.rb
 homepage: https://github.com/tomas/dagger
 licenses: []
@@ -139,8 +140,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubyforge_project: dagger
-rubygems_version: 2.7.3
+rubygems_version: 3.4.6
 signing_key: 
 specification_version: 4
 summary: Simplified Net::HTTP wrapper.