log instead of logger

Author: GrantBirki

lib/hooks/app/api.rb

@@ -11,14 +11,14 @@ module App
     # Factory for creating configured Grape API classes
     class API
       # Create a new configured API class
-      def self.create(config:, endpoints:, logger:, signal_handler:)
+      def self.create(config:, endpoints:, log:, signal_handler:)
         # Store startup time for uptime calculation
         start_time = Time.now
 
         # Capture values in local variables for closure
         captured_config = config
         captured_endpoints = endpoints
-        captured_logger = logger
+        captured_logger = log
         _captured_signal_handler = signal_handler
         captured_start_time = start_time
 
@@ -175,7 +175,7 @@ def determine_error_code(exception)
 
               # Use captured values
               config = captured_config
-              logger = captured_logger
+              log = captured_logger
 
               # Set request context for logging
               request_context = {
@@ -194,9 +194,7 @@ def determine_error_code(exception)
                   raw_body = request.body.read
 
                   # Verify/validate request if configured
-                  logger.info "validating request (id: #{request_id}, handler: #{handler_class_name})"
-                  logger.debug "raw body: #{raw_body.inspect}"
-                  logger.debug "headers: #{headers.inspect}"
+                  log.info "validating request (id: #{request_id}, handler: #{handler_class_name})"
                   validate_request(raw_body, headers, endpoint_config) if endpoint_config[:request_validator]
 
                   # Parse payload
@@ -212,15 +210,15 @@ def determine_error_code(exception)
                     config: endpoint_config
                   )
 
-                  logger.info "request processed successfully (id: #{request_id}, handler: #{handler_class_name})"
+                  log.info "request processed successfully (id: #{request_id}, handler: #{handler_class_name})"
 
                   # Return response as JSON string when using txt format
                   status 200  # Explicitly set status to 200
                   content_type "application/json"
                   (response || { status: "ok" }).to_json
 
                 rescue => e
-                  logger.error "request failed: #{e.message} (id: #{request_id}, handler: #{handler_class_name})"
+                  log.error "request failed: #{e.message} (id: #{request_id}, handler: #{handler_class_name})"
 
                   # Return error response
                   error_response = {
@@ -251,7 +249,7 @@ def determine_error_code(exception)
 
               # Use captured values
               config = captured_config
-              logger = captured_logger
+              log = captured_logger
 
               # Set request context for logging
               request_context = {
@@ -282,15 +280,15 @@ def determine_error_code(exception)
                     config: {}
                   )
 
-                  logger.info "request processed successfully with default handler (id: #{request_id})"
+                  log.info "request processed successfully with default handler (id: #{request_id})"
 
                   # Return response as JSON string when using txt format
                   status 200
                   content_type "application/json"
                   (response || { status: "ok" }).to_json
 
                 rescue StandardError => e
-                  logger.error "request failed: #{e.message} (id: #{request_id})"
+                  log.error "request failed: #{e.message} (id: #{request_id})"
 
                   # Return error response
                   error_response = {

lib/hooks/core/builder.rb

@@ -15,8 +15,8 @@ class Builder
       # @param config [String, Hash] Path to config file or config hash
       # @param log [Logger] Custom logger instance
       def initialize(config: nil, log: nil)
+        @log = log
         @config_input = config
-        @custom_logger = log
       end
 
       # Build and return Rack-compatible application
@@ -26,29 +26,31 @@ def build
         # Load and validate configuration
         config = load_and_validate_config
 
-        # Create logger
-        logger = LoggerFactory.create(
-          log_level: config[:log_level],
-          custom_logger: @custom_logger
-        )
+        # Create logger unless a custom logger is provided
+        if @log.nil?
+          @log = LoggerFactory.create(
+            log_level: config[:log_level],
+            custom_logger: @custom_logger
+          )
+        end
 
         # Setup signal handler for graceful shutdown
-        signal_handler = SignalHandler.new(logger)
+        signal_handler = SignalHandler.new(@log)
 
         # Load endpoints
         endpoints = load_endpoints(config)
 
         # Log startup
-        logger.info "starting hooks server v#{Hooks::VERSION}"
-        logger.info "config: #{endpoints.size} endpoints loaded"
-        logger.info "environment: #{config[:environment]}"
-        logger.info "available endpoints: #{endpoints.map { |e| e[:path] }.join(', ')}"
+        @log.info "starting hooks server v#{Hooks::VERSION}"
+        @log.info "config: #{endpoints.size} endpoints loaded"
+        @log.info "environment: #{config[:environment]}"
+        @log.info "available endpoints: #{endpoints.map { |e| e[:path] }.join(', ')}"
 
         # Build and return Grape API class
         Hooks::App::API.create(
           config: config,
           endpoints: endpoints,
-          logger: logger,
+          log: @log,
           signal_handler: signal_handler
         )
       end