Memoize the existing_scanners method

ddb38cf1 · Mehmet Emin INAC · 82143017 · ddb38cf1
Commit ddb38cf1 authored Sep 04, 2020 by Mehmet Emin INAC
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

ee/app/services/security/store_findings_metadata_service.rb ee/app/services/security/store_findings_metadata_service.rb +4 -4

No files found.
--- a/ee/app/services/security/store_findings_metadata_service.rb
+++ b/ee/app/services/security/store_findings_metadata_service.rb
@@ -56,10 +56,10 @@ module Security
    end

    def existing_scanners
-      project.vulnerability_scanners
-             .with_external_id(scanner_external_ids)
-             .group_by(&:external_id)
-             .transform_values(&:first)
+      @existing_scanners ||= project.vulnerability_scanners
+                                    .with_external_id(scanner_external_ids)
+                                    .group_by(&:external_id)
+                                    .transform_values(&:first)
    end

    def scanner_external_ids