kotovalexarian-likes-github/mperham--sidekiq
1
0
Fork 0
mirror of https://github.com/mperham/sidekiq.git synced 2022-11-09 13:52:34 -05:00
Code Releases Activity

Merge pull request #1349 from jonhyman/feature/password-scrub

Browse source 
Don't log Redis AUTH requirepass.
This commit is contained in:
Mike Perham 2013-11-17 07:35:11 -08:00
commit c9eba2789d
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
lib/sidekiq/redis_connection.rb
View file

@ -1,5 +1,6 @@
require 'connection_pool'
require 'redis'
require 'uri'
module Sidekiq
class RedisConnection
@ -53,10 +54,16 @@ module Sidekiq
end
def log_info(options)
# Don't log Redis AUTH password
scrubbed_options = options.dup
if scrubbed_options[:url] && (uri = URI.parse(scrubbed_options[:url])) && uri.password
uri.password = "REDACTED"
scrubbed_options[:url] = uri.to_s
end
if Sidekiq.server?
Sidekiq.logger.info("Booting Sidekiq #{Sidekiq::VERSION} with redis options #{options}")
Sidekiq.logger.info("Booting Sidekiq #{Sidekiq::VERSION} with redis options #{scrubbed_options}")
else
Sidekiq.logger.info("#{Sidekiq::NAME} client with redis options #{options}")
Sidekiq.logger.info("#{Sidekiq::NAME} client with redis options #{scrubbed_options}")
end
end