release: 2.0.0

Update rubocop to version 0.63.0 (#25)

.rubocop.yml

@@ -6,10 +6,19 @@ AllCops:
 Gemspec/OrderedDependencies:
   Enabled: false
 
+Layout/AlignHash:
+  Enabled: false
+
+Layout/EmptyLineAfterGuardClause:
+  Enabled: false
+
 # allows nicer usage of private_class_method
 Layout/EmptyLinesAroundArguments:
   Enabled: false
 
+Layout/LeadingBlankLines:
+  Enabled: false
+
 Layout/SpaceInsideHashLiteralBraces:
   Enabled: false
 

.travis.yml

@@ -2,6 +2,7 @@
 sudo: false
 language: ruby
 rvm:
+  - 2.6
   - 2.5
   - 2.4
   - 2.3

CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 2.0.0
+## 2.0.0 (January 25, 2019)
 
 IMPROVEMENTS:
 
@@ -8,7 +8,7 @@ IMPROVEMENTS:
 - Better protocol compliance by returning `badauth` in response body on HTTP 401 errors
 - Better code maintainability by refactorings
 - Update dependencies, mainly `rack` to new major version 2
-- Add Ruby 2.5 support
+- Add Ruby 2.5 and Ruby 2.6 support
 - Add experimental [OpenTracing](http://opentracing.io/) support with [CNCF Jaeger](https://github.com/jaegertracing/jaeger)
 - Support host offlining by deleting the associated DNS records
 - Add textfile reporter to write Graphite-style metrics (also compatible with [Prometheus](https://prometheus.io/)) into a file

Rakefile

@@ -1,8 +1,10 @@
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 require 'rubocop/rake_task'
+require 'bundler/audit/task'
 
 RSpec::Core::RakeTask.new(:spec)
 RuboCop::RakeTask.new
+Bundler::Audit::Task.new
 
-task default: [:rubocop, :spec]
+task default: [:rubocop, :spec, 'bundle:audit']

dyndnsd.gemspec

@@ -23,14 +23,14 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency 'rack', '~> 2.0'
   s.add_runtime_dependency 'json'
   s.add_runtime_dependency 'metriks'
-  s.add_runtime_dependency 'opentracing', '~> 0.4.0'
-  s.add_runtime_dependency 'rack-tracer', '~> 0.5.0'
-  s.add_runtime_dependency 'spanmanager', '~> 0.3.0'
-  s.add_runtime_dependency 'jaeger-client', '~> 0.4.0'
+  s.add_runtime_dependency 'opentracing', '~> 0.5.0'
+  s.add_runtime_dependency 'rack-tracer', '~> 0.8.0'
+  s.add_runtime_dependency 'jaeger-client', '~> 0.10.0'
 
   s.add_development_dependency 'bundler'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec'
   s.add_development_dependency 'rack-test'
-  s.add_development_dependency 'rubocop', '~> 0.55.0'
+  s.add_development_dependency 'rubocop', '~> 0.63.0'
+  s.add_development_dependency 'bundler-audit'
 end

lib/dyndnsd.rb

@@ -10,7 +10,6 @@ require 'metriks'
 require 'metriks/reporter/graphite'
 require 'opentracing'
 require 'rack/tracer'
-require 'spanmanager'
 
 require 'dyndnsd/generator/bind'
 require 'dyndnsd/updater/command_with_bind_zone'
@@ -266,8 +265,6 @@ module Dyndnsd
           host: host, port: port, service_name: service_name, flush_interval: 1
         )
       end
-      # always use SpanManager
-      OpenTracing.global_tracer = SpanManager::Tracer.new(OpenTracing.global_tracer)
     end
 
     private_class_method def self.setup_rack(config)

lib/dyndnsd/helper.rb

@@ -28,13 +28,24 @@ module Dyndnsd
     end
 
     def self.span(operation, &block)
-      span = OpenTracing.start_span(operation)
+      scope = OpenTracing.start_active_span(operation)
+      span = scope.span
       span.set_tag('component', 'dyndnsd')
       span.set_tag('span.kind', 'server')
       begin
         block.call(span)
+      rescue StandardError => e
+        span.set_tag('error', true)
+        span.log_kv(
+          event: 'error',
+          'error.kind': e.class.to_s,
+          'error.object': e,
+          message: e.message,
+          stack: e.backtrace.join("\n")
+        )
+        raise
       ensure
-        span.finish
+        scope.close
       end
     end
   end

lib/dyndnsd/version.rb

@@ -1,4 +1,4 @@
 
 module Dyndnsd
-  VERSION = '2.0.0.rc2'.freeze
+  VERSION = '2.0.0'.freeze
 end