Merge pull request #2413 from alphagov/ignore-signon-permissions-tokens

theseanything · web-flow · commit 771622f4ff22 · 2023-10-09T09:13:25.000+01:00
Ignore reporting signon permission token expiry metrics
diff --git a/lib/collectors/global_prometheus_collector.rb b/lib/collectors/global_prometheus_collector.rb
@@ -25,7 +25,7 @@ def metrics
     def token_expiry_info
       # Cache metric to prevent needless expensive calls to the database
       Rails.cache.fetch("token_expiry_info", expires_in: 1.hour) do
-        ApiUser.all.flat_map do |user|
+        ApiUser.where.not(email: SSOPushCredential::USER_EMAIL).flat_map do |user|
           user.authorisations.where(revoked_at: nil).map do |token|
             {
               expires_at: token.expires_at.to_i,

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ def metrics`
`25`	`25`	`def token_expiry_info`
`26`	`26`	`# Cache metric to prevent needless expensive calls to the database`
`27`	`27`	`Rails.cache.fetch("token_expiry_info", expires_in: 1.hour) do`
`28`		`- ApiUser.all.flat_map do \|user\|`
	`28`	`+ ApiUser.where.not(email: SSOPushCredential::USER_EMAIL).flat_map do \|user\|`
`29`	`29`	`user.authorisations.where(revoked_at: nil).map do \|token\|`
`30`	`30`	`{`
`31`	`31`	`expires_at: token.expires_at.to_i,`