Merge branch 'main' into copilot/fix-31

Author: GrantBirki

README.md

@@ -66,7 +66,7 @@ Here is a very high-level overview of how Hooks works:
           status: "success",
           handler: "MyCustomHandler",
           payload_received: payload,
-          timestamp: Time.now.iso8601
+          timestamp: Time.now.utc.iso8601
         }
       end
     end
@@ -229,7 +229,7 @@ class HelloHandler < Hooks::Plugins::Handlers::Base
     {
       message: "webhook processed successfully",
       handler: "HelloHandler",
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end
@@ -245,7 +245,7 @@ class GoodbyeHandler < Hooks::Plugins::Handlers::Base
     {
       message: "goodbye webhook processed successfully",
       handler: "GoodbyeHandler",
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end

docs/design.md

@@ -277,7 +277,7 @@ class CustomErrorPlugin < Hooks::Plugins::Lifecycle
     {
       error: "Custom error message",
       code: determine_error_code(exception),
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end

lib/hooks/app/endpoints/health.rb

@@ -10,7 +10,7 @@ class HealthEndpoint < Grape::API
         content_type "application/json"
         {
           status: "healthy",
-          timestamp: Time.now.iso8601,
+          timestamp: Time.now.utc.iso8601,
           version: Hooks::VERSION,
           uptime_seconds: (Time.now - Hooks::App::API.server_start_time).to_i
         }.to_json

lib/hooks/app/endpoints/version.rb

@@ -10,7 +10,7 @@ class VersionEndpoint < Grape::API
         content_type "application/json"
         {
           version: Hooks::VERSION,
-          timestamp: Time.now.iso8601
+          timestamp: Time.now.utc.iso8601
         }.to_json
       end
     end

lib/hooks/plugins/handlers/default.rb

@@ -15,7 +15,7 @@ def call(payload:, headers:, config:)
     {
       message: "webhook processed successfully",
       handler: "DefaultHandler",
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end

spec/acceptance/plugins/handlers/hello.rb

@@ -5,7 +5,7 @@ def call(payload:, headers:, config:)
     {
       status: "success",
       handler: self.class.name,
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end

spec/acceptance/plugins/handlers/team1_handler.rb

@@ -43,7 +43,7 @@ def call(payload:, headers:, config:)
         message: "Raw payload processed",
         payload_size: payload.length,
         environment: config.dig(:opts, :env),
-        timestamp: Time.now.iso8601
+        timestamp: Time.now.utc.iso8601
       }
     end
   end
@@ -62,7 +62,7 @@ def handle_deployment(payload, config)
       deployment_id: payload["deployment_id"],
       environment: payload["environment"] || config.dig(:opts, :env),
       teams_notified: config.dig(:opts, :teams),
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 
@@ -83,7 +83,7 @@ def handle_alert(payload, config)
       alert_id: payload["alert_id"],
       level: alert_level,
       channels_notified: config.dig(:opts, :notify_channels),
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 
@@ -98,7 +98,7 @@ def handle_generic(payload, config)
       handler: "Team1Handler",
       event_type: payload["event_type"],
       environment: config.dig(:opts, :env),
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end

spec/acceptance/plugins/handlers/test_handler.rb

@@ -7,7 +7,7 @@ def call(payload:, headers:, config:)
       handler: "TestHandler",
       payload_received: payload,
       config_opts: config[:opts],
-      timestamp: Time.now.iso8601
+      timestamp: Time.now.utc.iso8601
     }
   end
 end

spec/integration/global_lifecycle_hooks_spec.rb

@@ -92,7 +92,7 @@ def call(payload:, headers:, config:)
           {
             status: "success",
             handler: "IntegrationTestHandler",
-            timestamp: Time.now.iso8601,
+            timestamp: Time.now.utc.iso8601,
             payload_received: !payload.nil?
           }
         end

spec/integration/hooks_integration_spec.rb

@@ -48,7 +48,7 @@ def call(payload:, headers:, config:)
             status: "test_success",
             payload_received: payload,
             config_opts: config[:opts],
-            timestamp: Time.now.iso8601
+            timestamp: Time.now.utc.iso8601
           }
         end
       end