hooks-ruby 0.0.4 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2cab1bf1a1f61011d053be5beb9d48738bd439303b87743e41e10df0b4b9d65c
-  data.tar.gz: 9b9abc62fff8f0f2f77ad94949bc7e067fff5a2f072bbf73f8e6e55c03865714
+  metadata.gz: 9025d70b78a67040f436318091d556090c7f17e9dffe094b6437bde394ef7010
+  data.tar.gz: 24cf76930b63917dad1795a72113b9cf0646330671983b60bd367cc04d11d015
 SHA512:
-  metadata.gz: 40f769a697bd61c6c851eef217fadc66387c49d37c19639a283dd770ca8be8889a13eeba2785a41c53edcdef29327231527405a9de21b2b4620416196da792c2
-  data.tar.gz: 119a371d927f240aefe4e74c822b48318032250e7cbf060a9bd077b1375823137b419f7d1e2d27d775e6468d38843afb45ec9c8cd8a71a22d82226160c66043e
+  metadata.gz: 0e8acdf59a00128c3a68fb2db10864c0e0bf6a5e454aabceaf824cdf4ba62f8057f166da2dae0f17238a7a280137861ca4ba1da70b33165e6e890becb36be391
+  data.tar.gz: 33178114bcbbda82ecf91cd27bfd7633621f6d7f2ea64fc3891596a8769cc80dac55ceab5c4fc75cd035d2a0188c9da2821743442cfcdbbce0cef2d3d479e6b5

data/lib/hooks/app/api.rb

@@ -102,13 +102,13 @@ module Hooks
                     validate_auth!(raw_body, headers, endpoint_config, config)
                   end
 
-                  payload = parse_payload(raw_body, headers, symbolize: config[:symbolize_payload])
+                  payload = parse_payload(raw_body, headers, symbolize: false)
                   handler = load_handler(handler_class_name)
-                  normalized_headers = config[:normalize_headers] ? Hooks::Utils::Normalize.headers(headers) : headers
+                  processed_headers = config[:normalize_headers] ? Hooks::Utils::Normalize.headers(headers) : headers
 
                   response = handler.call(
                     payload:,
-                    headers: normalized_headers,
+                    headers: processed_headers,
                     config: endpoint_config
                   )
 

data/lib/hooks/app/helpers.rb

@@ -44,9 +44,9 @@ module Hooks
       #
       # @param raw_body [String] The raw request body
       # @param headers [Hash] The request headers
-      # @param symbolize [Boolean] Whether to symbolize keys in parsed JSON (default: true)
-      # @return [Hash, String] Parsed JSON as Hash (optionally symbolized), or raw body if not JSON
-      def parse_payload(raw_body, headers, symbolize: true)
+      # @param symbolize [Boolean] Whether to symbolize keys in parsed JSON (default: false)
+      # @return [Hash, String] Parsed JSON as Hash with string keys, or raw body if not JSON
+      def parse_payload(raw_body, headers, symbolize: false)
         # Optimized content type check - check most common header first
         content_type = headers["Content-Type"] || headers["CONTENT_TYPE"] || headers["content-type"] || headers["HTTP_CONTENT_TYPE"]
 
@@ -55,6 +55,7 @@ module Hooks
           begin
             # Security: Limit JSON parsing depth and complexity to prevent JSON bombs
             parsed_payload = safe_json_parse(raw_body)
+            # Note: symbolize parameter is kept for backward compatibility but defaults to false
             parsed_payload = parsed_payload.transform_keys(&:to_sym) if symbolize && parsed_payload.is_a?(Hash)
             return parsed_payload
           rescue JSON::ParserError, ArgumentError => e

data/lib/hooks/core/config_loader.rb

@@ -20,7 +20,6 @@ module Hooks
         production: true,
         endpoints_dir: "./config/endpoints",
         use_catchall_route: false,
-        symbolize_payload: true,
         normalize_headers: true
       }.freeze
 
@@ -141,7 +140,6 @@ module Hooks
           "HOOKS_ENVIRONMENT" => :environment,
           "HOOKS_ENDPOINTS_DIR" => :endpoints_dir,
           "HOOKS_USE_CATCHALL_ROUTE" => :use_catchall_route,
-          "HOOKS_SYMBOLIZE_PAYLOAD" => :symbolize_payload,
           "HOOKS_NORMALIZE_HEADERS" => :normalize_headers,
           "HOOKS_SOME_STRING_VAR" => :some_string_var # Added for test
         }
@@ -154,7 +152,7 @@ module Hooks
           case config_key
           when :request_limit, :request_timeout
             env_config[config_key] = value.to_i
-          when :use_catchall_route, :symbolize_payload, :normalize_headers
+          when :use_catchall_route, :normalize_headers
             # Convert string to boolean
             env_config[config_key] = %w[true 1 yes on].include?(value.downcase)
           else

data/lib/hooks/core/config_validator.rb

@@ -26,7 +26,6 @@ module Hooks
         optional(:environment).filled(:string, included_in?: %w[development production])
         optional(:endpoints_dir).filled(:string)
         optional(:use_catchall_route).filled(:bool)
-        optional(:symbolize_payload).filled(:bool)
         optional(:normalize_headers).filled(:bool)
       end
 

data/lib/hooks/plugins/handlers/base.rb

@@ -15,8 +15,8 @@ module Hooks
         # Process a webhook request
         #
         # @param payload [Hash, String] Parsed request body (JSON Hash) or raw string
-        # @param headers [Hash<String, String>] HTTP headers
-        # @param config [Hash] Merged endpoint configuration including opts section
+        # @param headers [Hash] HTTP headers (string keys, optionally normalized - default is normalized)
+        # @param config [Hash] Merged endpoint configuration including opts section (symbolized keys)
         # @return [Hash, String, nil] Response body (will be auto-converted to JSON)
         # @raise [NotImplementedError] if not implemented by subclass
         def call(payload:, headers:, config:)

data/lib/hooks/utils/normalize.rb

@@ -58,6 +58,39 @@ module Hooks
         normalized
       end
 
+      # Symbolize header keys in a hash
+      #
+      # @param headers [Hash, #each] Headers hash or hash-like object
+      # @return [Hash] Hash with symbolized keys (hyphens converted to underscores)
+      #
+      # @example Header symbolization
+      #   headers = { "content-type" => "application/json", "x-github-event" => "push" }
+      #   symbolized = Normalize.symbolize_headers(headers)
+      #   # => { content_type: "application/json", x_github_event: "push" }
+      #
+      # @example Handle various input types
+      #   Normalize.symbolize_headers(nil)  # => nil
+      #   Normalize.symbolize_headers({})   # => {}
+      def self.symbolize_headers(headers)
+        # Handle nil input
+        return nil if headers.nil?
+
+        # Fast path for non-enumerable inputs
+        return {} unless headers.respond_to?(:each)
+
+        symbolized = {}
+
+        headers.each do |key, value|
+          next if key.nil?
+
+          # Convert key to symbol, replacing hyphens with underscores
+          symbolized_key = key.to_s.tr("-", "_").to_sym
+          symbolized[symbolized_key] = value
+        end
+
+        symbolized
+      end
+
       # Normalize a single HTTP header name
       #
       # @param header [String] Header name to normalize

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.0.4".freeze
+  VERSION = "0.0.6".freeze
 end

metadata

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