From 0a6bce1384c8035ac9055db9e8699ed6a9625c2e Mon Sep 17 00:00:00 2001
From: shawn-higgins1 <23224097+shawn-higgins1@users.noreply.github.com>
Date: Tue, 23 Jul 2019 15:56:04 -0400
Subject: [PATCH] Fix health checks (#663)

---
 Gemfile                             |  2 +-
 Gemfile.lock                        |  2 --
 config/initializers/health_check.rb | 13 ++-----------
 3 files changed, 3 insertions(+), 14 deletions(-)

diff --git a/Gemfile b/Gemfile
index 72aaf145..46c7bbfe 100644
--- a/Gemfile
+++ b/Gemfile
@@ -35,7 +35,7 @@ gem 'turbolinks', '~> 5'
 gem 'jbuilder', '~> 2.5'
 
 # Use Redis adapter to run Action Cable in production
-gem 'redis', '~> 3.0'
+# gem 'redis', '~> 3.0'
 
 # Use ActiveModel has_secure_password
 gem 'bcrypt', '~> 3.1.7'
diff --git a/Gemfile.lock b/Gemfile.lock
index bb53ab03..e2cf0654 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -248,7 +248,6 @@ GEM
     recaptcha (5.0.0)
       json
     redcarpet (3.4.0)
-    redis (3.3.5)
     remote_syslog_logger (1.0.4)
       syslog_protocol
     rolify (5.2.0)
@@ -373,7 +372,6 @@ DEPENDENCIES
   random_password
   recaptcha
   redcarpet
-  redis (~> 3.0)
   remote_syslog_logger
   rolify
   rspec-rails (~> 3.7)
diff --git a/config/initializers/health_check.rb b/config/initializers/health_check.rb
index 44fd9203..35e5acf9 100644
--- a/config/initializers/health_check.rb
+++ b/config/initializers/health_check.rb
@@ -23,17 +23,11 @@ HealthCheck.setup do |config|
 
   config.http_status_for_error_object = 500
 
-  # bucket names to test connectivity - required only if s3 check used, access permissions can be mixed
-  config.buckets = { 'bucket_name' => [:R, :W, :D] }
-
   # You can customize which checks happen on a standard health check, eg to set an explicit list use:
-  config.standard_checks = %w(database migrations custom)
-
-  # Or to exclude one check:
-  config.standard_checks -= %w(emailconf)
+  config.standard_checks = %w(database migrations emailconf)
 
   # You can set what tests are run with the 'full' or 'all' parameter
-  config.full_checks = %w(database migrations custom email cache redis resque-redis sidekiq-redis s3)
+  config.full_checks = %w(database migrations email cache)
 
   # max-age of response in seconds
   # cache-control is public when max_age > 1 and basic_auth_username is not set
@@ -43,7 +37,4 @@ HealthCheck.setup do |config|
 
   # http status code used when the ip is not allowed for the request
   config.http_status_for_ip_whitelist_error = 403
-
-  # When redis url is non-standard
-  config.redis_url = 'redis_url'
 end