numerousapp 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7018d2d603b7836ff5ce90f71f2cdf324ffb60d9
-  data.tar.gz: 4d7b0bd9d9e9d3307b0e68546dfced0e193ff457
+  metadata.gz: 2cb812bb5dd0b6ff8eff69717086072183b6f403
+  data.tar.gz: f597ff55c83c748b333c715552540ae42d4c04e0
 SHA512:
-  metadata.gz: cb2e36d7ad7e315684c349c757097e1e1094e403535186abb76b874cb7ec804b92b58d7652c837d41cacc89138117bacdf77fafc509f98dff126f3c6a5134681
-  data.tar.gz: 92a86c605f1357060f10a5fc1f0f6b1ddc846b281ab9c61cfd365f5df11a715d61eb25ddf3c5f2b39bc11badb62191eb915b80c6dbc08b340c0e645d54cd50f5
+  metadata.gz: 62356ae9378340d0e2b3d82c617804a6bda4c5ed538da960202ce49d38acd349c9d4e8c201e6666cd2e638cbd5ccddc5317285d857192d40efd848dcecd43f3f
+  data.tar.gz: 4162109867626e763f8a70cb6e8176ffe504904f186c90c1f416c0f4a8332545a5e55f8568b91b480cc4ff5c54609a939008a306ef96e58d26cd5cf24d4d7f31

data/lib/numerousapp.rb

@@ -131,16 +131,16 @@ class NumerousClientInternals
             apiKey = Numerous.numerousKey()
         end
 
-        @serverName = server
         @auth = { user: apiKey, password: "" }
         u = URI.parse("https://"+server)
-        @http = Net::HTTP.new(server, u.port)
-        @http.use_ssl = true    # always required by NumerousApp
+        @serverName = server
+        @serverPort = u.port
 
         @agentString = "NW-Ruby-NumerousClass/" + VersionString +
                        " (Ruby #{RUBY_VERSION}) NumerousAPI/v2"
 
         @filterDuplicates = true     # see discussion elsewhere
+        @need_restart = true         # port will be opened in simpleAPI
 
         # throttling.
         # The arbitraryMaximum is just that: under no circumstances will we retry
@@ -212,7 +212,7 @@ class NumerousClientInternals
 
     protected
 
-    VersionString = '20150415-1.2.0'
+    VersionString = '20150617-1.2.1'
 
     MethMap = {
         GET: Net::HTTP::Get,
@@ -373,7 +373,29 @@ class NumerousClientInternals
             @statistics[:serverRequests] += 1
             t0 = Time.now
             begin
+
+                # see note immediately below re "need_restart"
+                # this is where the very FIRST start() happens but
+                # also where subsequent starts might be re-done after errors
+
+                if @need_restart or not @http.started?()
+
+                    # starting a session turns on keep-alive...
+                    @http = Net::HTTP.new(@serverName, @serverPort)
+                    @http.use_ssl = true    # always required by NumerousApp
+                    @http = @http.start()
+                end
+
+                # A note on this need_restart true/false dance:
+                #    I have to admit I'm not sure if this is necessary or if
+                #    it is even a good/bad/irrelevant idea, but the concept
+                #    here is that if an error is encountered we'll ditch
+                #    the current start() session and make a new one next time
+
+                @need_restart = true    # will redo session if raise out
                 resp = @http.request(rq)
+                @need_restart = false
+
             rescue StandardError => e
                 # it's PDB (pretty bogus) that we have to rescue
                 # StandardError but the underlying http library can just throw
@@ -1332,7 +1354,6 @@ class NumerousMetric < NumerousClientInternals
            val = self['value']
            rslt += "'#{self['label']}' [#@id] = #{val}>"
        rescue NumerousError => x
-           puts(x.code)
            if x.code == 400
                rslt += "**INVALID-ID** '#@id'>"
            elsif x.code == 404

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: numerousapp
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Neil Webber
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-15 00:00:00.000000000 Z
+date: 2015-06-17 00:00:00.000000000 Z
 dependencies: []
 description: Classes implementing the NumerousApp REST APIs for metrics. Requires
   Ruby 2.x