Simplify timestamp validator retrieval by removing memoization

GrantBirki · GrantBirki · commit fc11d283e422 · 2025-06-17T10:54:20.000-07:00
diff --git a/lib/hooks/plugins/auth/base.rb b/lib/hooks/plugins/auth/base.rb
@@ -58,7 +58,7 @@ def self.fetch_secret(config, secret_env_key_name: :secret_env_key)
         #
         # @return [TimestampValidator] Singleton timestamp validator instance
         def self.timestamp_validator
-          @timestamp_validator ||= TimestampValidator.new
+          TimestampValidator.new
         end
 
         # Find a header value by name with case-insensitive matching

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ def self.fetch_secret(config, secret_env_key_name: :secret_env_key)`
`58`	`58`	`#`
`59`	`59`	`# @return [TimestampValidator] Singleton timestamp validator instance`
`60`	`60`	`def self.timestamp_validator`
`61`		`- @timestamp_validator \|\|= TimestampValidator.new`
	`61`	`+ TimestampValidator.new`
`62`	`62`	`end`
`63`	`63`
`64`	`64`	`# Find a header value by name with case-insensitive matching`