fix(HTTPRequest): small overflow on mac in setRequestTarget if the ta…

…rget is an empty string
dantol29 · May 17, 2024 · 60cd38d · 60cd38d
1 parent c494d30
commit 60cd38d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/HTTPRequest.cpp b/src/HTTPRequest.cpp
@@ -130,7 +130,7 @@ void HTTPRequest::setMethod(std::string method)
 
 void HTTPRequest::setRequestTarget(std::string requestTarget)
 {
-	if (requestTarget != "/" && requestTarget[requestTarget.size() - 1] == '/')
+	if (!requestTarget.empty() && requestTarget != "/" && requestTarget.back() == '/')
 		requestTarget = requestTarget.substr(0, requestTarget.size() - 1);
 	_requestTarget = requestTarget;
 }

diff --git a/webserv_default.conf b/webserv_default.conf
@@ -24,4 +24,11 @@ server {
 	allow_methods GET POST DELETE;
 	autoindex off;
 	root var/;
+}
+
+server {
+	listen 8080;
+	server_name www.php_site;
+	allow_methods GET POST DELETE;
+	root var/;
 }