hooks-ruby 0.3.2 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1f6bca4c82b0d3c507e2f772257a0e6ed5a30a072bb51b61da45d1c2688d99ba
-  data.tar.gz: c0d8a9f0aea0b3c5f039b7cb3354d418b79fab87aa14a4738ffb7ee04b554a2d
+  metadata.gz: 27cbf50d13a78bab2fb359d40a11a8ffb6998712317a2927c95a1d083cdd3be3
+  data.tar.gz: b3473918310de8d23aeaf448af5d09b7601f18dce286a18cfcdc0efd78c7548a
 SHA512:
-  metadata.gz: 0afe8a3b475163b154d7016b939fcf4fcdd40312534ea732018cc8cfe407823b3266d7c31881d010725bfa670927dfe6e6fd84663ca68ccb54dc27179fb01293
-  data.tar.gz: '04844e2020cc32371f889986c1eadc5c4cbd83bcac06a566466784ff306525e02713eb7f75838a304aa283b230b08b61760dc80f1e4fef993b2d163aaf587fbd'
+  metadata.gz: e8ec9dc8250c234bad0821d62f4876808725d0aa56ac8b62ba03c5789e02b250e11b0c8b8bba581e691033f130d69ef6d57f29ca2d4c29592389457e75107161
+  data.tar.gz: 0733db6077bfd15c6723cdad92e443a2c4d262f32ab6c30a989dcee12b07223a7d9bfff4a7fcb50950a1c412cb1c1e1a0031382f3d657d799e89da5a1f0d55dd

data/lib/hooks/app/api.rb

@@ -38,8 +38,8 @@ module Hooks
           content_type :txt, "text/plain"
           content_type :xml, "application/xml"
           content_type :any, "*/*"
-          format :txt
-          default_format :txt
+
+          default_format config[:default_format] || :json
         end
 
         api_class.class_eval do
@@ -117,22 +117,21 @@ module Hooks
                   log.info("successfully processed webhook event with handler: #{handler_class_name}")
                   log.debug("processing duration: #{Time.now - start_time}s")
                   status 200
-                  content_type "application/json"
-                  response.to_json
+                  response
                 rescue Hooks::Plugins::Handlers::Error => e
                   # Handler called error! method - immediately return error response and exit the request
                   log.debug("handler #{handler_class_name} called `error!` method")
 
-                  error_response = nil
-
                   status e.status
                   case e.body
                   when String
+                    # if error! was called with a string, we assume it's a simple text error
+                    # example: error!("simple text error", 400) -> should return a plain text response
                     content_type "text/plain"
                     error_response = e.body
                   else
-                    content_type "application/json"
-                    error_response = e.body.to_json
+                    # Let Grape handle JSON conversion with the default format
+                    error_response = e.body
                   end
 
                   return error_response
@@ -163,8 +162,7 @@ module Hooks
                   error_response[:handler] = handler_class_name unless config[:production]
 
                   status determine_error_code(e)
-                  content_type "application/json"
-                  error_response.to_json
+                  error_response
                 end
               end
             end

data/lib/hooks/app/endpoints/catchall.rb

@@ -17,6 +17,13 @@ module Hooks
     class CatchallEndpoint < Grape::API
       include Hooks::App::Helpers
 
+      # Set up content types and default format to JSON to match main API
+      content_type :json, "application/json"
+      content_type :txt, "text/plain"
+      content_type :xml, "application/xml"
+      content_type :any, "*/*"
+      default_format :json
+
       def self.mount_path(config)
         # :nocov:
         "#{config[:root_path]}/*path"
@@ -81,8 +88,7 @@ module Hooks
               log.info("successfully processed webhook event with handler: #{handler_class_name}")
               log.debug("processing duration: #{Time.now - start_time}s")
               status 200
-              content_type "application/json"
-              response.to_json
+              response
             rescue StandardError => e
               err_msg = "Error processing webhook event with handler: #{handler_class_name} - #{e.message} " \
                 "- request_id: #{request_id} - path: #{full_path} - method: #{http_method} - " \
@@ -102,8 +108,7 @@ module Hooks
               error_response[:handler] = handler_class_name unless config[:production]
 
               status determine_error_code(e)
-              content_type "application/json"
-              error_response.to_json
+              error_response
             end
           end
         end

data/lib/hooks/app/endpoints/health.rb

@@ -6,14 +6,20 @@ require_relative "../../version"
 module Hooks
   module App
     class HealthEndpoint < Grape::API
+      # Set up content types and default format to JSON
+      content_type :json, "application/json"
+      content_type :txt, "text/plain"
+      content_type :xml, "application/xml"
+      content_type :any, "*/*"
+      default_format :json
+
       get do
-        content_type "application/json"
         {
           status: "healthy",
           timestamp: Time.now.utc.iso8601,
           version: Hooks::VERSION,
           uptime_seconds: (Time.now - Hooks::App::API.server_start_time).to_i
-        }.to_json
+        }
       end
     end
   end

data/lib/hooks/app/endpoints/version.rb

@@ -6,12 +6,18 @@ require_relative "../../version"
 module Hooks
   module App
     class VersionEndpoint < Grape::API
+      # Set up content types and default format to JSON
+      content_type :json, "application/json"
+      content_type :txt, "text/plain"
+      content_type :xml, "application/xml"
+      content_type :any, "*/*"
+      default_format :json
+
       get do
-        content_type "application/json"
         {
           version: Hooks::VERSION,
           timestamp: Time.now.utc.iso8601
-        }.to_json
+        }
       end
     end
   end

data/lib/hooks/core/config_loader.rb

@@ -20,7 +20,8 @@ module Hooks
         production: true,
         endpoints_dir: "./config/endpoints",
         use_catchall_route: false,
-        normalize_headers: true
+        normalize_headers: true,
+        default_format: :json
       }.freeze
 
       SILENCE_CONFIG_LOADER_MESSAGES = ENV.fetch(
@@ -141,6 +142,7 @@ module Hooks
           "HOOKS_ENDPOINTS_DIR" => :endpoints_dir,
           "HOOKS_USE_CATCHALL_ROUTE" => :use_catchall_route,
           "HOOKS_NORMALIZE_HEADERS" => :normalize_headers,
+          "HOOKS_DEFAULT_FORMAT" => :default_format,
           "HOOKS_SOME_STRING_VAR" => :some_string_var # Added for test
         }
 
@@ -155,6 +157,9 @@ module Hooks
           when :use_catchall_route, :normalize_headers
             # Convert string to boolean
             env_config[config_key] = %w[true 1 yes on].include?(value.downcase)
+          when :default_format
+            # Convert string to symbol
+            env_config[config_key] = value.to_sym
           else
             env_config[config_key] = value
           end

data/lib/hooks/core/config_validator.rb

@@ -27,6 +27,7 @@ module Hooks
         optional(:endpoints_dir).filled(:string)
         optional(:use_catchall_route).filled(:bool)
         optional(:normalize_headers).filled(:bool)
+        optional(:default_format).filled(:symbol, included_in?: %i[json txt xml any])
 
         optional(:ip_filtering).hash do
           optional(:ip_header).filled(:string)

data/lib/hooks/version.rb

@@ -4,5 +4,5 @@
 module Hooks
   # Current version of the Hooks webhook framework
   # @return [String] The version string following semantic versioning
-  VERSION = "0.3.2".freeze
+  VERSION = "0.4.0".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hooks-ruby
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.4.0
 platform: ruby
 authors:
 - github