[491] Bug fix - Sidekiq Redis Auth issue (#527)

* When the the .env file has line with REDIS_PASSWORD set as empty, the value for this in the initializers comes as an empty string "".
* Fixed this in a way that, if it's empty string, then it's taken as `nil` value so that password is skipped
This commit is contained in:
Sony Mathew 2020-02-20 10:52:26 +05:30 committed by GitHub
parent 30e5edf6dc
commit 04c62417fa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 6 additions and 6 deletions

View file

@ -1,7 +1,7 @@
development:
adapter: redis
url: <%= ENV.fetch('REDIS_URL', 'redis://127.0.0.1:6379') %>
password: <%= ENV.fetch('REDIS_PASSWORD', nil) %>
password: <%= ENV.fetch('REDIS_PASSWORD', nil).presence %>
test:
adapter: test
@ -9,9 +9,9 @@ test:
staging:
adapter: redis
url: <%= ENV.fetch('REDIS_URL', 'redis://127.0.0.1:6379') %>
password: <%= ENV.fetch('REDIS_PASSWORD', nil) %>
password: <%= ENV.fetch('REDIS_PASSWORD', nil).presence %>
production:
adapter: redis
url: <%= ENV.fetch('REDIS_URL', 'redis://127.0.0.1:6379') %>
password: <%= ENV.fetch('REDIS_PASSWORD', nil) %>
password: <%= ENV.fetch('REDIS_PASSWORD', nil).presence %>

View file

@ -1,6 +1,6 @@
app_redis_config = {
url: URI.parse(ENV.fetch('REDIS_URL', 'redis://127.0.0.1:6379')),
password: ENV.fetch('REDIS_PASSWORD', nil)
url: URI.parse(ENV.fetch('REDIS_URL', 'redis://127.0.0.1:6379')),
password: ENV.fetch('REDIS_PASSWORD', nil).presence
}
redis = Rails.env.test? ? MockRedis.new : Redis.new(app_redis_config)
Nightfury.redis = Redis::Namespace.new('reports', redis: redis)

View file

@ -1,6 +1,6 @@
sidekiq_redis_config = {
url: ENV.fetch('REDIS_URL', 'redis://127.0.0.1:6379'),
password: ENV.fetch('REDIS_PASSWORD', nil)
password: ENV.fetch('REDIS_PASSWORD', nil).presence
}
Sidekiq.configure_client do |config|