onlyoffice_bugzilla_helper 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7ff70cf1f92d58b5ee3fde49f516e53be60ad0f1589cae8624c5c8dc80efeb7c
-  data.tar.gz: ae5a6db51a4e414a0c23e997084763c47052b2494ad4ba6fb8848a6494133bd4
+  metadata.gz: c2c0df09ea18bbe7bd0b3e564c373bd2d489bc46aa782905b8bacf6664126add
+  data.tar.gz: e178efeb2a1fd38b49db763eeb41292dfccbf8836b206779c48bac485f0c5c4d
 SHA512:
-  metadata.gz: a959f75ef11abab71e6f82bc15e9307ceb46c7bbdc6befd18ef12b2869d4000adcb100577bf9b36c514fb683d6e06533d2cb5addad015955063cef16fa56a8c5
-  data.tar.gz: '018bd6c8e66fc1b4930f001a32d024adad5dc4e16c38bef9197bd27098caba20b2707dbcbc7d549f82f466338b099a58ce124e9dba9090bcbdc5784b8ea499b0'
+  metadata.gz: 700fc2d2050f00db7f1bacc9524067bcaf9225eef0de1ca81345fc4c4d11cbd8ff63ef5cad5cb9efba1faf2ea56e599ac50f08f29f65ba31dbf71459841c6af4
+  data.tar.gz: 333a43b5c73e213162e21460b9acedbc1e8d38b72a1c904701bd28de578394bfc0d5c9a731451df4dfd68dd2eaa891c2d9818886b9f976752a4bead969dc614e

data/lib/onlyoffice_bugzilla_helper.rb

@@ -6,6 +6,7 @@ require 'net/http'
 require 'uri'
 require 'onlyoffice_bugzilla_helper/bug_data'
 require 'onlyoffice_bugzilla_helper/comments'
+require 'onlyoffice_bugzilla_helper/logger_wrapper'
 require 'onlyoffice_bugzilla_helper/networking'
 require 'onlyoffice_bugzilla_helper/update_bug'
 require 'onlyoffice_bugzilla_helper/version'
@@ -16,6 +17,7 @@ module OnlyofficeBugzillaHelper
   class BugzillaHelper
     include BugData
     include Comments
+    include LoggerWrapper
     include Networking
     include UpdateBug
     attr_reader :url
@@ -70,9 +72,8 @@ module OnlyofficeBugzillaHelper
     # @param bug_id [Integer] id of bug
     # @return [Net::HTTPResponse] result of request
     def get_bug_result(bug_id)
-      Net::HTTP.start(@url.host, @url.port, use_ssl: use_ssl?) do |http|
-        http.get(bug_url(bug_id))
-      end
+      req = Net::HTTP::Get.new(bug_url(bug_id))
+      perform_request(req)
     end
   end
 end

data/lib/onlyoffice_bugzilla_helper/comments.rb

@@ -18,9 +18,7 @@ module OnlyofficeBugzillaHelper
       req = Net::HTTP::Post.new(bug_url(bug_id, '/comment'))
       req.body = { comment: comment }.to_json
       req.add_field('Content-Type', 'text/plain')
-      connection = Net::HTTP.new(@url.host, @url.port)
-      connection.use_ssl = use_ssl?
-      connection.start { |http| http.request(req) }
+      perform_request(req)
     end
   end
 end

data/lib/onlyoffice_bugzilla_helper/logger_wrapper.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'logger'
+
+module OnlyofficeBugzillaHelper
+  # Logger module for logging stuff
+  module LoggerWrapper
+    # @return [Logger] default logger
+    def logger
+      @logger ||= Logger.new($stdout)
+    end
+
+    # Log info about request
+    # @param [Net:HTTP] request to make
+    # @param [Net:HTTP] response in result
+    # @return [nil]
+    def log_request(request, response)
+      logger.info('Made request. '\
+                  "Type: #{request.method}. "\
+                  "Path: #{hide_keys(request.path)}. "\
+                  "Body: #{hide_keys(request.body)}. "\
+                  "Response: #{hide_keys(response.body)}")
+    end
+
+    # Hide sensitive info
+    # @param [String] string to hide
+    # @return [String] result
+    def hide_keys(string)
+      string&.gsub(@key, '[REMOVED]')
+    end
+  end
+end

data/lib/onlyoffice_bugzilla_helper/networking.rb

@@ -9,5 +9,16 @@ module OnlyofficeBugzillaHelper
     def use_ssl?
       @url.scheme == 'https'
     end
+
+    # Perform request for current bugzilla
+    # @param [Net:HTTP] request to make
+    # @return [Net:HTTPResponse] result of request
+    def perform_request(request)
+      connection = Net::HTTP.new(@url.host, @url.port)
+      connection.use_ssl = use_ssl?
+      result = connection.start { |http| http.request(request) }
+      log_request(request, result)
+      result
+    end
   end
 end

data/lib/onlyoffice_bugzilla_helper/update_bug.rb

@@ -11,9 +11,7 @@ module OnlyofficeBugzillaHelper
       req = Net::HTTP::Put.new(bug_url(bug_id))
       req.body = params.to_json
       req.add_field('Content-Type', 'text/plain')
-      connection = Net::HTTP.new(@url.host, @url.port)
-      connection.use_ssl = use_ssl?
-      connection.start { |http| http.request(req) }
+      perform_request(req)
     end
   end
 end

data/lib/onlyoffice_bugzilla_helper/version.rb

@@ -2,5 +2,5 @@
 
 module OnlyofficeBugzillaHelper
   # [String] version of Gem
-  VERSION = '0.5.0'
+  VERSION = '0.6.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: onlyoffice_bugzilla_helper
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - ONLYOFFICE
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-18 00:00:00.000000000 Z
+date: 2021-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faker
@@ -161,6 +161,7 @@ files:
 - lib/onlyoffice_bugzilla_helper.rb
 - lib/onlyoffice_bugzilla_helper/bug_data.rb
 - lib/onlyoffice_bugzilla_helper/comments.rb
+- lib/onlyoffice_bugzilla_helper/logger_wrapper.rb
 - lib/onlyoffice_bugzilla_helper/name.rb
 - lib/onlyoffice_bugzilla_helper/networking.rb
 - lib/onlyoffice_bugzilla_helper/update_bug.rb
@@ -189,7 +190,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.14
 signing_key: 
 specification_version: 4
 summary: Helper for bugzilla