Worked on the jj22ee's comment and added logic that convert in second…

…s only for rails 7.1 versions.
aws · Aug 21, 2024 · 373b1ac · 373b1ac
1 parent c5e7f56
commit 373b1ac
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/aws-xray-sdk/facets/rails/active_record.rb b/lib/aws-xray-sdk/facets/rails/active_record.rb
@@ -28,10 +28,9 @@ def record(transaction)
           subsegment = XRay.recorder.begin_subsegment name, namespace: 'remote'
           # subsegment is nil in case of context missing
           return if subsegment.nil?
-
-          subsegment.start_time = convert_time_in_seconds(transaction.time.to_f)
+          subsegment.start_time = (::Rails::VERSION::MAJOR == 7 and ::Rails::VERSION::MINOR == 1) ? convert_time_in_seconds(transaction.time.to_f) : transaction.time.to_f
           subsegment.sql = sql
-          XRay.recorder.end_subsegment end_time: convert_time_in_seconds(transaction.end.to_f)
+          XRay.recorder.end_subsegment end_time: (::Rails::VERSION::MAJOR == 7 and ::Rails::VERSION::MINOR == 1) ? convert_time_in_seconds(transaction.end.to_f) : transaction.end.to_f
         end
 
         private