Issue279 replace with alias #280

ayase-mstk · 2024-07-26T07:58:47Z

request uriがファイルパスである時に、location uriの部分をaliasに差し替えるようにしました。

ex)
listen4242;
locatino /alias/ {
   alias test;
}


localhost:4242/alias/index.html
↓
localhost:4242/test/index.html

YungTatyu

対応ありがとうございます。
挙動問題なさそうでした。

YungTatyu · 2024-07-26T11:43:19Z

src/http/HttpResponse.cpp

+      if (pos != std::string::npos) request.uri_ = request.uri_.substr(pos + location->uri_.length());
+    }


これはロジック的に、みつからないことはあるんですか？
if (pos != std::string::npos)

必ず見つかるはずですが一応つけておきました。気になるようでしたら、削除します！

ayase-mstk added 2 commits July 26, 2024 16:54

🐛 replace root with alias if request uri is a file path

a69bf4a

⚡ add alias test case in which request uri is a file path

6ddbe21

YungTatyu approved these changes Jul 26, 2024

View reviewed changes

YungTatyu merged commit dd2b225 into main Jul 26, 2024
5 checks passed

Provide feedback

		if (pos != std::string::npos) request.uri_ = request.uri_.substr(pos + location->uri_.length());
		}