Fix bug with broken requests

lib/plug/forward_request.ex

@@ -21,7 +21,7 @@ defmodule Heimdall.Plug.ForwardRequest do
   def init(opts), do: opts
 
   defp build_request_path(path_info) do
-    "/" <> Enum.join(path_info)
+    "/" <> Enum.join(path_info, "/")
   end
 
   defp build_query_string(query_string) do

test/plug/forward_request_test.exs

@@ -16,26 +16,39 @@ defmodule Heimdall.Test.Plug.ForwardRequestTest do
     :ok
   end
 
-  test "call sends request to configured forward location" do
-    forward_url = %{"forward_url" => "http://localhost:8088"}
-    conn =
-      :get
-      |> conn("http://localhost/forward-test")
-      |> ForwardRequest.call(ForwardRequest.init(forward_url))
+  describe "call" do
+    test "sends request to configured forward location" do
+      forward_url = %{"forward_url" => "http://localhost:8088"}
+      conn =
+        :get
+        |> conn("http://localhost/forward-test")
+        |> ForwardRequest.call(ForwardRequest.init(forward_url))
 
-    assert conn.status == 200
-    assert conn.resp_body == "forwarded"
-  end
+      assert conn.status == 200
+      assert conn.resp_body == "forwarded"
+    end
 
-  test "call with opts sends request to passed forward location" do
-    forward_url = %{"forward_url" => "http://localhost:8088"}
-    conn =
-      :get
-      |> conn("http://wrong-place.com/forward-test")
-      |> ForwardRequest.call(ForwardRequest.init(forward_url))
+    test "with opts sends request to passed forward location" do
+      forward_url = %{"forward_url" => "http://localhost:8088"}
+      conn =
+        :get
+        |> conn("http://wrong-place.com/forward-test")
+        |> ForwardRequest.call(ForwardRequest.init(forward_url))
 
-    assert conn.status == 200
-    assert conn.resp_body == "forwarded"
+      assert conn.status == 200
+      assert conn.resp_body == "forwarded"
+    end
+
+    test "properly creates request with extended path" do
+      forward_url = %{"forward_url" => "http://localhost:8088"}
+      conn =
+        :get
+        |> conn("http://localhost/forward-test/with/more")
+        |> ForwardRequest.call(ForwardRequest.init(forward_url))
+
+      assert conn.status == 200
+      assert conn.resp_body == "forwarded"
+    end
   end
 
   test "changing the path info changes the request path" do

test/test_router.ex

@@ -9,6 +9,11 @@ defmodule Heimdall.Test.TestRouter do
     |> resp(200, "forwarded")
   end
 
+  get "/forward-test/with/more" do
+    conn
+    |> resp(200, "forwarded")
+  end
+
   get "/test" do
     conn
     |> resp(200, "ok")