Actually set Client#client_id when creating a client

theplant · Aug 3, 2012 · 4202975 · 4202975
1 parent fe32afa
commit 4202975
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/rack/oauth2/models.rb b/lib/rack/oauth2/models.rb
@@ -13,8 +13,8 @@ def new_instance(klass, fields)
         end
 
         # Long, random and hexy.
-        def secure_random
-          OpenSSL::Random.random_bytes(32).unpack("H*")[0]
+        def secure_random bytes = 32
+          OpenSSL::Random.random_bytes(bytes).unpack("H*")[0]
         end
 
         # A ::DB object.

diff --git a/lib/rack/oauth2/models/active_record/client.rb b/lib/rack/oauth2/models/active_record/client.rb
@@ -32,6 +32,7 @@ def create(args)
                         :revoked=>nil }
 
             fields[:secret] = Server.secure_random
+            fields[:client_id] = Server.secure_random 4
 
             create! fields
           end