Return webfinger for ap_ids.
authorRoger Braun <roger@rogerbraun.net>
Mon, 1 May 2017 11:51:17 +0000 (13:51 +0200)
committerRoger Braun <roger@rogerbraun.net>
Mon, 1 May 2017 11:51:17 +0000 (13:51 +0200)
lib/pleroma/web/web_finger/web_finger.ex
test/web/web_finger/web_finger_test.exs

index 7ceca042b5703d80dadb6e54241e64306ac96201..f8f4d5e4206c028d0311e33d450ef8758c44426f 100644 (file)
@@ -18,11 +18,15 @@ defmodule Pleroma.Web.WebFinger do
   def webfinger(resource) do
     host = Pleroma.Web.host
     regex = ~r/(acct:)?(?<username>\w+)@#{host}/
-    case Regex.named_captures(regex, resource) do
-      %{"username" => username} ->
-        user = User.get_by_nickname(username)
+    with %{"username" => username} <- Regex.named_captures(regex, resource) do
+      user = User.get_by_nickname(username)
+      {:ok, represent_user(user)}
+    else _e ->
+      with user when not is_nil(user) <- User.get_cached_by_ap_id(resource) do
         {:ok, represent_user(user)}
-      _ -> nil
+      else _e ->
+        {:error, "Couldn't find user"}
+      end
     end
   end
 
index 303abe5294c78ca4eabb75615d04bbf202f00474..b48fdd0aa9939b532d2637b472684dbbf5868370 100644 (file)
@@ -11,6 +11,22 @@ defmodule Pleroma.Web.WebFingerTest do
     end
   end
 
+  describe "incoming webfinger request" do
+    test "works for fqns" do
+      user = insert(:user)
+
+      {:ok, result} = WebFinger.webfinger("#{user.nickname}@#{Pleroma.Web.host}")
+      assert is_binary(result)
+    end
+
+    test "works for ap_ids" do
+      user = insert(:user)
+
+      {:ok, result} = WebFinger.webfinger(user.ap_id)
+      assert is_binary(result)
+    end
+  end
+
   describe "fingering" do
     test "returns the info for a user" do
       user = "shp@social.heldscal.la"