Merge pull request #52 from sbc100/fix_openssl_random_usage

Use SecureRandom rather than OpenSSL::Random
mdp · Apr 21, 2016 · 27a59b3 · 27a59b3
2 parents c73784e + daaaa11
commit 27a59b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/rotp.rb b/lib/rotp.rb
@@ -1,6 +1,6 @@
 require 'cgi'
 require 'uri'
-require 'openssl'
+require 'securerandom'
 require 'rotp/base32'
 require 'rotp/otp'
 require 'rotp/hotp'

diff --git a/lib/rotp/base32.rb b/lib/rotp/base32.rb
@@ -15,7 +15,7 @@ def decode(str)
 
       def random_base32(length=16)
         b32 = String.new
-        OpenSSL::Random.random_bytes(length).each_byte do |b|
+        SecureRandom.random_bytes(length).each_byte do |b|
           b32 << CHARS[b % 32]
         end
         b32