Don't warn on Logger.warn

lib/x509/logger.ex

@@ -0,0 +1,16 @@
+defmodule X509.Logger do
+  @moduledoc false
+  alias X509.Util
+
+  require Logger
+
+  if Util.app_version(:logger) >= [1, 11, 0] do
+    def warn(message, metadata \\ []) do
+      Logger.warning(message, metadata)
+    end
+  else
+    def warn(message, metadata \\ []) do
+      Logger.warn(message, metadata)
+    end
+  end
+end

lib/x509/test/crl_server.ex

@@ -106,7 +106,7 @@ defmodule X509.Test.CRLServer do
       {:ok, :http_eoh} ->
         case Map.get(crl_map, path) do
           nil ->
-            X509.Util.warn("No CRL defined for #{path}")
+            X509.Logger.warn("No CRL defined for #{path}")
             respond(socket, 404)
             :gen_tcp.close(socket)
 

lib/x509/test/suite.ex

@@ -581,7 +581,7 @@ defmodule X509.Test.Suite do
         %__MODULE__{valid: valid, chain: chain, server_key: server_key},
         scenario
       ) do
-    X509.Util.warn("Unknown scenario: #{scenario}")
+    X509.Logger.warn("Unknown scenario: #{scenario}")
 
     [
       cert: X509.Certificate.to_der(valid),

lib/x509/util.ex

@@ -1,8 +1,6 @@
 defmodule X509.Util do
   @moduledoc false
 
-  require Logger
-
   def app_version(application) do
     application
     |> Application.spec()
@@ -11,16 +9,4 @@ defmodule X509.Util do
     |> String.split(".")
     |> Enum.map(&String.to_integer/1)
   end
-
-  # Create a utility function that handles checking for the 
-  # existence of Logger.warning/2 if not fallback to Logger.warn/2
-  if macro_exported?(Logger, :warning, 2) do
-    def warn(message, metadata \\ []) do
-      Logger.warning(message, metadata)
-    end
-  else
-    def warn(message, metadata \\ []) do
-      Logger.warn(message, metadata)
-    end
-  end
 end

test/x509/test/server_test.exs

@@ -840,7 +840,7 @@
       end
     end
   else
-    X509.Util.warn("ECDSA certificates can't be tested on the current OTP version")
+    X509.Logger.warn("ECDSA certificates can't be tested on the current OTP version")
   end
 
   #
@@ -897,7 +897,7 @@
   defp create_pem_files(context) do
     tmp_dir =
       System.tmp_dir!()
       |> Path.join("x509_server_test#{System.get_pid()}")
 
     File.mkdir(tmp_dir)