From ccf0e78b910e9a691f72f132993cc25605ffd8fa Mon Sep 17 00:00:00 2001 From: David Cook Date: Thu, 18 Jan 2024 12:26:21 +1100 Subject: [PATCH] Refactor It's not as beautiful as I hoped, but it enforces the rule. Am I going too far? Probably. --- spec/support/vcr_setup.rb | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/spec/support/vcr_setup.rb b/spec/support/vcr_setup.rb index f240f96ba1..d7c6e9b6fb 100644 --- a/spec/support/vcr_setup.rb +++ b/spec/support/vcr_setup.rb @@ -7,8 +7,10 @@ VCR.configure do |config| config.hook_into :webmock config.ignore_localhost = true config.configure_rspec_metadata! - config.filter_sensitive_data('') { ENV.fetch('STRIPE_SECRET_TEST_API_KEY', nil) } - config.filter_sensitive_data('') { ENV.fetch('STRIPE_CUSTOMER', nil) } - config.filter_sensitive_data('') { ENV.fetch('STRIPE_ACCOUNT', nil) } config.ignore_hosts('localhost', '127.0.0.1', '0.0.0.0', 'api.knapsackpro.com') + + # Filter sensitive environment variables + ['STRIPE_SECRET_TEST_API_KEY', 'STRIPE_CUSTOMER', 'STRIPE_ACCOUNT'].each do |env_var| + config.filter_sensitive_data("") { ENV.fetch(env_var, nil) } + end end