Memoize session.aggregation_key (#1892)

Author: sl0thentr0py

CHANGELOG.md

@@ -5,6 +5,11 @@
 - Support rack 3 [#1884](https://github.com/getsentry/sentry-ruby/pull/1884)
   - We no longer need the `HTTP_VERSION` check for ignoring the header
 
+### Bug Fixes
+
+- Memoize session.aggregation_key [#1892](https://github.com/getsentry/sentry-ruby/pull/1892)
+  - Fixes [#1891](https://github.com/getsentry/sentry-ruby/issues/1891)
+
 ## 5.4.2
 
 ### Bug Fixes

sentry-ruby/lib/sentry/session.rb

@@ -2,7 +2,7 @@
 
 module Sentry
   class Session
-    attr_reader :started, :status
+    attr_reader :started, :status, :aggregation_key
 
     # TODO-neel add :crashed after adding handled mechanism
     STATUSES = %i(ok errored exited)
@@ -11,6 +11,10 @@ class Session
     def initialize
       @started = Sentry.utc_now
       @status = :ok
+
+      # truncate seconds from the timestamp since we only care about
+      # minute level granularity for aggregation
+      @aggregation_key = Time.utc(@started.year, @started.month, @started.day, @started.hour, @started.min)
     end
 
     # TODO-neel add :crashed after adding handled mechanism
@@ -22,12 +26,6 @@ def close
       @status = :exited if @status == :ok
     end
 
-    # truncate seconds from the timestamp since we only care about
-    # minute level granularity for aggregation
-    def aggregation_key
-      Time.utc(started.year, started.month, started.day, started.hour, started.min)
-    end
-
     def deep_dup
       dup
     end