chore: Fix urlsafe_base64/1 generation

The test assumes 22 byte default, but had 32 because it was double-base64 encoding (once
regular, and once URL-safe). Without the double encoding, the tests fail because they were
expecting 32 (then 24), but the intent was URL-safe base encoding without padding.

Originally based on patricksrobertson#13, but fixes the tests and
padding.

Author: halostatue

lib/secure_random.ex

@@ -71,8 +71,8 @@ defmodule SecureRandom do
   """
   def urlsafe_base64(n \\ @default_length) do
     n
-    |> base64()
-    |> Base.url_encode64(padding: true)
+    |> random_bytes()
+    |> Base.url_encode64(padding: false)
   end
 
   @doc """

test/secure_random_test.exs

@@ -28,11 +28,11 @@ defmodule SecureRandomTest do
   end
 
   test "urlsafe_base64/1 defaults to a string length of 22" do
-    assert 32 == String.length(SecureRandom.urlsafe_base64())
+    assert 22 == String.length(SecureRandom.urlsafe_base64())
   end
 
   test "urlsafe_base64/1 takes a byte_length" do
-    assert 12 == String.length(SecureRandom.urlsafe_base64(4))
+    assert 6 == String.length(SecureRandom.urlsafe_base64(4))
   end
 
   test "uuid/0 returns a Binary string" do