Unverified Commit 0b7d4780 authored by Hong Minhee's avatar Hong Minhee
Browse files

Fix `lookupWebFinger()` throwing `TypeError` on redirection to relative URI 

Fix https://github.com/dahlia/fedify/issues/166
parent 6ed53bed

CHANGES.md

+5 −0
Original line number Diff line number Diff line
@@ -8,6 +8,11 @@ Version 0.15.9 


To be released.



 -  Fixed a bug where `lookupWebFinger()` function had thrown a `TypeError`

    when the *.well-known/webfinger* redirects to a relative URI.  [[#166]]



[#166]: https://github.com/dahlia/fedify/issues/166





Version 0.15.8

--------------

src/webfinger/lookup.test.ts

+5 −2
Original line number Diff line number Diff line
@@ -76,8 +76,11 @@ test("lookupWebFinger()", async (t) => { 


  mf.mock(

    "GET@/.well-known/webfinger",

    (req) =>

      Response.redirect(new URL("/.well-known/webfinger2", req.url), 302),

    (_) =>

      new Response("", {

        status: 302,

        headers: { Location: "/.well-known/webfinger2" },

      }),

  );

  mf.mock(

    "GET@/.well-known/webfinger2",

src/webfinger/lookup.ts

+4 −1
Original line number Diff line number Diff line
@@ -48,7 +48,10 @@ export async function lookupWebFinger( 
      response.status >= 300 && response.status < 400 &&

      response.headers.has("Location")

    ) {

      url = new URL(response.headers.get("Location")!);

      url = new URL(

        response.headers.get("Location")!,

        response.url == null || response.url === "" ? url : response.url,

      );

      continue;

    }

    if (!response.ok) {