Handle all blog URLs with redirect

lib/jola_dev_web/plugs/blog_redirect.ex

@@ -11,14 +11,17 @@ defmodule JolaDevWeb.Plugs.BlogRedirect do
       ids = JolaDev.Blog.ids()
       path = strip_path(conn.request_path)
 
-      if path in ids do
-        conn
-        |> put_resp_header("location", "https://jola.dev/posts/" <> path)
-        |> send_resp(:moved_permanently, "")
-        |> halt()
-      else
-        conn
-      end
+      path =
+        if path in ids do
+          "posts/" <> path
+        else
+          path
+        end
+
+      conn
+      |> put_resp_header("location", "https://jola.dev/" <> path)
+      |> send_resp(:moved_permanently, "")
+      |> halt()
     else
       conn
     end

test/jola_dev_web/plugs/blog_redirect_test.exs

@@ -18,16 +18,18 @@ defmodule JolaDevWeb.Plugs.BlogRedirectTest do
       assert conn.halted
     end
 
-    test "does not redirect invalid paths on blog.jola.dev", %{conn: conn} do
-      original_conn =
+    test "redirects invalid paths on blog.jola.dev to jola.dev without /posts prefix", %{
+      conn: conn
+    } do
+      conn =
         conn
         |> Map.put(:host, "blog.jola.dev")
         |> Map.put(:request_path, "/non-existent-post")
+        |> BlogRedirect.call([])
 
-      result_conn = BlogRedirect.call(original_conn, [])
-
-      assert result_conn == original_conn
-      refute result_conn.halted
+      assert conn.status == 301
+      assert get_resp_header(conn, "location") == ["https://jola.dev/non-existent-post"]
+      assert conn.halted
     end
 
     test "does not redirect on non-blog.jola.dev hosts", %{conn: conn} do