Add a comment around the use of safe_constantize

It triggers a Brakeman error that can be safely ignored
openfoodfoundation · Aug 11, 2023 · 7a563cd · 7a563cd
1 parent e9aa1da
commit 7a563cd
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/app/controllers/admin/vouchers_controller.rb b/app/controllers/admin/vouchers_controller.rb
@@ -9,6 +9,8 @@ def new
     end
 
     def create
+      # The use of "safe_constantize" here will trigger a Brakeman error, it can safely be ignored
+      # as it's a false positive : https://github.com/openfoodfoundation/openfoodnetwork/pull/10821
       voucher_type = params[:vouchers_flat_rate][:voucher_type]
       if Voucher::TYPES.include?(voucher_type)
         @voucher = voucher_type.safe_constantize.create(