pager_duty-connection 2.1.0 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ec4e78b2def787e07a82a8dfb19838dcace837e98647e386209af24003f26e9
-  data.tar.gz: 7d0885571bbf8e088de58ec22a12bff3354935b5c036017c0548babf8422e933
+  metadata.gz: 38445eaf7e12ef537c66dca5286ec2e35cc5b76659c1df5d0972c11479f6bb56
+  data.tar.gz: bc79b9edd688055b9fb06631cc1573bfc4a148f5994397b39b5ffe617c771e18
 SHA512:
-  metadata.gz: e30277c8080331cd8ae89dcf520d7d9df945a5ed5456f4b0acba12a40d43fa9ca038a5860650091762d47a08cd46c1149bbc5f93000d13b68f2fd793490e43c2
-  data.tar.gz: d47249fc1dab34ed3100177c81d13fc15b83f60201cbad4c9b1516f295da6cb156a723a54afbd753448e1cdfb1e7e2d6ba33c72eeffc53fb72ee9e87ae1e87db
+  metadata.gz: 57d66bf01db4ae1de573d108609503e877d77be6d2b9f31cb8d02cf45ba8e61c73154db44ec658191466042ba60932f4bfb17e3a58e63d8c844c380d86beab2f
+  data.tar.gz: 6ca00d5ce74e0cd573c052db347f7eddf371d0e80949b526d711747b04e289f61c2bc5f5977dcc85d605aad0972ed0c9d4269c108892ca10b9fc737abe369cb6

data/.ruby-version

@@ -1 +1 @@
-2.7.5
+3.2.2

data/lib/pager_duty/connection/version.rb

@@ -1,5 +1,5 @@
 module PagerDuty
   class Connection
-    VERSION = "2.1.0"
+    VERSION = "2.3.0"
   end
 end

data/lib/pager_duty/connection.rb

@@ -12,15 +12,39 @@ module PagerDuty
     API_VERSION = 2
     API_PREFIX = "https://api.pagerduty.com/"
 
-    class FileNotFoundError < RuntimeError
-    end
+    class FileNotFoundError < RuntimeError; end
+
+    class ApiError < RuntimeError; end
+
+    class RateLimitError < RuntimeError; end
 
-    class ApiError < RuntimeError
+    class UnauthorizedError < RuntimeError; end
+
+    class ForbiddenError < RuntimeError; end
+
+    class RaiseUnauthorizedOn401 < Faraday::Middleware
+      def call(env)
+        response = @app.call(env)
+        if response.status == 401
+          raise PagerDuty::Connection::UnauthorizedError, response.env[:url].to_s
+        else
+          response
+        end
+      end
     end
 
-    class RateLimitError < RuntimeError
+    class RaiseForbiddenOn403 < Faraday::Middleware
+      def call(env)
+        response = @app.call(env)
+        if response.status == 403
+          raise PagerDuty::Connection::ForbiddenError, response.env[:url].to_s
+        else
+          response
+        end
+      end
     end
 
+
     class RaiseFileNotFoundOn404 < Faraday::Middleware
       def call(env)
         response = @app.call env
@@ -40,10 +64,13 @@ module PagerDuty
           message = "Got HTTP #{response.status}: #{response.reason_phrase}\nFrom #{url}"
 
           if error = response.body
-            # TODO May Need to check error.errors too
-            message += "\n#{JSON.parse(error)}"
+            begin
+              # TODO May Need to check error.errors too
+              message += "\n#{JSON.parse(error)}"
+            rescue JSON::ParserError
+              message += "\n#{error}"
+            end
           end
-
           raise ApiError, message
         else
           response
@@ -67,9 +94,11 @@ module PagerDuty
       def call(env)
 
         body = env[:body]
-        TIME_KEYS.each do |key|
-          if body.has_key?(key)
-            body[key] = body[key].iso8601 if body[key].respond_to?(:iso8601)
+        unless body.nil?
+          TIME_KEYS.each do |key|
+            if body.has_key?(key)
+              body[key] = body[key].iso8601 if body[key].respond_to?(:iso8601)
+            end
           end
         end
 
@@ -154,13 +183,13 @@ module PagerDuty
       @connection = Faraday.new do |conn|
         conn.url_prefix = url
 
-        token_arg =
-          case token_type
-          when :Token then { token: token }
-          when :Bearer then token
-          else raise ArgumentError, "invalid token_type: #{token_type.inspect}"
-          end
-        conn.authorization(token_type, token_arg)
+        case token_type
+        when :Token
+          conn.request :token_auth, token
+        when :Bearer
+          conn.request :authorization, 'Bearer', token
+        else raise ArgumentError, "invalid token_type: #{token_type.inspect}"
+        end
 
         conn.use ConvertTimesParametersToISO8601
 
@@ -180,12 +209,20 @@ module PagerDuty
         conn.use RaiseApiErrorOnNon200
         conn.use RaiseFileNotFoundOn404
         conn.use RaiseRateLimitOn429
+        conn.use RaiseForbiddenOn403
+        conn.use RaiseUnauthorizedOn401
 
         conn.adapter  Faraday.default_adapter
       end
     end
 
     def get(path, request = {})
+      # The run_request() method body argument defaults to {}, which is incorrect for GET requests
+      # https://github.com/technicalpickles/pager_duty-connection/issues/56
+      # NOTE: PagerDuty support discourages GET requests with bodies, but not throwing an ArgumentError to prevent breaking
+      #   corner-case implementations.
+      request[:body] = nil if !request[:body]
+
       # paginate anything being 'get'ed, because the offset/limit isn't intuitive
       request[:query_params] = {} if !request[:query_params]
       page = request[:query_params].fetch(:page, 1).to_i

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pager_duty-connection
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.3.0
 platform: ruby
 authors:
 - Josh Nichols
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-13 00:00:00.000000000 Z
+date: 2024-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -109,7 +109,7 @@ files:
 homepage: http://github.com/technicalpickles/pager_duty-connection
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -124,8 +124,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.10
-signing_key: 
+rubygems_version: 3.4.18
+signing_key:
 specification_version: 4
 summary: Written with the power of faraday, pager_duty-connection tries to be a simple
   and usable Ruby API wrapper for the PagerDuty REST API