Use duration_ms helper

Author: solnic

sentry-rails/lib/sentry/rails/log_subscribers/action_controller_subscriber.rb

@@ -28,7 +28,6 @@ class ActionControllerSubscriber < Sentry::Rails::LogSubscriber
         # @param event [ActiveSupport::Notifications::Event] The controller action event
         def process_action(event)
           payload = event.payload
-          duration = event.duration.round(2)
 
           controller = payload[:controller]
           action = payload[:action]
@@ -39,7 +38,7 @@ def process_action(event)
             controller: controller,
             action: action,
             status: status,
-            duration_ms: duration,
+            duration_ms: duration_ms(event),
             method: payload[:method],
             path: payload[:path],
             format: payload[:format]

sentry-rails/lib/sentry/rails/log_subscribers/action_mailer_subscriber.rb

@@ -27,12 +27,12 @@ class ActionMailerSubscriber < Sentry::Rails::LogSubscriber
         # @param event [ActiveSupport::Notifications::Event] The email delivery event
         def deliver(event)
           payload = event.payload
+
           mailer = payload[:mailer]
-          duration = duration_ms(event)
 
           attributes = {
             mailer: mailer,
-            duration_ms: duration,
+            duration_ms: duration_ms(event),
             perform_deliveries: payload[:perform_deliveries]
           }
 

sentry-rails/lib/sentry/rails/log_subscribers/active_record_subscriber.rb

@@ -37,13 +37,12 @@ def sql(event)
           # Rails 5.0.0 doesn't include :cached in the payload, it was added in Rails 5.1
           cached = event.payload.fetch(:cached, false)
           connection_id = event.payload[:connection_id]
-          duration = duration_ms(event)
 
           db_config = extract_db_config(event.payload)
 
           attributes = {
             sql: sql,
-            duration_ms: duration,
+            duration_ms: duration_ms(event),
             cached: cached
           }