BUGFIX: use a more widely compatible version of sadd

This commit is contained in:
Sam 2014-01-15 12:38:59 +11:00
parent 7122fe0eed
commit aa577f11fd

View File

@ -41,7 +41,10 @@ class CommonPasswords
def self.load_passwords
passwords = File.readlines(PASSWORD_FILE)
redis.sadd LIST_KEY, passwords[0,5000].map!(&:chomp)
passwords[0,5000].map!(&:chomp).each do |pwd|
# slower, but a tad more compatible
redis.sadd LIST_KEY, pwd
end
rescue Errno::ENOENT
# tolerate this so we don't block signups
Rails.logger.error "Common passwords file #{PASSWORD_FILE} is not found! Common password checking is skipped."