more type fixes

Author: ndbroadbent

lib/log_struct/integrations/shrine.rb

@@ -45,10 +45,10 @@ def self.setup(config)
             Log::Shrine::Upload.new(
               storage: storage_sym,
               location: payload[:location],
-              uploader: payload[:uploader],
+              uploader: payload[:uploader]&.to_s,
               upload_options: payload[:upload_options],
               options: payload[:options],
-              duration_ms: event.duration,
+              duration_ms: event.duration.to_f,
               additional_data: payload.except(:storage, :location, :uploader, :upload_options, :options)
             )
           when Event::Download

lib/log_struct/log/shrine/metadata.rb

@@ -36,7 +36,7 @@ class Metadata < T::Struct
 
         # Event-specific fields
         const :storage, Symbol
-        const :location, String
+        const :location, T.nilable(String), default: nil
         const :metadata, T.nilable(T::Hash[String, T.untyped]), default: nil
 
         # Additional data
@@ -63,13 +63,13 @@ def self.base_hash
             timestamp: T.untyped).returns(T::Hash[LogStruct::LogField, T.untyped])
         }
         def self.build(storage:,
-          location:,
+          location: nil,
           metadata: nil,
           additional_data: nil,
           timestamp: Time.now)
           h = base_hash
           h[LogField::Storage] = storage
-          h[LogField::Location] = location
+          h[LogField::Location] = location unless location.nil?
           h[LogField::Metadata] = metadata unless metadata.nil?
           h
         end

schemas/log_sources/shrine.yml

@@ -39,7 +39,9 @@ events:
   Metadata:
     fields:
       Storage: Symbol
-      Location: String
+      Location:
+        type: String
+        required: false
       Metadata:
         type: 'T::Hash[String, T.untyped]'
         required: false