Merge pull request #15 from MortarStone/master

Add headers to results and handle 429 error
planningcenter · Sep 21, 2020 · 3568b59 · 3568b59
2 parents 070b6ce + 3871e80
commit 3568b59
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/pco/api/endpoint.rb b/lib/pco/api/endpoint.rb
@@ -69,7 +69,9 @@ def delete
       def _build_response(result)
         case result.status
         when 200..299
-          result.body
+          res = result.body
+          res['headers'] = result.headers
+          res
         when 400
           fail Errors::BadRequest, result
         when 401
@@ -82,6 +84,8 @@ def _build_response(result)
           fail Errors::MethodNotAllowed, result
         when 422
           fail Errors::UnprocessableEntity, result
+        when 429
+          fail Errors::TooManyRequests, result
         when 400..499
           fail Errors::ClientError, result
         when 500

diff --git a/lib/pco/api/errors.rb b/lib/pco/api/errors.rb
@@ -52,6 +52,7 @@ class Forbidden           < ClientError; end # 403
       class NotFound            < ClientError; end # 404
       class MethodNotAllowed    < ClientError; end # 405
       class UnprocessableEntity < ClientError; end # 422
+      class TooManyRequests     < ClientError; end # 429
 
       class ServerError         < BaseError;   end # 500..599
       class InternalServerError < ServerError; end # 500