From fc72e681017e4253980ba7943549f6716c959b6a Mon Sep 17 00:00:00 2001
From: "Juan J. Martinez" <jjm@usebox.net>
Date: Wed, 24 Nov 2021 19:26:43 +0000
Subject: Fix: host subcompoent in the URL is case insensitive

Referemce: https://datatracker.ietf.org/doc/html/rfc3986/#section-3.2.2
---
 server/src/net/usebox/gemini/server/Server.scala |  2 +-
 server/test/src/ServerSpec.scala                 | 24 ++++++++++++++++++++++++
 2 files changed, 25 insertions(+), 1 deletion(-)

(limited to 'server')

diff --git a/server/src/net/usebox/gemini/server/Server.scala b/server/src/net/usebox/gemini/server/Server.scala
index a14d829..ce840df 100644
--- a/server/src/net/usebox/gemini/server/Server.scala
+++ b/server/src/net/usebox/gemini/server/Server.scala
@@ -72,7 +72,7 @@ case class Server(conf: ServiceConf) {
           uri.getScheme(),
           uri.getHost(),
           uri.getPath().decode(),
-          vHosts.find(_.host == uri.getHost())
+          vHosts.find(_.host == uri.getHost().toLowerCase)
         ) match {
           case (null, _, _, _) =>
             logger.debug(s"no scheme")
diff --git a/server/test/src/ServerSpec.scala b/server/test/src/ServerSpec.scala
index ea40798..c9a6947 100644
--- a/server/test/src/ServerSpec.scala
+++ b/server/test/src/ServerSpec.scala
@@ -152,6 +152,11 @@ class ServerSpec extends AnyFlatSpec with Matchers {
       .handleReq("gemini://localhost:8080/", "127.0.0.1") should be(a[Success])
   }
 
+  it should "handle host case insensitive" in {
+    Server(TestData.conf)
+      .handleReq("gemini://LOCALHOST/", "127.0.0.1") should be(a[Success])
+  }
+
   it should "return proxy request refused when the vhost is not found" in {
     Server(TestData.conf)
       .handleReq("gemini://otherhost/", "127.0.0.1") should be(
@@ -461,6 +466,25 @@ class ServerSpec extends AnyFlatSpec with Matchers {
     }
   }
 
+  it should "execute a CGI: host is case-insensitive and the value in the conf is used" in {
+    Server(TestData.cgiConf).handleReq(
+      "gemini://LOCALHOST/dir/cgi",
+      "127.0.0.1"
+    ) should matchPattern {
+      case Cgi(
+            _,
+            _,
+            "",
+            "",
+            "cgi",
+            TestData.host,
+            TestData.portStr,
+            _,
+            _
+          ) =>
+    }
+  }
+
   it should "execute a CGI: query string" in {
     Server(TestData.cgiConf).handleReq(
       "gemini://localhost/dir/cgi?query&string",
-- 
cgit v1.2.3