Don't match getty on bad uri with colon at start of string

nulib · Dec 12, 2024 · 82686fa · 82686fa
1 parent 33a4a20
commit 82686fa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/authoritex/getty/base.ex b/lib/authoritex/getty/base.ex
@@ -18,7 +18,9 @@ defmodule Authoritex.Getty.Base do
 
       @impl Authoritex
       def can_resolve?(unquote(http_uri) <> _id), do: true
-      def can_resolve?(unquote(prefix) <> _id), do: true
+      def can_resolve?(unquote(prefix) <> _ = id) do
+        unquote(prefix) != ":"
+      end
       def can_resolve?(_), do: false
 
       @impl Authoritex

diff --git a/test/authoritex_test.exs b/test/authoritex_test.exs
@@ -44,6 +44,11 @@ defmodule AuthoritexTest do
       assert Authoritex.fetch("info:fake/no-authority/12345") ==
                {:error, :unknown_authority}
     end
+
+    test "bad uri" do
+      assert Authoritex.fetch(":http://id.loc.gov/authorities/names/no2009131449") ==
+               {:error, :unknown_authority}
+    end
   end
 
   describe "search/2" do