Merge pull request #1967 from SumOfUs/deploy-patch

Nil guard for deployments
SumOfUs · Jun 10, 2022 · 3a69824 · 3a69824
2 parents 9f58b10 + eb12e68
commit 3a69824
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/app/lib/secrets_manager.rb b/app/lib/secrets_manager.rb
@@ -20,7 +20,6 @@ def secret_name(secret_id)
       if secret_id.include? '/'
         secret_id
       else
-        puts "Prefix is ##{prefix}"
         [prefix, secret_id].compact.join('/')
       end
     end

diff --git a/config/initializers/application_secrets.rb b/config/initializers/application_secrets.rb
@@ -1,6 +1,6 @@
 require './app/lib/secrets_manager'
 
-if Rails.env == 'production'
+if Rails.env == 'production' && Settings.aws_secrets_manager_prefix.present?
   omniauth_secrets = SecretsManager.get_value('omniauth')
   ak_secrets = SecretsManager.get_value('prod/actionKitApi')
   database_secrets = SecretsManager.get_value('champaignDB')