Merge remote-tracking branch 'origin/master' into eric_config_parser

+ minors changes
+ Super new epoll loop output :)

.gitignore

@@ -22,3 +22,5 @@ webserv
 *.log
 
 large.jpg
+webserv_tester
+webserv_tester2

1

@@ -0,0 +1,3 @@
+
+------
+

42.config

@@ -0,0 +1,23 @@
+
+server {
+
+
+	server_name server1;
+	listen 0.0.0.0:4040;
+	client_body_limit 1000;
+
+	index youpi.bla; # this is another comment
+
+	root ./YoupiBanane/;
+
+	error_page 404 ./www/error_pages/error_404.html;
+
+	location / {
+		allow_methods GET;
+	}
+
+	location /directory {
+		index youpi.bad_extention;
+	}
+
+}

Makefile

@@ -5,7 +5,8 @@ CXX			 =	c++
 CXXFLAGS	 =	-Wall -Wextra #-Werror
 CXXFLAGS	+=	$(HEADERS_D:%=-I%)
 CXXFLAGS	+=	-std=c++98
-CXXFLAGS	+=	-g3
+CXXFLAGS	+=	-g
+#CXXFLAGS	+=	-fno-limit-debug-info
 CXXFLAGS	+=	-MMD -MP #header dependencie
 #CXXFLAGS	+=	-O3
 
@@ -17,7 +18,7 @@ HEADERS_D	 =	srcs \
 
 SRCS_D		 =	srcs \
 				srcs/webserv \
-				srcs/config
+				srcs/config \
 
 SRCS		 =	main.cpp \
 				base.cpp init.cpp close.cpp epoll_update.cpp signal.cpp \
@@ -28,16 +29,19 @@ SRCS		 =	main.cpp \
 				extraConfig.cpp \
 				postProcessing.cpp \
 				utils.cpp \
-				cgi_script.cpp \
-				Client.cpp \
+				cgi.cpp \
+				Client.cpp Client_multipart_body.cpp \
 
 OBJS_D		 =	builds
 OBJS		 =	$(SRCS:%.cpp=$(OBJS_D)/%.o)
 DEPS		 =	$(OBJS:.o=.d) #header dependencie
 
-# --------------------
-# ------ RULES -------
-# --------------------
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - #
+#             .  target: prerequisites    .   $@ : target             #
+#    RULES    .          recipe           .   $< : 1st prerequisite   #
+#             .          @recipe (silent) .   $^ : all prerequisites  #
+#             .  target: VAR = assignment .   |  : order-only prereq. #
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - #
 
 all: $(NAME)
 
@@ -52,11 +56,23 @@ $(NAME): $(OBJS)
 	$(CXX) $^ -o $(NAME)
 	echo "$(_GREEN)\r\33[2K\r$(NAME) created 😎$(_END)"
 
+# CGI
+cgi:
+	make -C srcs/cgi-bin
+cgiclean:
+	make clean -C srcs/cgi-bin
+cgifclean:
+	make fclean -C srcs/cgi-bin
+cgire:
+	make re -C srcs/cgi-bin
+
 clean:
 	rm -rf $(OBJS_D)
+	echo "$(_RED).o Files Deleted  😱$(_END)"
 
 fclean: clean
 	rm -f $(NAME)
+	echo "$(_RED)$(NAME) Deleted  😱$(_END)"
 
 re: fclean all
 

README.md

@@ -1,17 +1,6 @@
 
 ## work together
 
-#### next commit
-
-#### questions
-- how should we handle a wrong url like `http://localhost/cgi-bin/wrong.phpp/good.php` ?
-	- do we serve `./srcs/cgi-bin/good.php` ?
-	- or do we return 404 "not found" ?
-->	- for now, execve would crash, but that doesn't produce a 404 error, rather a 500, is it bad ?
-	- could we use errno after execve to choose an appropriate http error ? subject says : "Checking the value of errno is strictly forbidden after a read or a write operation"
-- if a url has a file with extension, but it's not a cgi extension, is it necessary to look further ?
-	- ex. `http://localhost/file.php/file.py` for `cgi_ext py;` ?
-- the response page is received long after the cgi-script is done, why ?
 
 ---
 ## man
@@ -243,45 +232,61 @@ SERVER_SOFTWARE		: the server software you're using (e.g. Apache 1.3)
 REDIRECT_STATUS		: for exemple, 200
 ```
 
+g 50 34 48
+p 30 23 32
+l 20 14 20
+	 71
 
 ---
-## http errors
-
-#### HTTP Client Errors
-- 400 	Bad Request 							This error code indicates that the request cannot be processed because of wrong syntax usage by the client.
-- 401 	Unauthorized 							This error code indicates that the client is not authorized to receive the requested data, without authentication. A login name and password based authentication might be required to access the requested data.
-- 403 	Forbidden 								There is no way you can access the requested data. A 403 error announces that the data is off limits.
-- 404 	Not Found 								This error indicates that the resources requested by the client are currently unavailable.
-- 405 	Method Not Allowed 						This error indicates wrong usage of request method. Depending on the kind of data requested, the appropriate request method must be chosen.
-- 406 	Not Acceptable 							When the data provided by a web server does not match the specifications made in ‘Accept’ header of the client HTTP request, this error is the result.
-- 407 	Proxy Authentication Required 			This error clearly indicates that an authentication from the proxy server is required to gain access to requested resources.
-- 408 	Request Timeout 						This type of error indicates that the client was delayed in making a request, within the specified time allocated to it, by the server.
-- 409 	Conflict 								This error code is displayed when the server perceives a conflict between two requests made simultaneously by different clients, for the same resource.
-- 410 	Gone 									This error code indicates that the requested data is no longer hosted on the server and therefore further requests made for it, would be futile.
-- 411 	Length Required 						If the request made by the client does not include information about the length of the requested data or resource, this error code is displayed.
-- 412 	Precondition Failed 					Some requests made by clients come attached with conditions that need to be satisfied by the server, before data transaction may happen. If these conditions are not met, error 412 results.
-- 413 	Request Entity Too Large 				When a client makes a request which is too overwhelming for the server’s resources to handle, it presents this error code.
-- 414 	Requested URI Too Long 					A Uniform Resource Identifier (URI) is a character string used to describe a data stream or resource on a server. Error 414 occurs when the server is unable to process the URI, because of limited resources and long string length.
-- 415 	Unsupported Media Type 					A server may be designed to allow only certain formats for media files. When error 415 is displayed, it indicates that the format of file being uploaded through a client request, does not match the requisite format.
-- 416 	Request Range Not Satisfiable 			Sometimes, a client may request for only a small part of a file, instead of asking for the entire file. If this request is not specified properly and the part of the file requested does not exist, this error is displayed.
-- 417 	Expectation Failed 						This error code is displayed when the server cannot meet the specifications provided in the request.
-- 422 	Unprocessable Entity 					This error is displayed when the request made, cannot be processed due to an error in semantic structure.
-- 423 	Locked 									This error is displayed when a requested piece of data or resource has been locked, making it inaccessible for a server.
-- 424 	Failed Dependency			 			A server may process a succession of requests from a client with the fulfillment of each, dependent on the one made before. This error is displayed when a request made before is not fulfilled, due to which the current request cannot be processed.
-- 426 	Upgrade Required 						This error signifies that the client may need to switch over to a secure protocol like TLS to get the request processed.
-- 444 	No Response								This error signifies that the server has simply rejected the client request and terminated connection.
-- 449 	Retry With 								This is a request made by the server to the client, to make the request again after executing certain actions or making specific changes in request. This is an error code introduced by Microsoft.
-- 499 	Client Closed Request	 				When client terminates a connection made with the server, while its processing the associated request, this error code is displayed.
-- 450 	Blocked By Windows Parental Controls 	Another error code introduced by Microsoft, this one is displayed when a URL is blocked by parental control settings on the web browsers.
-
-#### HTTP Server Errors
-- 500 	Internal Server Error 					A generic message displayed by the server, when the problem with the request cannot be specified by any other appropriate code.
-- 501 	Not Implemented							This error indicates the inability of the server to process a request, as it hasn’t been configured to respond to the request method used.
-- 502 	Bad Gateway 							Sometimes, hosted pages on web servers are transmitted to client via proxy servers. If the proxy server (to which a client has sent a request), fails connecting with the web server (known as the upstream server), error 502 results.
-- 503 	Service Unavailable 					When the server is already overloaded with multiple requests, it will temporarily stop entertaining new requests, by displaying a 503 error code.
-- 504 	Gateway Timeout 						When the request made by a proxy server to the web server hosting a resource times out, error 504 is reported.
-- 505 	HTTP Version Not Supported 				An error code seen rarely, it is displayed when the web server does not support the protocol version of the client request.
+## http status
+[rfc 2616](https://datatracker.ietf.org/doc/html/rfc2616#section-10)
 
+#### Informational
+- 100	Continue
+- 101	Switching Protocols
+#### Successful
+- 200	OK
+- 201	Created
+- 202	Accepted
+- 203	Non-Authoritative Information
+- 204	No Content
+- 205	Reset Content
+- 206	Partial Content
+#### Redirection
+- 300	Multiple Choices
+- 301	Moved Permanently
+- 302	Found
+- 303	See Other
+- 304	Not Modified
+- 305	Use Proxy
+- 306	(Unused)
+- 307	Temporary Redirect
+#### Client Error
+- 400	Bad Request
+- 401	Unauthorized
+- 402	Payment Required
+- 403	Forbidden
+- 404	Not Found
+- 405	Method Not Allowed
+- 406	Not Acceptable
+- 407	Proxy Authentication Required
+- 408	Request Timeout
+- 409	Conflict
+- 410	Gone
+- 411	Length Required
+- 412	Precondition Failed
+- 413	Request Entity Too Large
+- 414	Request-URI Too Long
+- 415	Unsupported Media Type
+- 416	Requested Range Not Satisfiable
+- 417	Expectation Failed
+#### Server Error
+- 500	Internal Server Error
+- 501	Not Implemented
+- 502	Bad Gateway
+- 503	Service Unavailable
+- 504	Gateway Timeout
+- 505	HTTP Version Not Supported
 
 ---
 ## ressources

Tester/expected_results/expected_path_test.txt

@@ -0,0 +1,116 @@
+telnet> Trying 127.0.0.1...
+Connected to localhost.
+Escape character is '^]'.
+HTTP/1.1 200 OK
+Server: Webserv/0.1
+Connection: keep-alive
+Content-Type: text/html; charset=UTF-8
+Content-Length: 193
+
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Le Webserv</title>
+</head>
+<body>
+	<h1 style="text-align:center">Le index (˘ ͜ʖ˘)</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>
+HTTP/1.1 200 OK
+Server: Webserv/0.1
+Connection: keep-alive
+Content-Type: text/html; charset=UTF-8
+Content-Length: 290
+
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test index</title>
+<!--	<link rel="stylesheet" href="stylesheet/style.css"> -->
+	<link rel="stylesheet" href="/stylesheet/style.css">
+	
+</head>
+<body>
+	<h1>Webserv Test Index</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>
+HTTP/1.1 200 OK
+Server: Webserv/0.1
+Connection: keep-alive
+Content-Type: text/html; charset=UTF-8
+Content-Length: 290
+
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test index</title>
+<!--	<link rel="stylesheet" href="stylesheet/style.css"> -->
+	<link rel="stylesheet" href="/stylesheet/style.css">
+	
+</head>
+<body>
+	<h1>Webserv Test Index</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>
+HTTP/1.1 
+Server: Webserv/0.1
+Connection: keep-alive
+Content-Type: text/html; charset=UTF-8
+Content-Length: 1277
+
+<!DOCTYPE html><html><head><title>Index of ./Tester/www/</title></head><body><h1>Index of ./Tester/www/</h1><hr><pre><a style="font-size:1.5em" href="/list/..">..</a>
+<a style="font-size:1.5em" href="/list/upload_form.html">upload_form.html</a>
+<a style="font-size:1.5em" href="/list/form_get.html">form_get.html</a>
+<a style="font-size:1.5em" href="/list/kermit.ico">kermit.ico</a>
+<a style="font-size:1.5em" href="/list/upload_form_single.html">upload_form_single.html</a>
+<a style="font-size:1.5em" href="/list/Cagneyc_intro.gif">Cagneyc_intro.gif</a>
+<a style="font-size:1.5em" href="/list/Van_Eyck_Portrait_Arnolfini.jpg">Van_Eyck_Portrait_Arnolfini.jpg</a>
+<a style="font-size:1.5em" href="/list/root.png">root.png</a>
+<a style="font-size:1.5em" href="/list/test">test</a>
+<a style="font-size:1.5em" href="/list/drill.jpg">drill.jpg</a>
+<a style="font-size:1.5em" href="/list/punpun.png">punpun.png</a>
+<a style="font-size:1.5em" href="/list/favicon.ico">favicon.ico</a>
+<a style="font-size:1.5em" href="/list/index.html">index.html</a>
+<a style="font-size:1.5em" href="/list/rfc2119.html">rfc2119.html</a>
+<a style="font-size:1.5em" href="/list/error_pages">error_pages</a>
+<a style="font-size:1.5em" href="/list/rfc2119_files">rfc2119_files</a>
+</pre><hr></body></html>HTTP/1.1 404 Not Found
+Server: Webserv/0.1
+Connection: keep-alive
+Content-Type: text/html; charset=UTF-8
+Content-Length: 210
+
+<!DOCTYPE html>
+<html>
+<head>
+	<title>404 Not Found</title>
+</head>
+<body>
+	<h1 style="text-align:center">Check it UP 404 Not Found</h1>
+	<hr>
+	<p style=\"text-align:center\">Le Webserv/0.1</p>
+</body>
+</html>
+HTTP/1.1 404 Not Found
+Server: Webserv/0.1
+Connection: keep-alive
+Content-Type: text/html; charset=UTF-8
+Content-Length: 210
+
+<!DOCTYPE html>
+<html>
+<head>
+	<title>404 Not Found</title>
+</head>
+<body>
+	<h1 style="text-align:center">Check it UP 404 Not Found</h1>
+	<hr>
+	<p style=\"text-align:center\">Le Webserv/0.1</p>
+</body>
+</html>

Tester/telnet_test.sh

@@ -0,0 +1,95 @@
+#! /bin/bash
+
+
+# DO I EVEN NEED THIS FILE ? could i just put this stuff in maint_test.sh? or would it not modify the vars...
+
+
+# you need to put absolutely everything in ""
+
+
+#test_file=$1
+#source $test_file
+
+connect_to_telnet="open $host $port"
+
+
+start_telnet()
+{
+	echo "open $host $port"
+	sleep 1
+}
+
+
+run()
+{
+	echo "$connect_to_telnet"
+	sleep 1
+
+	run_this_test
+
+}
+
+
+#### DEPRICATED ######
+
+run_telnet()
+{
+	echo "$connect_to_telnet"
+	sleep 1
+	echo -e "$request"
+	echo
+	sleep 1
+
+# ret = $(arg | telnet)
+
+}
+
+run_a_test()
+{
+	{
+	echo "----- $test_name -----"
+	echo -e "$_RED$request$_END"
+	} &> test.log
+
+	echo -e "$request" | telnet
+}
+
+
+#######
+# This is where stuff is launched
+#######
+
+#rm -rf telnet.out
+
+#./webserv $config_file 2>&1 > webserv.log & run_all
+#./webserv $config_file 2>&1 > webserv.log &
+#./webserv $config_file &> /dev/null &
+#./webserv $config_file 1>&1 > webserv.log &
+
+#./webserv $config_file &> webserv.log &
+
+#run_all
+#run_this_test
+
+#echo -e "${_GREEN}Running Telnet Test on '$test_name'${_END}"
+
+#sleep 1
+
+#run | telnet >> telnet.out
+#run | telnet
+
+#pkill webserv
+
+
+
+
+
+
+
+
+
+
+
+
+
+

Tester/telnet_test2.sh

@@ -0,0 +1,16 @@
+#! /bin/bash
+
+echo "open duckduckgo.com 80"
+sleep 1
+echo "GET / HTTP/1.1"
+echo "Host: duckduckgo.com"
+echo
+echo
+sleep 2
+
+#(
+#echo open duckduckgo.com 80
+#echo "GET / HTTP/1.1"
+#echo "Host: duckduckgo.com"
+#echo "exit"
+#) | telnet

Tester/test1.config

@@ -0,0 +1,97 @@
+
+server {
+
+# this is a comment 
+
+	server_name server1;
+
+	listen 0.0.0.0:4040;
+
+#	client_body_limit asdfa;
+	client_body_limit 5000;
+					 # Max == 18446744073709551615 / 1024 == 18014398509481984
+
+	index index.html; # this is another comment
+	#index mdr.html; # this is another comment
+
+	root ./Tester/www/;
+
+	error_page 404 ./Tester/www/error_pages/error_404.html;
+
+
+
+	location / {
+		allow_methods GET;
+		root ./Tester/www/;
+	}
+
+	location /srcs/cgi-bin/ {
+		root ./srcs/cgi-bin/;
+		allow_methods POST;
+		cgi_ext php;
+	}
+
+	location /list {
+		autoindex on;
+	}
+
+	location /cgi-bin {
+		root ./srcs/cgi-bin/;
+		cgi_ext out php sh;
+	}
+
+	location /upload {
+		allow_methods POST;
+		autoindex on;
+		upload_dir ./Tester/www/user_files/;
+		# root doesn’t matter if used only with POST and no CGI
+	}
+
+	location /the_dump {
+		allow_methods GET DELETE;
+		root ./Tester/www/user_files;
+		autoindex on;
+	}
+
+	location /redirect {
+		redirect 307 https://fr.wikipedia.org/wiki/Ketchup;
+#		redirect 307 https://www.youtube.com/watch?v=rG6b8gjMEkw;
+	}
+
+	location /test {
+		index index1.html subdex.html;
+		root ./Tester/www/test/;
+	}
+
+	location /stylesheet {
+		root ./stylesheet/;
+	}
+
+	location /test/index1.html {
+		root ./Tester/www/test/index1.html;
+		index index1.html subdex.html;
+	}
+
+	location /hilarious_404/ {
+		redirect 301 https://berniesanders.com/404/;
+	}
+
+	location /test/something.html {
+	#	allow_methods DELETE;
+		root ./Tester/www/test/something.html;
+	}
+
+	location /test/test_deeper/ {
+#		allow_methods
+		root ./Tester/www/test/test_deeper/;
+	}
+
+	location /test/test_deeper/super_deep {
+		root ./Tester/www/test/test_deeper/super_deep/;
+	}
+
+#	location /test/test_deeper/something.html {
+#		allow_methods DELETE;
+#	}
+
+}

Tester/test2.config

@@ -0,0 +1,95 @@
+
+server {
+
+# this is a comment 
+
+	server_name server1;
+
+	listen 0.0.0.0:8080;
+
+	client_body_limit 1;
+					 # Max == 18446744073709551615 / 1024 == 18014398509481984
+
+	index index.html; # this is another comment
+
+	root ./Tester/www/;
+
+	error_page 404 ./Tester/www/error_pages/error_404.html;
+
+
+
+	location / {
+		allow_methods GET;
+		root ./Tester/www/;
+	}
+
+	location /srcs/cgi-bin/ {
+		root ./srcs/cgi-bin/;
+		allow_methods POST;
+		cgi_ext php;
+	}
+
+	location /list {
+		autoindex on;
+	}
+
+	location /cgi-bin {
+		root ./srcs/cgi-bin/;
+		cgi_ext out php sh;
+	}
+
+	location /upload {
+		allow_methods POST;
+		autoindex on;
+		upload_dir ./Tester/www/user_files/;
+		# root doesn’t matter if used only with POST and no CGI
+	}
+
+	location /the_dump {
+		allow_methods GET DELETE;
+		root ./Tester/www/user_files;
+		autoindex on;
+	}
+
+	location /redirect {
+		redirect 307 https://fr.wikipedia.org/wiki/Ketchup;
+#		redirect 307 https://www.youtube.com/watch?v=rG6b8gjMEkw;
+	}
+
+	location /test {
+		index index1.html subdex.html;
+		root ./Tester/www/test/;
+	}
+
+	location /stylesheet {
+		root ./stylesheet/;
+	}
+
+	location /test/index1.html {
+		root ./Tester/www/test/index1.html;
+		index index1.html subdex.html;
+	}
+
+	location /hilarious_404/ {
+		redirect 301 https://berniesanders.com/404/;
+	}
+
+	location /test/something.html {
+	#	allow_methods DELETE;
+		root ./Tester/www/test/something.html;
+	}
+
+	location /test/test_deeper/ {
+#		allow_methods
+		root ./Tester/www/test/test_deeper/;
+	}
+
+	location /test/test_deeper/super_deep {
+		root ./Tester/www/test/test_deeper/super_deep/;
+	}
+
+#	location /test/test_deeper/something.html {
+#		allow_methods DELETE;
+#	}
+
+}

Tester/test3.config

@@ -0,0 +1,98 @@
+
+server {
+
+# this is a comment 
+
+	server_name server1;
+
+	listen 8080;
+#	listen 70000;
+
+#	client_body_limit asdfa;
+	client_body_limit 1000;
+					 # Max == 18446744073709551615 / 1024 == 18014398509481984
+
+	index index.html; # this is another comment
+	#index mdr.html; # this is another comment
+
+	root ./Tester/www/;
+
+	error_page 404 ./Tester/www/error_pages/error_404.html;
+
+
+
+	location / {
+		allow_methods GET;
+		root ./Tester/www/;
+	}
+
+	location /srcs/cgi-bin/ {
+		root ./srcs/cgi-bin/;
+		allow_methods POST;
+		cgi_ext php;
+	}
+
+	location /list {
+		autoindex on;
+	}
+
+	location /cgi-bin {
+		root ./srcs/cgi-bin/;
+		cgi_ext out php sh;
+	}
+
+	location /upload {
+		allow_methods POST;
+		autoindex on;
+		upload_dir ./Tester/www/user_files/;
+		# root doesn’t matter if used only with POST and no CGI
+	}
+
+	location /the_dump {
+		allow_methods GET DELETE;
+		root ./Tester/www/user_files;
+		autoindex on;
+	}
+
+	location /redirect {
+		redirect 307 https://fr.wikipedia.org/wiki/Ketchup;
+#		redirect 307 https://www.youtube.com/watch?v=rG6b8gjMEkw;
+	}
+
+	location /test {
+		index index1.html subdex.html;
+		root ./Tester/www/test/;
+	}
+
+	location /stylesheet {
+		root ./stylesheet/;
+	}
+
+	location /test/index1.html {
+		root ./Tester/www/test/index1.html;
+		index index1.html subdex.html;
+	}
+
+	location /hilarious_404/ {
+		redirect 301 https://berniesanders.com/404/;
+	}
+
+	location /test/something.html {
+	#	allow_methods DELETE;
+		root ./Tester/www/test/something.html;
+	}
+
+	location /test/test_deeper/ {
+#		allow_methods
+		root ./Tester/www/test/test_deeper/;
+	}
+
+	location /test/test_deeper/super_deep {
+		root ./Tester/www/test/test_deeper/super_deep/;
+	}
+
+#	location /test/test_deeper/something.html {
+#		allow_methods DELETE;
+#	}
+
+}

Tester/test_body.sh

@@ -0,0 +1,88 @@
+#! /bin/bash
+
+test_name="Body Test"
+
+config_file="./Tester/test2.config"
+port=8080
+host="localhost"
+
+#paths=("/" "/test" "/test/" "/list" "list" "/wrong")	# you can add many
+paths=("/")	# you can add many
+#methods=("GET" "POST" "DELETE")
+methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+
+header="Host: $host\n"
+body="this is a message \n"
+body+="this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message this is a message \n"
+
+# turns out this does nothing...
+#header_cycle=("Content-Length: 17\n" "Content-Length: 14" "Content-Length: 25")
+#header_cycle=("Content-Length: 17\n" "\n")
+#header_cycle=("Content-Length: 17\n")
+#header_cycle=()
+#header+="Content-Length: 8\n"
+
+run_this_test()
+{
+	echo size is ${#body} >> telnet.log
+	
+#	for i in "${header_cycle[@]}"
+	for i in "${paths[@]}"
+	do
+#		l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+		l1="${methods[0]} $i ${httpz[0]}"
+#		header_send="$header$i"
+		header_send="$header"
+	## if you add an extra \n after header send you get a status code 200 OK and 400 bad request, but like both
+	#	request="$l1\n$header_send\n$body\n"
+		request="$l1\n$header_send\n$body"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+		echo -e "$request"
+		sleep 1
+		echo -e "\n" >> telnet.log
+	done
+
+}
+
+
+
+# expected result...
+
+
+files=()
+
+file="expected_body_test.txt"
+files+=("expected_body_test.txt")
+#files+=("expected_path_root_test.txt")
+#files+=("expected_path_testnoslash_test.txt")
+#files+=("expected_path_testslash_test.txt")
+#files+=("expected_path_list_test.txt")
+#files+=("expected_path_badlist_test.txt")
+#files+=("expected_path_wrong_test.txt")
+
+
+#local_expected_test_file=$file
+local_expected_test_files=()
+test_path=""
+
+add_path()
+{
+	for i in "${files[@]}"
+	do
+		local_expected_test_files+=("$test_path$i")
+	done
+}
+
+#add_path
+
+
+
+

Tester/test_header.sh

@@ -0,0 +1,83 @@
+#! /bin/bash
+
+test_name="Header Test"
+
+config_file="./Tester/test1.config"
+port=4040
+host="localhost"
+
+paths=("/")	# you can add many
+#methods=("GET" "POST" "DELETE")
+methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+
+# the parts we will send to webserv
+
+
+# let main.sh handle the l1
+
+l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+
+header="Host: $host\n"
+header+="Nonsense: fu\n"
+
+header_cycle=("Transfer-encoding: fu" "Content-encoding: fu")
+
+#header+="Transfer-encoding: fu\n"
+#header+="Content-endcoding: fu\n"
+body=
+
+
+run_this_test()
+{
+	
+	for i in "${header_cycle[@]}"
+	do
+		header_send="$header$i"
+		request="$l1\n$header_send\n$body\n"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+		echo -e "$request"
+		sleep 1
+		echo -e "\n\n" >> telnet.log
+	done
+
+}
+
+
+
+# expected result...
+
+files=()
+
+file="expected_header_test.txt"
+files+=("expected_header_test.txt")
+#files+=("expected_path_root_test.txt")
+
+
+local_expected_test_files=()
+test_path=""
+
+add_path()
+{
+	for i in "${files[@]}"
+	do
+		local_expected_test_files+=("$test_path$i")
+	done
+}
+
+#add_path
+
+
+
+
+
+
+
+
+

Tester/test_method.sh

@@ -0,0 +1,83 @@
+#! /bin/bash
+
+test_name="Method Test"
+
+config_file="./Tester/test1.config"
+port=4040
+host="localhost"
+
+paths=("/")	# you can add many
+methods=("GET" "POST" "DELETE" "GUMBALL")
+#methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+
+# the parts we will send to webserv
+
+
+# let main.sh handle the l1
+
+
+
+header="Host: $host\n"
+header+="Nonsense: fu\n"
+
+#header_cycle=("Transfer-encoding: fu" "Content-encoding: fu")
+
+#header+="Transfer-encoding: fu\n"
+#header+="Content-endcoding: fu\n"
+body=
+
+
+run_this_test()
+{
+	
+	for i in "${methods[@]}"
+	do
+		l1="$i ${paths[0]} ${httpz[0]}"
+		request="$l1\n$header\n$body\n"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+		echo -e "$request"
+		sleep 1
+		echo -e "\n\n" >> telnet.log
+	done
+
+}
+
+
+
+# expected result...
+
+files=()
+
+file="expected_method_test.txt"
+files+=("expected_method_test.txt")
+#files+=("expected_path_root_test.txt")
+
+
+local_expected_test_files=()
+test_path=""
+
+add_path()
+{
+	for i in "${files[@]}"
+	do
+		local_expected_test_files+=("$test_path$i")
+	done
+}
+
+#add_path
+
+
+
+
+
+
+
+
+

Tester/test_path.sh

@@ -0,0 +1,74 @@
+#! /bin/bash
+
+test_name="Path Test"
+
+config_file="./Tester/test1.config"
+port=4040
+host="localhost"
+
+paths=("/" "/test" "/test/" "/list" "list" "/wrong")	# you can add many
+#paths=("/")	# you can add many
+#methods=("GET" "POST" "DELETE")
+methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+
+header="Host: $host"
+body=
+
+
+run_this_test()
+{
+	
+	for i in "${paths[@]}"
+	do
+		l1="${methods[0]} $i ${httpz[0]}"
+		request="$l1\n$header\n$body\n"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+		echo -e "$request"
+		sleep 1
+		echo -e "\n" >> telnet.log
+	done
+
+}
+
+
+
+# expected result...
+
+
+files=()
+
+file="expected_path_test.txt"
+files+=("expected_path_test.txt")
+#files+=("expected_path_root_test.txt")
+#files+=("expected_path_testnoslash_test.txt")
+#files+=("expected_path_testslash_test.txt")
+#files+=("expected_path_list_test.txt")
+#files+=("expected_path_badlist_test.txt")
+#files+=("expected_path_wrong_test.txt")
+
+
+#local_expected_test_file=$file
+local_expected_test_files=()
+test_path=""
+
+add_path()
+{
+	for i in "${files[@]}"
+	do
+		local_expected_test_files+=("$test_path$i")
+	done
+}
+
+#add_path
+
+
+
+

Tester/test_port.sh

@@ -0,0 +1,74 @@
+#! /bin/bash
+
+test_name="Port Test"
+
+config_file="./Tester/test3.config"
+port=8080
+host="localhost"
+
+#paths=("/" "/test" "/test/" "/list" "list" "/wrong")	# you can add many
+paths=("/")	# you can add many
+#methods=("GET" "POST" "DELETE")
+methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+
+header="Host: $host"
+body=
+
+
+run_this_test()
+{
+	
+	for i in "${paths[@]}"
+	do
+		l1="${methods[0]} $i ${httpz[0]}"
+		request="$l1\n$header\n$body\n"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+		echo -e "$request"
+		sleep 1
+		echo -e "\n" >> telnet.log
+	done
+
+}
+
+
+
+# expected result...
+
+
+files=()
+
+file="expected_path_test.txt"
+files+=("expected_path_test.txt")
+#files+=("expected_path_root_test.txt")
+#files+=("expected_path_testnoslash_test.txt")
+#files+=("expected_path_testslash_test.txt")
+#files+=("expected_path_list_test.txt")
+#files+=("expected_path_badlist_test.txt")
+#files+=("expected_path_wrong_test.txt")
+
+
+#local_expected_test_file=$file
+local_expected_test_files=()
+test_path=""
+
+add_path()
+{
+	for i in "${files[@]}"
+	do
+		local_expected_test_files+=("$test_path$i")
+	done
+}
+
+#add_path
+
+
+
+

Tester/test_template.sh

@@ -0,0 +1,62 @@
+#! /bin/bash
+
+test_name="Template test all good"
+
+config_file="./Tester/test1.config"
+port=4040
+host="localhost"
+
+paths=("/")	# you can add many
+methods=("GET" "POST" "DELETE")
+#methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+
+# the parts we will send to webserv
+
+
+# let main.sh handle the l1
+
+l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+
+header="Host: $host"
+body=
+
+
+run_this_test()
+{
+	
+	for i in "${methods[@]}"
+	do
+		l1="$i ${paths[0]} ${httpz[0]}"
+		request="$l1\n$header\n$body\n"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+#		} &>> test.log
+#		} 2>>&1
+#		} &>> /dev/stdout
+		echo -e "$request"
+#		echo
+		sleep 1
+#		{
+#			echo -e "\n------\n"
+#		} > /dev/stdout
+	#	echo -e "\n\n\n------\n\n\n" >> telnet.out 
+	#	echo -e "\n\n\n------\n\n\n" &> /dev/stdout
+		echo -e "\n\n" >> telnet.log
+#		echo -e "\n\n------\n\n" >> telnet.out
+	#	echo -e "\n------\n" > /dev/stdout
+#		run_a_test
+#		echo
+	done
+
+}
+
+
+
+# expected result...
+

Tester/test_valid_uri.sh

@@ -0,0 +1,66 @@
+#! /bin/bash
+
+test_name="Valid URIs Test"
+
+config_file="./Tester/test1.config"
+port=4040
+host="localhost"
+
+paths=("/" "/stylesheet/style.css" "/test/something.html" "/test/something.html/" "/test/" "/list" "/hilarious_404" "/redirect")	# you can add many
+#paths=("/")	# you can add many
+#methods=("GET" "POST" "DELETE")
+methods=("GET")
+httpz=("HTTP/1.1")
+
+
+
+l1="${methods[0]} ${paths[0]} ${httpz[0]}"
+
+header="Host: $host"
+body=
+
+
+run_this_test()
+{
+	
+	for i in "${paths[@]}"
+	do
+		l1="${methods[0]} $i ${httpz[0]}"
+		request="$l1\n$header\n$body\n"
+		{
+			echo "----- $test_name -----"
+			echo -e "$_RED$request$_END"
+		} >> telnet.log
+		echo -e "$request"
+		sleep 1
+		echo -e "\n" >> telnet.log
+	done
+
+}
+
+
+
+# expected result...
+
+
+files=()
+
+file="expected_valid_uri_test.txt"
+files+=("expected_valid_files_test.txt")
+
+local_expected_test_files=()
+test_path=""
+
+add_path()
+{
+	for i in "${files[@]}"
+	do
+		local_expected_test_files+=("$test_path$i")
+	done
+}
+
+#add_path
+
+
+
+

Tester/www/error_pages/error_404.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>404 Not Found</title>
+</head>
+<body>
+	<h1 style="text-align:center">Check it UP 404 Not Found</h1>
+	<hr>
+	<p style=\"text-align:center\">Le Webserv/0.1</p>
+</body>
+</html>

Tester/www/index.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Le Webserv</title>
+</head>
+<body>
+	<h1 style="text-align:center">Le index (˘ ͜ʖ˘)</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>

Tester/www/rfc2119.html

@@ -0,0 +1,312 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<!-- saved from url=(0057)https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html -->
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"><head profile="http://dublincore.org/documents/2008/08/04/dc-html/"><meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
+        
+        <meta name="robots" content="index,follow">
+
+        <link rel="icon" href="https://www.rfc-editor.org/rfc/inline-errata/css/images/rfc.png" type="image/png">
+        <link rel="shortcut icon" href="https://www.rfc-editor.org/rfc/inline-errata/css/images/rfc.png" type="image/png">
+        <title>rfc2119</title>
+    
+        <link rel="stylesheet" type="text/css" href="./rfc2119_files/errata-base.css">
+        <link rel="stylesheet" type="text/css" href="./rfc2119_files/errata-color.css" title="Default: Basic Colors">
+        <link rel="alternative stylesheet" type="text/css" href="./rfc2119_files/errata-monochrome.css" title="Monochrome">
+        <link rel="alternative stylesheet" type="text/css" href="./rfc2119_files/errata-printer.css" title="Printer">
+
+        <script src="./rfc2119_files/errata.js"></script>
+    </head>
+    <body>
+        <div class="Verified-headnote-styling">
+<span style="font-weight: bold;">This is a purely informative rendering of an RFC that includes verified errata.  This rendering may not be used as a reference.</span>
+<br>
+<br>
+The following 'Verified' errata have been incorporated in this document:
+ <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#eid493">EID 493</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#btn_494">EID 494</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#btn_495">EID 495</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#btn_494">EID 496</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#eid498">EID 498</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#btn_499">EID 499</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#eid500">EID 500</a>,  <a href="https://www.rfc-editor.org/rfc/inline-errata/rfc2119.html#btn_5101">EID 5101</a>
+</div>
+
+        <pre>Network Working Group                                         S. Bradner
+Request for Comments: 2119                            Harvard University
+BCP: 14                                                       March 1997
+Category: Best Current Practice
+
+
+        Key words for use in RFCs to Indicate Requirement Levels
+
+Status of this Memo
+
+   This document specifies an Internet Best Current Practices for the
+   Internet Community, and requests discussion and suggestions for
+   improvements.  Distribution of this memo is unlimited.
+
+Abstract
+
+   In many standards track documents several words are used to signify
+   the requirements in the specification.  These words are often
+   capitalized.  This document defines these words as they should be
+   interpreted in IETF documents.  Authors who follow these guidelines
+   should incorporate this phrase near the beginning of their document:
+
+      <span class="Verified-inline-styling" id="inline-499">       The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL <button id="btn_499" target="expand_499" onclick="hideFunction(&quot;expand_499&quot;)">Expand</button>
+       NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT 
+       RECOMMENDED",  "MAY", and "OPTIONAL" in this document are to 
+       be interpreted as described in RFC 2119.</span>
+<div class="nodeCloseClass" id="expand_499"><div class="Verified-endnote-styling" id="eid499">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid499">EID 499</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> Abstract
+
+<b>Original Text:</b>
+
+       The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL
+       NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED",  "MAY", and
+       "OPTIONAL" in this document are to be interpreted as described in
+       RFC 2119.
+
+<b>Corrected Text:</b>
+
+       The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL
+       NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT 
+       RECOMMENDED",  "MAY", and "OPTIONAL" in this document are to 
+       be interpreted as described in RFC 2119.
+</pre>
+<b>Notes:</b><br>
+The phrase "NOT RECOMMENDED" is missing from this sentence.
+</div>
+</div>
+   Note that the force of these words is modified by the requirement
+   level of the document in which they are used.
+
+<span class="Verified-inline-styling" id="inline-495">1. MUST   This word, or the terms "REQUIRED" or "SHALL", means that the <button id="btn_495" target="expand_495" onclick="hideFunction(&quot;expand_495&quot;)">Expand</button>
+   definition is an absolute requirement of the specification.
+<div class="Verified-endnote-styling" id="eid493">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid493">EID 493</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 1
+
+<b>Original Text:</b>
+
+2. MUST NOT   This phrase, or the phrase "SHALL NOT", mean that the
+   definition is an absolute prohibition of the specification.
+
+<b>Corrected Text:</b>
+
+2. MUST NOT   This phrase, or the phrase "SHALL NOT", means that the
+   definition is an absolute prohibition of the specification.
+</pre>
+<b>Notes:</b><br>
+
+</div>
+<div class="Verified-endnote-styling" id="eid498">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid498">EID 498</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 1
+
+<b>Original Text:</b>
+
+4. SHOULD NOT   This phrase, or the phrase "NOT RECOMMENDED" mean that
+   there may exist valid reasons in particular circumstances when the
+   particular behavior is acceptable or even useful, but the full
+   implications should be understood and the case carefully weighed
+   before implementing any behavior described with this label.
+
+<b>Corrected Text:</b>
+
+4. SHOULD NOT   This phrase, or the phrase "NOT RECOMMENDED", means that
+   there may exist valid reasons in particular circumstances when the
+   particular behavior is acceptable or even useful, but the full
+   implications should be understood and the case carefully weighed
+   before implementing any behavior described with this label.
+</pre>
+<b>Notes:</b><br>
+
+</div>
+<div class="Verified-endnote-styling" id="eid500">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid500">EID 500</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 1
+
+<b>Original Text:</b>
+
+3. SHOULD   This word, or the adjective "RECOMMENDED", mean that there
+   may exist valid reasons in particular circumstances to ignore a
+   particular item, but the full implications must be understood and
+   carefully weighed before choosing a different course.
+
+<b>Corrected Text:</b>
+
+3. SHOULD   This word, or the adjective "RECOMMENDED", means that there
+   may exist valid reasons in particular circumstances to ignore a
+   particular item, but the full implications must be understood and
+   carefully weighed before choosing a different course.
+</pre>
+<b>Notes:</b><br>
+
+</div>
+</span>
+<div class="nodeCloseClass" id="expand_495"><div class="Verified-endnote-styling" id="eid495">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid495">EID 495</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 1
+
+<b>Original Text:</b>
+
+1. MUST   This word, or the terms "REQUIRED" or "SHALL", mean that the
+   definition is an absolute requirement of the specification.
+
+
+<b>Corrected Text:</b>
+
+1. MUST   This word, or the terms "REQUIRED" or "SHALL", means that the
+   definition is an absolute requirement of the specification.
+</pre>
+<b>Notes:</b><br>
+ 
+</div>
+</div>
+2. MUST NOT   This phrase, or the phrase "SHALL NOT", mean that the
+   definition is an absolute prohibition of the specification.
+
+3. SHOULD   This word, or the adjective "RECOMMENDED", mean that there
+   may exist valid reasons in particular circumstances to ignore a
+   particular item, but the full implications must be understood and
+   carefully weighed before choosing a different course.
+
+4. SHOULD NOT   This phrase, or the phrase "NOT RECOMMENDED" mean that
+   there may exist valid reasons in particular circumstances when the
+   particular behavior is acceptable or even useful, but the full
+   implications should be understood and the case carefully weighed
+   before implementing any behavior described with this label.
+
+<span class="Verified-inline-styling" id="inline-5101">5. MAY   This word, or the adjective "OPTIONAL", mean that an item is <button id="btn_5101" target="expand_5101" onclick="hideFunction(&quot;expand_5101&quot;)">Expand</button>
+   truly optional.  One vendor may choose to include the item because a
+   particular marketplace requires it or because the vendor feels that
+   it enhances the product while another vendor may omit the same item.
+   An implementation which does not include a particular option MUST be
+   prepared to interoperate with another implementation which does
+   include the option, though perhaps with reduced functionality. In the
+   same vein an implementation which does include a particular option
+   MUST be prepared to interoperate with another implementation which
+   does not include the option (except, of course, for the feature the
+   option provides).</span>
+<div class="nodeCloseClass" id="expand_5101"><div class="Verified-endnote-styling" id="eid5101">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid5101">EID 5101</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 5
+
+<b>Original Text:</b>
+
+5. MAY   This word, or the adjective "OPTIONAL", mean that an item is
+   truly optional.  One vendor may choose to include the item because a
+   particular marketplace requires it or because the vendor feels that
+   it enhances the product while another vendor may omit the same item.
+   An implementation which does not include a particular option MUST be
+   prepared to interoperate with another implementation which does
+   include the option, though perhaps with reduced functionality. In the
+   same vein an implementation which does include a particular option
+   MUST be prepared to interoperate with another implementation which
+   does not include the option (except, of course, for the feature the
+   option provides.)
+
+<b>Corrected Text:</b>
+
+5. MAY   This word, or the adjective "OPTIONAL", mean that an item is
+   truly optional.  One vendor may choose to include the item because a
+   particular marketplace requires it or because the vendor feels that
+   it enhances the product while another vendor may omit the same item.
+   An implementation which does not include a particular option MUST be
+   prepared to interoperate with another implementation which does
+   include the option, though perhaps with reduced functionality. In the
+   same vein an implementation which does include a particular option
+   MUST be prepared to interoperate with another implementation which
+   does not include the option (except, of course, for the feature the
+   option provides).
+</pre>
+<b>Notes:</b><br>
+Full stop should appear outside the parentheses in the last sentence.
+</div>
+</div>
+6. Guidance in the use of these Imperatives
+
+   Imperatives of the type defined in this memo must be used with care
+   and sparingly.  In particular, they MUST only be used where it is
+   actually required for interoperation or to limit behavior which has
+   potential for causing harm <span class="Verified-inline-styling" id="inline-494">(e.g., limiting retransmissions)</span>  For <button id="btn_494" target="expand_494" onclick="hideFunction(&quot;expand_494&quot;)">Expand Multiple</button>
+<div class="nodeCloseClass" id="expand_494"><div class="Verified-endnote-styling" id="eid494">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid494">EID 494</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 6
+
+<b>Original Text:</b>
+
+(e.g., limiting retransmisssions)
+
+<b>Corrected Text:</b>
+
+(e.g., limiting retransmissions)
+</pre>
+<b>Notes:</b><br>
+
+</div>
+<div class="Verified-endnote-styling" id="eid496">
+<pre><b><i><a href="https://www.rfc-editor.org/errata/eid496">EID 496</a> (Verified) is as follows:</i></b>
+
+<b>Section:</b> 6
+
+<b>Original Text:</b>
+
+   In particular, they MUST only be used where it is actually required
+   for interoperation or to limit behavior which has potential for
+   causing harm (e.g., limiting retransmisssions)  For example, they
+   must not be used to try to impose a particular method on
+   implementors where the method is not required for interoperability.   
+
+<b>Corrected Text:</b>
+
+   In particular, they MUST only be used where it is actually required
+   for interoperation or to limit behavior which has potential for
+   causing harm (e.g., limiting retransmissions).  For example, they
+   must not be used to try to impose a particular method on
+   implementors where the method is not required for interoperability.
+</pre>
+<b>Notes:</b><br>
+
+</div>
+</div>   example, they must not be used to try to impose a particular method
+   on implementors where the method is not required for
+   interoperability.
+
+7. Security Considerations
+
+   These terms are frequently used to specify behavior with security
+   implications.  The effects on security of not implementing a MUST or
+   SHOULD, or doing something the specification says MUST NOT or SHOULD
+   NOT be done may be very subtle. Document authors should take the time
+   to elaborate the security implications of not following
+   recommendations or requirements as most implementors will not have
+   had the benefit of the experience and discussion that produced the
+   specification.
+
+8. Acknowledgments
+
+   The definitions of these terms are an amalgam of definitions taken
+   from a number of RFCs.  In addition, suggestions have been
+   incorporated from a number of people including Robert Ullmann, Thomas
+   Narten, Neal McBurnett, and Robert Elz.
+
+9. Author's Address
+
+      Scott Bradner
+      Harvard University
+      1350 Mass. Ave.
+      Cambridge, MA 02138
+
+      phone - +1 617 495 3864
+
+      email - sob@harvard.edu
+
+
+        
+        
+    
+
+</pre></body></html>

Tester/www/rfc2119_files/errata-base.css

@@ -0,0 +1,139 @@
+<style type="text/css">
+	@media only screen 
+	  and (min-width: 992px)
+	  and (max-width: 1199px) {
+	    body { font-size: 14pt; }
+            div.content { width: 96ex; margin: 0 auto; }
+        }
+	@media only screen 
+	  and (min-width: 768px)
+	  and (max-width: 991px) {
+            body { font-size: 14pt; }
+            div.content { width: 96ex; margin: 0 auto; }
+        }
+	@media only screen 
+	  and (min-width: 480px)
+	  and (max-width: 767px) {
+            body { font-size: 11pt; }
+            div.content { width: 96ex; margin: 0 auto; }
+        }
+	@media only screen 
+	  and (max-width: 479px) {
+            body { font-size: 8pt; }
+            div.content { width: 96ex; margin: 0 auto; }
+        }
+	@media only screen 
+	  and (min-device-width : 375px) 
+	  and (max-device-width : 667px) {
+            body { font-size: 9.5pt; }
+            div.content { width: 96ex; margin: 0; }
+        }
+	@media only screen 
+	  and (min-device-width: 1200px) {
+            body { font-size: 10pt; margin: 0 4em; }
+            div.content { width: 96ex; margin: 0; }
+        }
+        h1, h2, h3, h4, h5, h6, .h1, .h2, .h3, .h4, .h5, .h6 {
+	    font-weight: bold;
+            line-height: 0pt;
+            display: inline;
+            white-space: pre;
+            font-family: monospace;
+            font-size: 1em;
+	    font-weight: bold;
+        }
+        pre {
+            font-size: 1em;
+            margin-top: 0px;
+            margin-bottom: 0px;
+        }
+	.pre {
+	    white-space: pre;
+	    font-family: monospace;
+	}
+	.header{
+	    font-weight: bold;
+	}
+        .newpage {
+            page-break-before: always;
+        }
+        .invisible {
+            text-decoration: none;
+            color: white;
+        }
+        a.selflink {
+          color: black;
+          text-decoration: none;
+        }
+        @media print {
+            body {
+                font-family: monospace;
+                font-size: 10.5pt;
+            }
+            h1, h2, h3, h4, h5, h6 {
+                font-size: 1em;
+            }
+        
+            a:link, a:visited {
+                color: inherit;
+                text-decoration: none;
+            }
+            .noprint {
+                display: none;
+            }
+        }
+	@media screen {
+	    .grey, .grey a:link, .grey a:visited {
+		color: #777;
+	    }
+            .docinfo {
+                background-color: #EEE;
+            }
+            .top {
+                border-top: 7px solid #EEE;
+            }
+            .bgwhite  { background-color: white; }
+            .bgred    { background-color: #F44; }
+            .bggrey   { background-color: #666; }
+            .bgbrown  { background-color: #840; }            
+            .bgorange { background-color: #FA0; }
+            .bgyellow { background-color: #EE0; }
+            .bgmagenta{ background-color: #F4F; }
+            .bgblue   { background-color: #66F; }
+            .bgcyan   { background-color: #4DD; }
+            .bggreen  { background-color: #4F4; }
+
+            .legend   { font-size: 90%; }
+            .cplate   { font-size: 70%; border: solid grey 1px; }
+	}
+
+
+    .Verified-headnote-styling, .Held-headnote-styling, .Reported-headnote-styling, .Rejected-headnote-styling {
+        border:dashed;
+        margin:8px;
+        padding;24px;
+        overflow-wrap: normal;
+        padding: 1em;
+    }
+        
+    .Verified-endnote-styling, .Held-endnote-styling, .Reported-endnote-styling, .Rejected-endnote-styling {
+        border:dashed;
+        margin:8px;
+        padding:24px;
+        overflow:auto;
+        white-space: pre-wrap;
+    }
+
+    .Verified-ineline-styling, .Held-ineline-styling, .Reported-ineline-styling, .Rejected-ineline-styling {
+        white-space: pre-wrap;
+    }
+
+    .nodeCloseClass {
+        display:none;
+    }
+
+    .nodeOpenClass {
+        display:inline;
+    }
+    
+</style>

Tester/www/rfc2119_files/errata-color.css

@@ -0,0 +1,43 @@
+<style type="text/css">
+    .verified {color: green}
+    .supplementary-styling {
+        background-color: yellow
+    }
+    .old-text {color: red}
+
+    .Verified-headnote-styling, .Verified-endnote-styling  {
+        background-color: LightGreen;
+        color: black;
+    }
+
+    .Verified-inline-styling {
+        color: Green;
+    }
+
+    .Held-headnote-styling, .Held-endnote-styling {
+        background-color: LightBlue;
+        color: black;
+    }
+
+    .Held-inline-styling {
+        color: Blue;
+    }
+
+    .Reported-headnote-styling, .Reported-endnote-styling {
+        background-color: Beige;
+    }
+
+    .Reported-inline-styling {
+        color:GoldenRod;
+    }
+
+    .Rejected-headnote-styling, .Rejected-endnote-styling {
+        background-color: LightPink;
+        color: black;
+    }
+
+    .Rejected-inline-styling {
+        color: red;
+    }
+
+</style>

Tester/www/rfc2119_files/errata-monochrome.css

@@ -0,0 +1,34 @@
+<style type="text/css">
+    .Verified-headnote-styling, .Verified-endnote-styling  {
+        font-wieght: bold;
+    }
+    .Verified-inline-styling {
+        font-wieght: bold;
+        background-color: lightGrey;
+    }
+
+    .Held-headnote-styling, .Held-endnote-styling {
+        font-style: italic;
+    }
+
+    .Held-inline-styling {
+        font-style: italic;
+        background-color: lightGrey;
+    }
+
+    .Reported-headnote-styling, .Reported-endnote-styling {
+        background-color: Beige;
+    }
+    
+    .Reported-inline-styling {
+        color: Yellow;
+    }
+
+    .Rejected-headnote-styling, .Rejected-endnote-styling {
+        background-color: LightPink;
+    }
+
+    .Rejected-inline-styling {
+        color: red;
+    }
+</style>

Tester/www/rfc2119_files/errata-printer.css

@@ -0,0 +1,43 @@
+<style type="text/css">
+    .Verified-headnote-styling, .Verified-endnote-styling  {
+        font-wieght: bold;
+    }
+    .Verified-inline-styling {
+        font-wieght: bold;
+        background-color: lightGrey;
+    }
+
+    .Held-headnote-styling, .Held-endnote-styling {
+        font-style: italic;
+    }
+
+    .Held-inline-styling {
+        font-style: italic;
+        background-color: lightGrey;
+    }
+
+    .Reported-headnote-styling, .Reported-endnote-styling {
+        background-color: Beige;
+    }
+    
+    .Reported-inline-styling {
+        color: Yellow;
+    }
+
+    .Rejected-headnote-styling, .Rejected-endnote-styling {
+        background-color: LightPink;
+    }
+
+    .Rejected-inline-styling {
+        color: red;
+    }
+
+    .nodeCloseClass {
+        display:inline;
+    }
+
+    .nodeOpenClass {
+        display:inline;
+    }
+    
+</style>

Tester/www/rfc2119_files/errata.js

@@ -0,0 +1,4 @@
+function hideFunction(nodeId) {
+   var ul = document.getElementById(nodeId)
+   ul.className = (ul.className=="nodeOpenClass") ? "nodeCloseClass" : "nodeOpenClass"
+}

Tester/www/rfc2119_files/errata.js.téléchargement

@@ -0,0 +1,4 @@
+function hideFunction(nodeId) {
+   var ul = document.getElementById(nodeId)
+   ul.className = (ul.className=="nodeOpenClass") ? "nodeCloseClass" : "nodeOpenClass"
+}

Tester/www/test/index1.html

@@ -0,0 +1,12 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test index</title>
+	<link rel="stylesheet" href="/stylesheet/style.css">
+</head>
+<body>
+	<h1>Webserv Test Index</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>

Tester/www/test/something.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test Something</title>
+</head>
+<body>
+	<h1 style="text-align:center">Webserv Test Something</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>

Tester/www/test/submit_form.html

@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test Something</title>
+</head>
+<body>
+	<h1 style="text-align:center">Webserv in Test</h1>
+	<hr>
+	<p style="text-align:center">Time to submit something:</p>
+<!--	<form action="./srcs/cgi-bin/upload_file.php" method="post" enctype="multipart/form-data"> -->
+	<form action="/uploaded" method="post" enctype="multipart/form-data">
+<!--		<input type="hidden" name="upload_dir" value="./www/uploaded/"> -->
+		<input type="file" id="fileToUpload" name="myFile">
+		<input type="submit" value="Upload File">
+	</form>
+</body>
+</html>

Tester/www/test/test_deeper/index1.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv Test Deeper Index</title>
+</head>
+<body>
+	<h1 style="text-align:center">Webserv Test Deeper Index</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>

Tester/www/test/test_deeper/something.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test deeper Something</title>
+</head>
+<body>
+	<h1 style="text-align:center">Webserv Test Deeper Something</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>

Tester/www/test/test_deeper/super_deep/something.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+	<title>Webserv test deeper Something</title>
+</head>
+<body>
+	<h1 style="text-align:center">Webserv Test Super Deep Something</h1>
+	<hr>
+	<p style="text-align:center">(˚3˚)</p>
+</body>
+</html>

Tester/www/upload_form.html

@@ -0,0 +1,12 @@
+<!-- 
+	https://www.w3schools.com/howto/howto_html_file_upload_button.asp
+	https://www.filestack.com/fileschool/html/html-file-upload-tutorial-example/
+	https://www.rfc-editor.org/rfc/rfc9110#name-multipart-types
+	https://www.rfc-editor.org/rfc/rfc2046#section-5.1.1
+ -->
+<form action="/upload" method="post" enctype="multipart/form-data">
+	<input type="file" name="upload_file1">
+	<input type="file" name="upload_file2">
+	<input type="file" name="upload_file3">
+	<input type="submit">
+</form>

Tester/www/upload_form_single.html

@@ -0,0 +1,10 @@
+<!-- 
+	https://www.w3schools.com/howto/howto_html_file_upload_button.asp
+	https://www.filestack.com/fileschool/html/html-file-upload-tutorial-example/
+	https://www.rfc-editor.org/rfc/rfc9110#name-multipart-types
+	https://www.rfc-editor.org/rfc/rfc2046#section-5.1.1
+ -->
+<form action="/upload" method="post" enctype="multipart/form-data">
+	<input type="file" name="upload_file1">
+	<input type="submit">
+</form>

compare.txt

@@ -0,0 +1,4 @@
+telnet> Trying 127.0.0.1...
+telnet> ?Invalid command
+telnet> ?Invalid command
+telnet> telnet> telnet> 

default.config

@@ -1,20 +1,36 @@
+
 server {
 
 # this is a comment 
 
-	server_name our_server;
+	server_name server1;
 
 	listen 0.0.0.0:4040;
 
 #	client_body_limit asdfa;
-#	client_body_limit 400;
+	client_body_limit 5000;
+					 # Max == 18446744073709551615 / 1024 == 18014398509481984
 
 	index index.html; # this is another comment
+	#index mdr.html; # this is another comment
 
 	root ./www/;
 
 	error_page 404 ./www/error_pages/error_404.html;
 
+
+
+	location / {
+		allow_methods GET;
+		root ./www/;
+	}
+
+	location /srcs/cgi-bin/ {
+		root ./srcs/cgi-bin/;
+		allow_methods POST;
+		cgi_ext php;
+	}
+
 	location /list {
 		autoindex on;
 	}
@@ -24,6 +40,30 @@ server {
 		cgi_ext out php sh;
 	}
 
+	location /upload {
+		allow_methods POST;
+	#	autoindex on;
+	#	root ./www/;
+	#	index upload_form_single.html;
+
+	#	upload_dir ./www/user_files/;
+		# root doesn’t matter if used only with POST and no CGI
+	}
+
+	location /uploaded {
+		allow_methods GET;
+		autoindex on;
+	#	upload_dir ./www/user_files/;
+		root ./www/user_files;
+		# root doesn’t matter if used only with POST and no CGI
+	}
+
+	location /the_dump {
+		allow_methods GET DELETE;
+		root ./www/user_files;
+		autoindex on;
+	}
+
 	location /redirect {
 		redirect 307 https://fr.wikipedia.org/wiki/Ketchup;
 #		redirect 307 https://www.youtube.com/watch?v=rG6b8gjMEkw;
@@ -34,8 +74,12 @@ server {
 		root ./www/test/;
 	}
 
+	location /stylesheet {
+		root ./stylesheet/;
+	}
+
 	location /test/index1.html {
-		root ./www/test/;
+		root ./www/test/index1.html;
 		index index1.html subdex.html;
 	}
 
@@ -43,30 +87,33 @@ server {
 		redirect 301 https://berniesanders.com/404/;
 	}
 
-	location /stylesheet/ {
-#		root ./www/../;
-		root ./styelsheet/;
-	}
-
 	location /test/something.html {
 	#	allow_methods DELETE;
-		root ./www/test/;
+		root ./www/test/something.html;
 	}
 
 	location /test/test_deeper/ {
 #		allow_methods
-		autoindex on;
 		root ./www/test/test_deeper/;
+		index index1.html;
 	}
 
 	location /test/test_deeper/super_deep {
 		root ./www/test/test_deeper/super_deep/;
-		autoindex on;
+		index something.html;
 	}
 
 #	location /test/test_deeper/something.html {
 #		allow_methods DELETE;
 #	}
 
-
+}
+
+server {
+	server_name server2;
+
+	listen 0.0.0.0:4040;
+
+	index index.html;
+	root ./www2/;
 }

default_error_pages/404.html

@@ -1,11 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-	<title>404 Not Found</title>
-</head>
-<body>
-	<h1 style="text-align:center">404 Not Found</h1>
-	<hr>
-	<p style="text-align:center">Le Webserv/0.1</p>
-</body>
-</html>

main_test.sh

@@ -0,0 +1,118 @@
+#! /bin/bash
+
+
+# import and run all tests
+
+
+##########
+# Colors
+##########
+
+
+_GREY='\033[30m'
+_RED='\033[0;31m'
+_GREEN='\033[32m'
+_YELLOW='\033[33m'
+_BLUE='\033[34m'
+_PURPLE='\033[35m'
+_CYAN='\033[36m'
+_WHITE='\033[37m'
+_END='\033[0m'
+
+
+#test_file_names=("test_template.sh" "test_header.sh" "test_path.sh")
+#test_file_names=("test_method.sh" "test_header.sh" "test_path.sh")
+test_file_names=("test_port.sh")
+#test_file_names=("test_body.sh")
+#test_file_names=("test_valid_uri.sh")
+#test_file_names=("test_path.sh")
+
+test_files=()
+expected_result_files=()
+
+fill_test_files()
+{
+	for i in "${test_file_names[@]}"
+	do
+		test_files+=("./Tester/$i")
+	done
+}
+
+# needs to be invoked for each test_file!
+# acutally kinda useless...
+fill_expected_files()
+{
+	for i in "${local_expected_test_files[@]}"
+	do
+		expected_result_files+=("./Tester/expected_results/$i")
+	done
+}
+
+
+
+test_all()
+{
+
+	arg=${1}
+	c=0
+
+	make
+
+	rm -rf telnet.log
+	rm -rf webserv.log
+	rm -rf compare.txt
+	rm -rf expected.txt
+
+	fill_test_files
+
+	for i in "${test_files[@]}"
+	do
+		source $i
+		source ./Tester/telnet_test.sh
+
+		./webserv $config_file &>> webserv.log &
+		echo -e "${_GREEN}Running Telnet Test on '$test_name'${_END}"
+		sleep 1
+		run | telnet | tee compare.txt >> telnet.log
+#		run | telnet > compare.txt >> telnet.log
+#		run | telnet >> telnet.log
+		pkill webserv
+		echo -e "\n\n------\n" &>> webserv.log
+
+		fill_expected_files
+		expected_result_file="./Tester/expected_results/$file"
+
+		if [ "$arg" = "diff" ];
+		then
+			DIFF=$(diff -q compare.txt $expected_result_file)
+			if [ "$DIFF" == "" ];
+			then
+				echo "Good diff"
+			else
+				diff compare.txt $expected_result_file
+			fi
+		elif [ "$arg" = "create" ];
+		then 
+			if ! test -f $expected_result_file;
+			then
+				cat compare.txt > $expected_result_file
+			fi
+		elif [ "$arg" = "replace" ];
+		then 
+			cat compare.txt > $expected_result_file
+		fi
+
+		c+=1
+
+	done
+
+	if [ "$arg" = "" ];
+	then
+		cat telnet.log
+	fi
+
+
+}
+
+test_all $1
+

memo.txt

@@ -1,19 +1,45 @@
-IN 42 SUBJECT AND/OR PRIORITY :
+----Priorité élevée------------------------
 - CGI (TODO HUGO)
-- chunked request (WIP, a bit difficult)
 
-- Need to test normal body parsing
-- basic html upload page for testing request of web browser
-- upload files with config "upload_dir"
+- Need to test normal body parsing (Verif avec HUGO)
+
+- curl --resolve, for testing hostname
+	curl -v --resolve server1:4040:127.0.0.1 --resolve server2:4040:127.0.0.1 server1:4040
+- test limit de connexions sur listen()
 
-- Ecrire des tests !
 - handle redirection (Work, but weird behavior need deeper test)
-- _determine_location() review (New version to complete and test)
------------------------------
-Si ce n'est pas deja fait :
-- dans config, check erreur si port > 16bits
-(peut-être check si ip > 32bits)
------------------------------
+- Ecrire des tests !
+
+- cgi_cpp_status.cpp with POST dont show error page. Normal or not ?
+
+For non blocking CGI :
+	// We could maybe,
+	// add FD_RD_FR_CHLD to epoll,
+	// return to the main loop,
+	// read FD_RD_FR_CHLD each time epoll say its ready,
+	// then try waitpid() with WNOHANG after each read.
+	// when waitpid() tell us its finish (or maybe when epoll return EPOLLHUP)
+	// then actually parse the script_output and send it to the client.
+
+- check status in autoindex
+
+- merge changes from hugo5 to master (attention a pas casse svp :clown:) 
+
+
+----Priorité modérée------------------------
+- namespace utils ?
+- change "std::string" to reference "std::string &" in most functions
+and add "const" if apropriate.
+- peut-être check si ip > 32bits
+
+
+----Priorité faible------------------------
+- idealy, we should not clear() raw_request after a response,
+	but just the part we actually parsed for this response.
+	I think the client could send multiples request on the same connection one after the other without waiting for response.
+	So raw_request could contain more than the first request we handle.
+- chunked request (need testing)
+- client_body_limit 0 valeur special pour desactiver dans config
 - gerer le champ "Accept" du client
 - gerer les ".." dans un URL (verifier que l'on ne sort pas du dossier "root")
 - do correct handling of special character in url (/rfc2119_files/errata.js.t%C3%A9l%C3%A9chargement -> /rfc2119_files/errata.js.téléchargement)
@@ -25,10 +51,28 @@ Si ce n'est pas deja fait :
 and add "const" if apropriate.
 - Il faut vérifier le path de la requête, voir si le serveur est bien censé délivrer cette ressource et si le client y a accès, avant d'appeler le CGI.
 
+Valgrind error RESOLVED ! :
+==847174== 1,314 bytes in 1 blocks are definitely lost in loss record 1 of 1
+==847174==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
+==847174==    by 0x49AA35D: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_mutate(unsigned long, unsigned long, char const*, unsigned long) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
+==847174==    by 0x49ABB52: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_append(char const*, unsigned long) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
+==847174==    by 0x426BC5: Webserv::_read_cgi_output(cgi_pipe_rfd&) (run_loop.cpp:39) // (LUKE: its an string.append() call)
+==847174==    by 0x427962: Webserv::run() (run_loop.cpp:153)
+==847174==    by 0x4052E9: main (main.cpp:38)
 
-__________________________
---------------------------
-
-----Discord 42------------
-Un truc cool et surtout bien utile ici c'est d'utiliser un proxy entre ton navigateur et ton serveur pour vérifier ce qui est envoyé en raw. Les navigateurs peuvent avoir des comportements différents. 
-Vous avez des modules sur vos navigateur ou des logiciels externe. C'est assez rapide et gratuit.
+RESOLVED, AGAIN :D !
+--loop epoll()
+EPOLLHUP on client fd 8
+47067----loop epoll()
+EPOLLHUP on client fd 8
+47068----loop epoll()
+EPOLLHUP on client fd 8
+47069----loop epoll()
+EPOLLHUP on client fd 8
+47070----loop epoll()
+EPOLLHUP on client fd 8
+47071----loop epoll()
+EPOLLHUP on client fd 8
+47072----loop epoll()
+EPOLLHUP on client fd 8
+47073----loop epoll()

multipart_request.txt

@@ -0,0 +1,53 @@
+POST /upload HTTP/1.1
+Host: localhost:4040
+Connection: keep-alive
+Content-Length: 364
+Cache-Control: max-age=0
+sec-ch-ua: "Chromium";v="104", " Not A;Brand";v="99", "Google Chrome";v="104"
+sec-ch-ua-mobile: ?0
+sec-ch-ua-platform: "Windows"
+Upgrade-Insecure-Requests: 1
+Origin: http://localhost:4040
+Content-Type: multipart/form-data; boundary=----WebKitFormBoundaryeIV4xrEzThmNUcJf
+User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/104.0.0.0 Safari/537.36
+Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9
+Sec-Fetch-Site: same-origin
+Sec-Fetch-Mode: navigate
+Sec-Fetch-User: ?1
+Sec-Fetch-Dest: document
+Referer: http://localhost:4040/upload_form.html
+Accept-Encoding: gzip, deflate, br
+Accept-Language: fr-FR,fr;q=0.9,en-US;q=0.8,en;q=0.7
+dnt: 1
+sec-gpc: 1
+
+------WebKitFormBoundaryeIV4xrEzThmNUcJf
+Content-Disposition: form-data; name="upload_file"; filename=".gitignore"
+Content-Type: text/plain
+
+.DS_Store
+Thumbs.db
+*.o
+*.d
+*.swp
+*.out
+*.exe
+*.stackdump
+*.a
+*.so
+*.dSYM
+.vscode
+*.lnk
+*.zip
+
+builds
+
+ubuntu_tester
+ubuntu_cgi_tester
+webserv
+!**/webserv/
+*.log
+
+large.jpg
+
+------WebKitFormBoundaryeIV4xrEzThmNUcJf--

srcs/Client.cpp

@@ -10,9 +10,10 @@ Client::Client()
 	header_complete(false),
 	body_complete(false),
 	request_complete(false),
-	read_body_size(0),
 	assigned_server(NULL),
 	assigned_location(NULL),
+	cgi_pipe_rfd(0),
+	cgi_pid(0),
 	_fd(0),
 	_port(""),
 	_ip(""),
@@ -26,9 +27,10 @@ Client::Client(int afd, listen_socket *lsocket, std::string aport, std::string a
 	header_complete(false),
 	body_complete(false),
 	request_complete(false),
-	read_body_size(0),
 	assigned_server(NULL),
 	assigned_location(NULL),
+	cgi_pipe_rfd(0),
+	cgi_pid(0),
 	_fd(afd),
 	_port(aport),
 	_ip(aip),
@@ -45,7 +47,6 @@ Client::~Client() {
 Client::Client( Client const & src )
 	: status			( src.status ),
 	header_complete		( src.header_complete ),
-	read_body_size		( src.read_body_size ),
 	assigned_server		( src.assigned_server ),
 	assigned_location	( src.assigned_location ),
 	_fd					( src._fd ),
@@ -75,10 +76,12 @@ Client &	Client::operator=( Client const & rhs )
  * PUBLIC MEMBER FUNCTIONS
  *********************************************/
 
-// http headers :
-// https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers
-// https://www.ibm.com/docs/en/cics-ts/5.3?topic=protocol-http-requests
-// https://www.tutorialspoint.com/http/http_requests.htm
+/* 	HTTP Headers :
+	https://www.iana.org/assignments/http-fields/http-fields.xhtml
+	https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers
+	https://www.ibm.com/docs/en/cics-ts/5.3?topic=protocol-http-requests
+	https://www.tutorialspoint.com/http/http_requests.htm
+*/
 void	Client::parse_request_headers(std::vector<ServerConfig> &servers)
 {
 	if (raw_request.find(CRLF CRLF) == NPOS)
@@ -96,12 +99,12 @@ void	Client::parse_request_headers(std::vector<ServerConfig> &servers)
 
 	if (status)
 		return;
-	assigned_server = ::_determine_process_server(this, servers);
-	assigned_location = ::_determine_location(*assigned_server, _request.abs_path);
+	assigned_server = _determine_process_server(this, servers);
+	assigned_location = _determine_location(*assigned_server, _request.abs_path);
 	_check_request_errors();
 	if (status)
 		return;
-	_parse_port_hostname(this->get_rq_headers("Host")); // use getter for headers because it works case insensitive
+	_parse_port_hostname(get_rq_headers("Host")); // use getter for headers because it works case insensitive
 
 // DEBUG
 // std::cerr << get_rq_method_str() << " " << get_rq_target() << " " << get_rq_version() << "\n";
@@ -121,14 +124,40 @@ void	Client::parse_request_body()
 	{
 		std::cerr << "parse_request_body() bad call, header incomplete\n";
 		// QUESTION from hugo : don't we change the status here ?
+		// RESPONSE from luke : C'est vrai. Peut-être mettre un 500, c'etait plus du debug à la base.
+		// C'est seulement si on appelle la fonction au mauvais endroit, avant d'avoir un header complet, que ça arrive.
 		return;
 	}
-	pos += CRLF_SIZE*2;
 
-	// Chunked decoding WIP. Dont work.
 	if (!get_rq_headers("Transfer-Encoding").empty()
 	&& get_rq_headers("Transfer-Encoding") == "chunked")
 	{
+		// Chunked decoding WIP. How to test this ? dont know how to send chunks with telnet.
+		_parse_chunked_body(pos + CRLF_SIZE*2);
+	}
+	else if (raw_request.size() - pos >= std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10))
+	{
+	std::cerr << "Content-Length = " << std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10) << "\n";
+		if (get_rq_headers("Content-Type").find("multipart/form-data") != NPOS)
+			_parse_multipart_body(pos);
+		else
+			_request.body = raw_request.substr(pos + CRLF_SIZE*2);
+		body_complete = true;
+	}
+
+	std::cerr << "Content-Length = " << std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10) << "\n";
+	std::cerr << "raw_request.size() - pos = " << raw_request.size() - pos << "\n";
+	_request.body = raw_request.substr(pos);
+	std::cerr << "_request.body.size() = " << _request.body.size() << "\n";
+
+///////////////
+// Body checks
+	if (_request.body.size() > assigned_server->client_body_limit)
+		status = 413; // HTTP Client Errors
+}
+
+void	Client::_parse_chunked_body(size_t pos)
+{
 		size_t	chunk_size = 1;
 		size_t	chunk_field_end = 0;
 		char	*endptr = NULL;
@@ -159,8 +188,11 @@ void	Client::parse_request_body()
 			/* endptr_copy = endptr; */
 			(void)endptr_copy;
 			chunk_size = std::strtoul(&_request.body[pos], &endptr, 16);
-/* 			if (chunk_size == LONG_MAX && errno == ERANGE)
-				status = 413; */
+			if (errno == ERANGE)
+			{
+				status = 413;
+				return ;
+			}
 		/* 	if (endptr == endptr_copy)
 			{
 				std::cerr << "parse_request_body(), no conversion possible\n";
@@ -183,41 +215,14 @@ void	Client::parse_request_body()
 
 		_request.headers.erase("Transfer-Encoding");
 		body_complete = true;
-	}
-	else
-	{
-		std::cerr << "Content-Length = " << std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10) << "\n";
-		std::cerr << "raw_request.size() - pos = " << raw_request.size() - pos << "\n";
-		_request.body = raw_request.substr(pos);
-		std::cerr << "_request.body.size() = " << _request.body.size() << "\n";
-		
-		if (raw_request.size() - pos >= std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10))
-		{
-			_request.body = raw_request.substr(pos);
-			body_complete = true;
-		}
-
-		/* Should be equivalent */
-			// _request.body = raw_request.substr(pos);
-			// if (_request.body.size() >= std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10))
-			// 	body_complete = true;
-	}
-
-///////////////
-// Body checks
-	if (_request.body.size() > assigned_server->client_body_limit)
-		status = 413; // HTTP Client Errors
 }
 
+
+
 void 	Client::fill_script_path(std::string &path, size_t pos)
 {
 	std::string	tmp;
 
-	if (path[0] == '.')
-	{
-		path.erase(0, 1);
-		pos--;
-	}
 	_request.script.path = path.substr(0, pos);
 	_request.script.info = path.substr(pos);
 }
@@ -225,15 +230,17 @@ void 	Client::fill_script_path(std::string &path, size_t pos)
 void	Client::clear()
 {
 	clear_request();
-	header_complete = false;
-	body_complete = false;
-	request_complete = false;
-	read_body_size = 0;
-	assigned_server = NULL;
-	assigned_location = NULL;	
 	raw_request.clear();
 	response.clear();
 	status = 0;
+	header_complete = false;
+	body_complete = false;
+	request_complete = false;
+	assigned_server = NULL;
+	assigned_location = NULL;
+	cgi_pipe_rfd = 0;
+	cgi_pid = 0;
+	cgi_output.clear();
 }
 
 void	Client::clear_request()
@@ -244,6 +251,7 @@ void	Client::clear_request()
 	_request.version.clear();
 	_request.headers.clear();
 	_request.body.clear();
+	_request.multi_bodys.clear();
 	_request.abs_path.clear();
 	_request.query.clear();
 	_request.port.clear();
@@ -260,8 +268,7 @@ void	Client::clear_script()
 void	Client::print_client(std::string message)
 {
 	std::map<std::string, std::string>::iterator	it;
-
-	std::cout << "\n=== DEBUG PRINT CLIENT ===\n";
+std::cout << "\n=== DEBUG PRINT CLIENT ===\n";
 	std::cout << message << ":\n----------\n\n" << "raw_request:\n__\n";
 	::print_special(raw_request);
 	std::cout << "\n__\n"
@@ -334,6 +341,7 @@ void	Client::_parse_request_line()
 	{
 		std::cerr << "err _parse_first_line(): wrong number of elements (" << line.size() << " instead of 3)\n";
 		status = 400; // "bad request"
+		// if the header is fucked up, then this will be triggered, but for some reason in test_body.sh case another response 200 OK is sent first?
 	}
 	else
 	{
@@ -354,6 +362,8 @@ void	Client::_parse_request_target( std::string target )
 	else
 		_request.query = "";
 	_request.abs_path = target.substr(0, pos);
+	if (_request.abs_path[_request.abs_path.size() - 1] == '/')
+		_request.abs_path.erase(_request.abs_path.size() - 1);
 }
 
 void	Client::_parse_request_fields()
@@ -366,7 +376,7 @@ void	Client::_parse_request_fields()
 	// delete first line
 	pos = headers.find(CRLF);
 	if (pos != NPOS)
-		headers.erase(0, pos + std::string(CRLF).size());
+		headers.erase(0, pos + CRLF_SIZE);
 	// delete body part
 	pos = headers.find(CRLF CRLF);
 	if (pos != NPOS)
@@ -384,6 +394,7 @@ void	Client::_parse_request_fields()
 	::str_map_key_tolower(_request.headers);
 }
 
+// TODO : I think its now useless. Probably to delete.
 void	Client::_parse_port_hostname(std::string host)
 {
 	size_t	pos;
@@ -405,6 +416,9 @@ void	Client::_parse_port_hostname(std::string host)
 
 void	Client::_check_request_errors()
 {
+	/* Debug */ std::cerr << "Content-Length=" << get_rq_headers("Content-Length") << "\n";
+	/* Debug */ std::cerr << "strtoul=" << std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10) << "\n";
+	/* Debug */ std::cerr << "client_body_limit=" <<  assigned_server->client_body_limit << "\n";
 ///////////////////////
 // Request line checks
 	if (_request.method == UNKNOWN)
@@ -429,13 +443,13 @@ void	Client::_check_request_errors()
 
 //////////////////
 // Headers checks
-	else if (!this->get_rq_headers("Content-Length").empty()
-	&& std::strtoul(this->get_rq_headers("Content-Length").c_str(), NULL, 10) > assigned_server->client_body_limit)
+	else if (!get_rq_headers("Content-Length").empty()
+	&& std::strtoul(get_rq_headers("Content-Length").c_str(), NULL, 10) > assigned_server->client_body_limit)
 		status = 413;
-	else if (!this->get_rq_headers("Transfer-Encoding").empty()
-	&& this->get_rq_headers("Transfer-Encoding") != "chunked" )
+	else if (!get_rq_headers("Transfer-Encoding").empty()
+	&& get_rq_headers("Transfer-Encoding") != "chunked" )
 		status = 501;
-	else if (!this->get_rq_headers("Content-Encoding").empty())
+	else if (!get_rq_headers("Content-Encoding").empty())
 	{
 		status = 415;
 		response.append("Accept-Encoding:"); // empty, no encoding accepted
@@ -445,6 +459,99 @@ void	Client::_check_request_errors()
 	return;
 }
 
+ServerConfig	*Client::_determine_process_server(Client *client, std::vector<ServerConfig> &servers)
+{
+/*
+	Behavior like this :
+	http://nginx.org/en/docs/http/request_processing.html
+*/
+
+	std::string	server_name = client->get_rq_headers("Host");
+	// /* Debug */ std::cerr << "server_name = " << server_name << "\n";
+	size_t	pos = server_name.rfind(':');
+	if (pos != NPOS)
+		server_name.erase(pos);
+	// /* Debug */ std::cerr << "server_name = " << server_name << "\n";
+
+	std::vector<ServerConfig>::iterator	it = servers.begin();
+	std::vector<ServerConfig>::iterator	default_server = servers.end();
+
+	while (it != servers.end())
+	{
+		if (it->host == client->get_cl_lsocket()->host
+		&&	it->port == client->get_cl_lsocket()->port)
+		{
+			if ( std::find(it->server_name.begin(), it->server_name.end(), server_name) != it->server_name.end() )
+				break;
+			else if (default_server == servers.end())
+				default_server = it;
+		}
+		++it;
+	}
+	if (it != servers.end())
+		return (&(*it));
+	else
+		return (&(*default_server));
+}
+
+// const?
+const LocationConfig	*Client::_determine_location(const ServerConfig &server, const std::string &path)
+{
+/* RULES ***
+
+If a path coresponds exactly to a location, use that one
+if no path coresponds then use the most correct one
+	most correct means the most precise branch that is still above
+	the point we are aiming for
+
+New Rule for location paths, they never end in /
+Sooo
+If we get a url that ends in / ignore the last /
+
+*/
+
+// TODO: technically i think this is useless...
+	std::string	uri = path;
+	if (uri[uri.size() - 1] == '/' && uri.size() != 1)
+		uri.erase(uri.size() - 1);
+
+
+	for (std::vector<LocationConfig>::const_iterator it = server.locations.begin(); it != server.locations.end(); it++)
+	{
+//		std::cout << it->path << "  --  ";
+		if (it->path.size() > uri.size())
+			continue ;
+
+		if (uri.compare(0, it->path.size(), it->path) == 0)
+		{
+			if (it->path.size() == uri.size())
+				return (&(*it));
+			else if (uri[it->path.size()] == '/')
+				return (&(*it));
+	// this works cuz only ever looking for a / burried in a longer path
+		}
+	}
+	return (&(server.locations.back()));
+
+
+//	/test/mdr
+//	/test/mdr/
+//	/test/mdrBST
+
+/////// More stuff to check this still works with
+
+// /test/test_
+// /test/test_/
+// /test/test_deeper
+// /test/test_deeper/
+// /test/test_deepei
+// /test/test_deepei/
+// /test/test_deeperi
+// /test/test_deeper/super_deep/
+// /test/aaaaaaaaaaa/super_deep/
+
+}
+
 /*********************************************
  * OVERLOAD
  *********************************************/

srcs/Client.hpp

@@ -19,6 +19,12 @@ struct Script
 	std::string							info;
 };
 
+struct MultipartBody
+{
+	std::map<std::string, std::string>	headers;
+	std::string							body;
+};
+
 struct Request
 {
 	http_method							method;
@@ -28,6 +34,7 @@ struct Request
 	std::string							version;
 	std::map<std::string, std::string>	headers;
 	std::string							body;
+	std::vector<MultipartBody>			multi_bodys;
 	std::string							port;
 	std::string							hostname;
 	struct Script						script;
@@ -48,9 +55,12 @@ class Client
 		bool					header_complete;
 		bool					body_complete;
 		bool					request_complete;
-		size_t					read_body_size; // unused for now
+		// size_t					read_body_size; // unused for now
 		ServerConfig			*assigned_server; // cant be const cause of error_pages.operator[]
 		const LocationConfig	*assigned_location;
+		int						cgi_pipe_rfd;
+		pid_t					cgi_pid;
+		std::string				cgi_output;
 
 		// getters
 		int						get_cl_fd() const;
@@ -72,6 +82,9 @@ class Client
 		std::string		get_rq_script_info() const;
 		std::string		get_rq_headers(const std::string & key) const;
 
+		const std::vector<MultipartBody>	&get_rq_multi_bodys() const;
+		const std::string	get_rq_multi_bodys_headers(const std::string & key, std::vector<MultipartBody>::const_iterator body_it) const;
+
 		void	parse_request_headers(std::vector<ServerConfig> &servers);
 		void	parse_request_body();
 		void	clear();
@@ -92,17 +105,18 @@ class Client
 		void	_parse_request_fields();
 		void	_parse_request_target( std::string target );
 		void	_parse_port_hostname(std::string host);
-
+		void	_parse_chunked_body(size_t pos);
+		void	_parse_multipart_body(size_t pos);
 		void	_check_request_errors();
+
+		ServerConfig*
+			_determine_process_server(Client *client, std::vector<ServerConfig> &servers);
+		const LocationConfig*
+			_determine_location(const ServerConfig &server, const std::string &path);
 };
 
 bool	operator==(const Client& lhs, const Client& rhs);
 bool	operator==(const Client& lhs, int fd);
 bool	operator==(int fd, const Client& rhs);
 
-// Temporary Global Scope. Probably move to Client in the future.
-ServerConfig	*_determine_process_server(Client *client, std::vector<ServerConfig> &servers);
-const LocationConfig	*_determine_location(const ServerConfig &server, const std::string &path);
-
 #endif
-

srcs/Client_multipart_body.cpp

@@ -0,0 +1,103 @@
+
+#include "Client.hpp"
+
+const std::vector<MultipartBody>	&Client::get_rq_multi_bodys() const { return _request.multi_bodys; }
+const std::string	Client::get_rq_multi_bodys_headers(const std::string & key, std::vector<MultipartBody>::const_iterator body_it) const
+{
+	std::map<std::string, std::string>::const_iterator	it;
+
+	it = body_it->headers.find(::str_tolower(key));
+	if (it == body_it->headers.end())
+		return ""; // IF return reference compiler "warning: returning reference to local temporary"
+	return it->second;
+}
+
+void	Client::_parse_multipart_body(size_t pos)
+{
+/*
+**	Parsing roughly like described in :
+**	https://www.rfc-editor.org/rfc/rfc2046#section-5.1.1
+*/
+	MultipartBody	new_body;
+	std::string		boundary;
+	size_t			start_pos;
+	size_t			end_pos;
+	std::string		tmp;
+	size_t			tmp_pos;
+	size_t			ret;
+	
+// Get boundary
+	boundary = get_rq_headers("Content-Type");
+	start_pos = boundary.find("boundary=");
+	if (start_pos == NPOS)
+	{
+		status = 400; std::cerr << "_parse_multipart_body() error 1\n";
+		return;
+	}
+	start_pos += sizeof("boundary=")-1;
+	boundary = boundary.substr(start_pos);
+	std::cerr << "boundary =|" << boundary << "|\n";
+
+
+// Search boundary
+	start_pos = raw_request.find("--" + boundary, pos);
+	if (start_pos == NPOS || start_pos + sizeof("--")-1 + boundary.size() > raw_request.size())
+	{
+		status = 400; std::cerr << "_parse_multipart_body() error 2\n";
+		return;
+	}
+	start_pos += sizeof("--")-1 + boundary.size() + CRLF_SIZE;
+
+	while (1) // TODO : test loop for multi body
+	{
+		end_pos = raw_request.find("--" + boundary, start_pos);
+		if (end_pos == NPOS)
+		{
+			status = 400; std::cerr << "_parse_multipart_body() error 3\n";
+			return;
+		}
+		/* // Maye useful for multi body (remove "start_pos - CRLF_SIZE" if used)
+			end_pos = raw_request.rfind(CRLF, end_pos); if (end_pos == NPOS) {status = 400; return; } */
+
+		new_body.body = raw_request.substr(start_pos, end_pos - start_pos - CRLF_SIZE);
+
+// Split headers from body
+		tmp_pos = new_body.body.find(CRLF CRLF);
+		if (tmp_pos != NPOS)
+		{
+			ret = ::parse_http_headers(new_body.body.substr(0, tmp_pos), new_body.headers);
+			::str_map_key_tolower(new_body.headers);
+			if (ret)
+			{
+				status = 400; std::cerr << "_parse_multipart_body() error 4\n";
+				return;
+			}
+			tmp_pos += CRLF_SIZE*2;
+			new_body.body.erase(0, tmp_pos);
+			// ::print_map(new_body.headers);
+		}
+		else
+		{ // No headers case
+			tmp_pos = new_body.body.find(CRLF);
+			if (tmp_pos != 0)
+			{
+				status = 400; std::cerr << "_parse_multipart_body() error 5\n";
+				return;
+			}
+		}
+
+		_request.multi_bodys.push_back(new_body);
+
+//	Move start for next loop
+		start_pos = end_pos + sizeof("--")-1 + boundary.size();
+		if ( start_pos + 2 + CRLF_SIZE == raw_request.size()
+		&& raw_request[start_pos] == '-'
+		&& raw_request[start_pos+1] == '-')
+			break;
+
+/* 		::print_special(raw_request);
+		std::cerr << "start_pos = " << start_pos << "\n";
+		std::cerr << "raw_request.size() = " << raw_request.size() << "\n";
+		std::cerr << raw_request.substr(start_pos); */
+	}
+}

srcs/cgi-bin/Makefile

@@ -0,0 +1,95 @@
+
+# - - - - - - #
+#             #
+#   COLORS    #
+#             #
+# - - - - - - #
+
+GRAY     = "\e[0;30m"
+RED      = "\e[0;31m"
+GREEN    = "\e[0;32m"
+YELLOW   = "\e[0;33m"
+BLUE     = "\e[0;34m"
+PURPLE   = "\e[0;35m"
+CYAN     = "\e[0;36m"
+WHITE    = "\e[0;37m"
+
+B_GRAY   = "\e[1;30m"
+B_RED    = "\e[1;31m"
+B_GREEN  = "\e[1;32m"
+B_YELLOW = "\e[1;33m"
+B_BLUE   = "\e[1;34m"
+B_PURPLE = "\e[1;35m"
+B_CYAN   = "\e[1;36m"
+B_WHITE  = "\e[1;37m"
+
+RESET    = "\e[0m"
+
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - #
+#               . name = value \          . += append to a variable   #
+#   VARIABLES   .        value            . != set result of command  #
+#               . name is case sensitive  . ?= set if not already set #
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - #
+
+NAME		 =	$(SRCS_X:.cpp=.out)
+
+CXX			 =	c++
+CXXFLAGS	 =	-Wall -Wextra #-Werror
+CXXFLAGS	+=	$(HEADERS_D:%=-I%)
+CXXFLAGS	+=	-std=c++98
+CXXFLAGS	+=	-g
+
+VPATH		 =	$(SRCS_D)
+HEADERS_D	 =	.
+SRCS_D		 =	.
+
+SRCS		 =	cgi_utils.cpp
+SRCS_X		 =	\
+				cgi_cpp.cpp \
+				cgi_cpp_bad_headers.cpp \
+				cgi_cpp_empty.cpp \
+				cgi_cpp_empty_lines.cpp \
+				cgi_cpp_len.cpp \
+				cgi_cpp_len_big.cpp \
+				cgi_cpp_len_small.cpp \
+				cgi_cpp_no_body.cpp \
+				cgi_cpp_no_headers.cpp \
+				cgi_cpp_only_crlf.cpp \
+				cgi_cpp_sleep.cpp \
+				cgi_cpp_status.cpp \
+				cgi_cpp_download.cpp \
+
+OBJS_D		 =	builds
+OBJS		 =	$(SRCS:%.cpp=$(OBJS_D)/%.o)
+OBJS_X		 =	$(SRCS_X:%.cpp=$(OBJS_D)/%.o)
+
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - #
+#             .   target: prerequisites   .   $@ : target             #
+#    RULES    .           recipe          .   $< : 1st prerequisite   #
+#             .           recipe          .   $^ : all prerequisites  #
+# - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - #
+
+all: $(NAME)
+
+$(OBJS_D)/%.o: %.cpp | $(OBJS_D)
+	@echo $(B_GREEN)"compilation :" $@ $(RESET)
+	$(CXX) $(CXXFLAGS) -c $< -o $@
+
+$(OBJS_D):
+	mkdir $@
+
+$(NAME): $(OBJS) $(OBJS_X)
+$(NAME):
+	@echo $(B_YELLOW)"linkage :" $@ $(RESET)
+	$(CXX) $(OBJS) $(@:%.out=$(OBJS_D)/%.o) -o $@
+
+clean:
+	rm -rf $(OBJS_D)
+
+fclean: clean
+	rm -f $(NAME)
+
+re: fclean all
+
+.PHONY : all clean fclean re
+

srcs/cgi-bin/cgi

@@ -0,0 +1,4 @@
+#! /bin/bash
+echo "status: 100\r\n"
+echo "\r\n\r\n"
+echo "hiii"

srcs/cgi-bin/cgi_cpp.cpp

@@ -1,114 +1,25 @@
-# include <iostream>
-# include <string>
-# include <sstream>
-# include <vector>
-# include <stdlib.h> // getenv
 
-# define CR "\r"
-# define LF "\n"
-# define CRLF CR LF
-# define NPOS std::string::npos
+# include "cgi_utils.hpp"
+# include <unistd.h>
+# include <cstdio>
 
-std::string	trim(std::string str, char del)
+int main (int ac, char **av, char ** env)
 {
-	size_t	pos;
+	std::string	http_header;
+	std::string	http_body;
+	std::string	rq_body;
 
-	// delete leadings del
-	pos = str.find_first_not_of(del);
-	if (pos == NPOS)
-		pos = str.size();
-	str = str.substr(pos);
+	std::cin >> rq_body;
 
-	// delete trailing del
-	pos = str.find_last_not_of(del);
-	if (pos != NPOS)
-		str = str.substr(0, pos + 1);
+	(void)ac;
+	(void)av;
 
-	return str;
-}
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
 
-std::vector<std::string>
-	split(const std::string & input, std::string delim, char ctrim = '\0')
-{
-	std::vector<std::string>	split_str;
-	std::string					tmp;
-	size_t						start = 0;
-	size_t						end = 0;
-	size_t						len = 0;
+	fill_body_basic(env, http_body, rq_body);
 
-	while (end != NPOS)
-	{
-		end = input.find(delim, start);
-		len = end - start;
-		if (end == NPOS)
-			len = end;
-		tmp = input.substr(start, len);
-		if (ctrim != '\0')
-			tmp = trim(tmp, ctrim);
-		if (tmp.size() != 0)
-			split_str.push_back( tmp );
-		start = end + delim.size();
-	}
-	return split_str;
-}
+	std::cout << http_header << CRLF << http_body;
 
-int main (int ac, char **av, char **en)
-{
-	std::vector<std::string>                    split_str;
-	std::vector<std::string>                    sub_split_str;
-	std::vector<std::string>::const_iterator    it;
-	char *                                      tmp;
-	std::string                                 input;
-	std::string                                 http_header;
-	std::string                                 http_body;
-	std::ostringstream                          strs;
-	size_t                                      pos;
-
-	std::cin >> input;
-
-	http_header = "Content-Type: text/html; charset=UTF-8" CRLF;
-	http_header += "Content-Length: ";
-
-	http_body  = "\
-				  <!DOCTYPE html>\
-				  <html>\
-				  <head>\
-				  <title>CGI</title>\
-				  </head>\
-				  <body>\
-				  <h2>cgi</h2>\
-				  ";
-
-	http_body += "<h3>";
-	tmp = getenv("REQUEST_METHOD");
-	if (tmp != NULL)
-		http_body += tmp;
-	else
-		http_body = "method not foud";
-	http_body += "</h3>";
-
-	split_str = split(input, "&");
-	for (it = split_str.begin(); it != split_str.end(); ++it)
-	{
-		sub_split_str = split(*it, "=");
-		http_body += "<h3>";
-		http_body += sub_split_str[0];
-		http_body += "</h3>";
-		http_body += "<p>";
-		http_body += sub_split_str[1];
-		http_body += "</p>";
-	}
-
-	http_body += "\
-				  </body>\
-				  </html>\
-				  ";
-
-	strs << http_body.size();
-	http_header  += strs.str();
-	http_header  += CRLF CRLF;
-
-	std::cout << http_header << CRLF CRLF << http_body;
 	return 0;
 }
 

srcs/cgi-bin/cgi_cpp_bad_headers.cpp

@@ -0,0 +1,24 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string	http_header;
+	std::string	http_body;
+	std::string	http_status;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Bad-Headers: wrong";
+
+	fill_body_basic(env, http_body, rq_body);
+
+	std::cout << http_header << CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_content_length.cpp

@@ -1,133 +0,0 @@
-# include <iostream>
-# include <string>
-# include <sstream>
-# include <vector>
-# include <stdlib.h> // getenv
-
-# define CR "\r"
-# define LF "\n"
-# define CRLF CR LF
-# define NPOS std::string::npos
-
-std::string	trim(std::string str, char del)
-{
-	size_t	pos;
-
-	// delete leadings del
-	pos = str.find_first_not_of(del);
-	if (pos == NPOS)
-		pos = str.size();
-	str = str.substr(pos);
-
-	// delete trailing del
-	pos = str.find_last_not_of(del);
-	if (pos != NPOS)
-		str = str.substr(0, pos + 1);
-
-	return str;
-}
-
-std::vector<std::string>
-	split(const std::string & input, std::string delim, char ctrim = '\0')
-{
-	std::vector<std::string>	split_str;
-	std::string					tmp;
-	size_t						start = 0;
-	size_t						end = 0;
-	size_t						len = 0;
-
-	while (end != NPOS)
-	{
-		end = input.find(delim, start);
-		len = end - start;
-		if (end == NPOS)
-			len = end;
-		tmp = input.substr(start, len);
-		if (ctrim != '\0')
-			tmp = trim(tmp, ctrim);
-		if (tmp.size() != 0)
-			split_str.push_back( tmp );
-		start = end + delim.size();
-	}
-	return split_str;
-}
-
-int main (int ac, char **av, char **en) {
-	std::vector<std::string>					split_str;
-	std::vector<std::string>					sub_split_str;
-	std::vector<std::string>::const_iterator	it;
-	char *										tmp;
-	std::string									output;
-	std::ostringstream							strs;
-	size_t										pos;
-
-	std::cout << "Content-Type: text/html; charset=UTF-8" << CRLF CRLF;
-
-	std::cout
-		<< "<!DOCTYPE html>"
-		<< "<html>"
-		<< "<head>"
-		<< "	<title>CGI</title>"
-		<< "</head>"
-		<< "<body>"
-		<< "	<h2>cgi</h2>"
-		<< "	<h3>";
-
-	tmp = getenv("REQUEST_METHOD");
-	if (tmp != NULL)
-		output = tmp;
-	else
-		output = "method not foud";
-
-	std::cout
-		<< output
-		<< "	</h3>"
-		<< "	<h3>http-request-body-message content :</h3>";
-
-
-	std::cin >> output;
-	split_str = split(output, "&");
-	output.clear();
-	for (it = split_str.begin(); it != split_str.end(); ++it)
-	{
-		sub_split_str = split(*it, "=");
-
-		std::cout
-			<< "<p>"
-			<< sub_split_str[0]
-			<< " : "
-			<< sub_split_str[1]
-			<< "</p>";
-	}
-
-	tmp = getenv("QUERY_STRING");
-	if (tmp == NULL)
-		std::cout << "query not foud";
-
-	std::cout
-		<< "	<h3>http-uri-query content :</h3>";
-
-	output = tmp;
-	split_str = split(output, "&");
-	output.clear();
-	for (it = split_str.begin(); it != split_str.end(); ++it)
-	{
-		sub_split_str = split(*it, "=");
-
-		std::cout
-			<< "<h3>"
-			<< sub_split_str[0]
-			<< "</h3>"
-			<< "<p>"
-			<< sub_split_str[1]
-			<< "</p>";
-	}
-
-
-	std::cout
-		<< "</body>"
-		<< "</html>";
-
-	return 0;
-}
-

srcs/cgi-bin/cgi_cpp_download.cpp

@@ -0,0 +1,55 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string			http_header;
+	std::string			http_body;
+	std::string			rq_body;
+	std::string 		form_infos;
+	std::string 		path;
+	std::ifstream		ifd;
+	std::stringstream	buf;
+	size_t		 		status;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+	(void)env;
+
+	http_header  = "Content-Type: image/jpeg" CRLF;
+
+	form_infos = get_form_infos(rq_body);
+	path = get_value("file", rq_body);
+	path = "./www/" + path;
+
+	status = ::eval_file_read(path);
+	if (status)
+	{
+		std::cout << "Status: " << status << CRLF CRLF;
+		return 0;
+	}
+
+	ifd.open(path.c_str());
+	if (!ifd)
+	{
+		std::cout << "Status: " << 500 << CRLF CRLF;
+		return 0;
+	}
+	else
+	{
+		buf << ifd.rdbuf();
+		if (!ifd || !buf)
+		{
+			std::cout << "Status: " << 500 << CRLF CRLF;
+			return 0;
+		}
+	}
+
+	std::cout << http_header << CRLF << buf.str();
+
+	return 0;
+}
+
+

srcs/cgi-bin/cgi_cpp_empty.cpp

@@ -0,0 +1,12 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	(void)ac;
+	(void)av;
+	(void)env;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_empty_lines.cpp

@@ -0,0 +1,23 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string                                 http_header;
+	std::string                                 http_body;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+	std::cout << http_header << CRLF CRLF CRLF CRLF CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_len.cpp

@@ -0,0 +1,25 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string                                 http_header;
+	std::string                                 http_body;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+	http_header += "Content-Length: " + itos(http_body.size()) + CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+
+	std::cout << http_header << CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_len_big.cpp

@@ -0,0 +1,25 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string	http_header;
+	std::string	http_body;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+	http_header += "Content-Length: " + itos(http_body.size() + 100) + CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+
+	std::cout << http_header << CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_len_small.cpp

@@ -0,0 +1,24 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string	http_header;
+	std::string	http_body;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+	http_header += "Content-Length: " + itos(http_body.size() - 100) + CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+	std::cout << http_header << CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_no_body.cpp

@@ -0,0 +1,22 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string	http_header;
+	std::string	http_status;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+	(void)env;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+
+	std::cout << http_header << CRLF;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_no_headers.cpp

@@ -0,0 +1,24 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string	http_header;
+	std::string	http_body;
+	std::string	http_status;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+	std::cout << CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_only_crlf.cpp

@@ -0,0 +1,15 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	(void)ac;
+	(void)av;
+	(void)env;
+
+	std::cout << CRLF CRLF;
+
+	return 0;
+}
+
+

srcs/cgi-bin/cgi_cpp_sleep.cpp

@@ -0,0 +1,29 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string			http_header;
+	std::string			http_body;
+	std::string			rq_body;
+	size_t				time;
+	std::stringstream	ss;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+	ss << get_value("sleep", rq_body);
+	ss >> time;
+	sleep(time);
+
+	std::cout << http_header << CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_cpp_status.cpp

@@ -0,0 +1,27 @@
+
+# include "cgi_utils.hpp"
+
+int main (int ac, char **av, char ** env)
+{
+	std::string	http_header;
+	std::string	http_body;
+	std::string	http_status;
+	std::string	rq_body;
+
+	std::cin >> rq_body;
+
+	(void)ac;
+	(void)av;
+
+	http_header  = "Content-Type: text/html; charset=UTF-8" CRLF;
+
+	fill_body_basic(env, http_body, rq_body);
+
+	http_status = get_value("Status", rq_body);
+	http_header += "Status: " + http_status +  CRLF;
+
+	std::cout << http_header << CRLF CRLF << http_body;
+
+	return 0;
+}
+

srcs/cgi-bin/cgi_style.css

@@ -0,0 +1,3 @@
+h1, h2, h3, p {
+	display: inline;
+}

srcs/cgi-bin/cgi_utils.cpp

@@ -0,0 +1,199 @@
+#include "cgi_utils.hpp"
+
+std::string	str_tolower(std::string str)
+{
+	std::transform(str.begin(), str.end(), str.begin(), ::tolower);
+	return str;
+}
+
+std::string	trim(std::string str, char del)
+{
+	size_t	pos;
+
+	// delete leadings del
+	pos = str.find_first_not_of(del);
+	if (pos == NPOS)
+		pos = str.size();
+	str = str.substr(pos);
+
+	// delete trailing del
+	pos = str.find_last_not_of(del);
+	if (pos != NPOS)
+		str = str.substr(0, pos + 1);
+
+	return str;
+}
+
+std::vector<std::string>
+	split(const std::string & input, std::string delim, char ctrim)
+{
+	std::vector<std::string>	split_str;
+	std::string					tmp;
+	size_t						start = 0;
+	size_t						end = 0;
+	size_t						len = 0;
+
+	while (end != NPOS)
+	{
+		end = input.find(delim, start);
+		len = end - start;
+		if (end == NPOS)
+			len = end;
+		tmp = input.substr(start, len);
+		if (ctrim != '\0')
+			tmp = trim(tmp, ctrim);
+		if (tmp.size() != 0)
+			split_str.push_back( tmp );
+		start = end + delim.size();
+	}
+	return split_str;
+}
+
+std::string	itos(int n)
+{
+	std::stringstream	strs;
+
+	strs << n;
+	return ( strs.str() );
+}
+
+std::string	parse_env(const std::string & env)
+{
+	std::string ret = "";
+	char *		ret_env;
+
+	ret_env = getenv(env.c_str());
+	if (ret_env != NULL)
+		ret = ret_env;
+
+	return ret;
+}
+
+std::string print_env(char **env, std::string tag)
+{
+	std::string	ret = "";
+
+	for (int i = 0; env[i] != NULL; ++i)
+	{
+		ret += "<" + tag + ">";
+		ret += env[i];
+		ret += "</" + tag + "><br>";
+	}
+	return ret;
+}
+
+std::string
+	print_form(std::string form, std::string tag_key, std::string tag_val)
+{
+	std::vector<std::string>                    split_str;
+	std::vector<std::string>                    sub_split_str;
+	std::vector<std::string>::const_iterator    it;
+	std::string									ret = "";
+	std::string									key;
+
+	split_str = split(form, "&");
+	for (it = split_str.begin(); it != split_str.end(); ++it)
+	{
+		sub_split_str = split(*it, "=");
+		key = sub_split_str[0];
+		if (key == "fname")
+			key = "first name";
+		else if (key == "lname")
+			key = "last name";
+		ret += "<br><" + tag_key + ">" + key + ": </" + tag_key + ">";
+		ret += "<" + tag_val + ">" + sub_split_str[1] + "</" + tag_val + ">";
+	}
+	return ret;
+}
+
+std::string	get_form_infos(const std::string & rq_body)
+{
+	std::string form_infos;
+	std::string method;
+
+	method = parse_env("REQUEST_METHOD");
+
+	if (method == "POST")
+		form_infos = rq_body;
+	else if (method == "GET")
+		form_infos = parse_env("QUERY_STRING");
+
+	return form_infos;
+}
+
+std::string	get_value(const std::string & key, const std::string & rq_body)
+{
+	std::string	infos;
+	std::string	ret;
+	size_t		pos;
+	size_t      end;
+	size_t      len;
+
+	infos = get_form_infos(rq_body);
+	pos = str_tolower(infos).find(str_tolower(key));
+	if (pos == NPOS)
+		return "";
+	pos = infos.find_first_of("=", pos);
+	if (pos == NPOS)
+		return "";
+	pos++;
+	end = infos.find_first_of("&", pos);
+	if (end == NPOS)
+		end = infos.size();
+	len = end - pos;
+	ret = infos.substr(pos, len);
+
+	return ret;
+}
+
+void
+	fill_body_basic(char **env, std::string & http_body, const std::string & rq_body)
+{
+	std::string rq_method = "not found";
+	std::string rq_query;
+	std::string form_infos;
+
+	rq_method = parse_env("REQUEST_METHOD");
+	rq_query = parse_env("QUERY_STRING");
+
+	if (rq_method == "POST")
+		form_infos = rq_body;
+	else if (rq_method == "GET")
+		form_infos = rq_query;
+
+	http_body  = HTML_BODY_TOP;
+
+	http_body += "<br><h3>method used: </h3>";
+	http_body += "<p>" + rq_method + "</p>";
+
+	http_body += "<br><h3>form body: </h3>";
+	http_body += "<p>" + rq_body + "</p>";
+
+	http_body += "<br><h3>form query: </h3>";
+	http_body += "<p>" + rq_query + "</p>";
+
+	http_body += "<br><br><h3>output:</h3><br>";
+	http_body += print_form(form_infos, "p", "p");
+
+	http_body += "<br><br><h3>cgi_env_variables:</h3><br>";
+	http_body += print_env(env, "p");
+
+	http_body += HTML_BODY_BOTTOM;
+}
+
+size_t	eval_file_read(const std::string &path)
+{
+	if (::access(path.c_str(), F_OK) == -1)
+	{
+		std::perror("err access()");
+		return 404; // NOT_FOUND, file doesn't exist
+	}
+
+	if (::access(path.c_str(), R_OK) == -1)
+	{
+		std::perror("err access()");
+		return 403; // FORBIDDEN, file doesn't have access permission
+	}
+	return 0;
+}
+

srcs/cgi-bin/cgi_utils.hpp

@@ -0,0 +1,68 @@
+
+#ifndef CGI_UTILS_HPP
+# define CGI_UTILS_HPP
+
+# include <iostream>
+# include <string>
+# include <sstream>
+# include <fstream>
+# include <vector>
+# include <stdlib.h>	// getenv
+# include <algorithm>	// transform
+# include <unistd.h>	// sleep, close, access
+
+# define CR "\r"
+# define LF "\n"
+# define CRLF CR LF
+# define CRLF_SIZE 2
+# define NPOS std::string::npos
+
+# define HTML_BODY_TOP		"<!DOCTYPE html>"\
+							"<html>"\
+							"	<head>"\
+							"		<meta charset=\"UTF-8\">"\
+							"		<meta name=\"viewport\" content=\"width=device-width, initial-scale=1.0\">"\
+							"		<title>CGI</title>"\
+							"		<link href=\"./cgi_style.css\" type=\"text/css\" rel=\"stylesheet\">"\
+							"	</head>"\
+							"	<body>"\
+							"		<h1>CGI</h1><br>"
+# define HTML_BODY_BOTTOM	"		<br><h1>END CGI</h1>"\
+							"	</body>"\
+							"</html>"
+
+std::string
+str_tolower(std::string str);
+
+std::string
+trim(std::string str, char del);
+
+std::vector<std::string>
+split(const std::string & input, std::string delim, char ctrim = '\0');
+
+std::string
+itos(int n);
+
+std::string
+parse_env(const std::string & env);
+
+std::string
+print_env(char **env, std::string tag = "p");
+
+std::string
+get_form_infos(const std::string & rq_body);
+
+std::string
+get_value(const std::string & key, const std::string & rq_body);
+
+std::string
+print_form(std::string form, std::string key = "p", std::string val = "p");
+
+void
+fill_body_basic(char **env, std::string & http_body, const std::string & rq_body);
+
+size_t
+eval_file_read(const std::string &path);
+
+#endif
+

srcs/cgi-bin/tmp.php

@@ -0,0 +1,10 @@
+#! /usr/bin/php
+
+<?php
+
+	if(isset($_FILE["fileToUpload"]))
+		echo "there is a file";
+	else
+		echo "\n\nHeader: something\r\n\r\nno file\r\n";
+
+?>

srcs/cgi-bin/upload_file.php

@@ -0,0 +1,56 @@
+#! /usr/bin/php
+
+
+<?php
+
+
+
+$target_dir = "user_files/";
+$target_file = $target_dir . basename($_FILES["fileToUpload"]["name"]);
+$uploadOk = 1;
+$imageFileType = strtolower(pathinfo($target_file,PATHINFO_EXTENSION));
+
+// Check if image file is a actual image or fake image
+if(isset($_POST["submit"])) {
+  $check = getimagesize($_FILES["fileToUpload"]["tmp_name"]);
+  if($check !== false) {
+    echo "File is an image - " . $check["mime"] . ".";
+    $uploadOk = 1;
+  } else {
+    echo "File is not an image.";
+    $uploadOk = 0;
+  }
+}
+
+// Check if file already exists
+if (file_exists($target_file)) {
+  echo "Sorry, file already exists.";
+  $uploadOk = 0;
+}
+
+// Check file size
+if ($_FILES["fileToUpload"]["size"] > 500000) {
+  echo "Sorry, your file is too large.";
+  $uploadOk = 0;
+}
+
+// Allow certain file formats
+if($imageFileType != "jpg" && $imageFileType != "png" && $imageFileType != "jpeg"
+&& $imageFileType != "gif" ) {
+  echo "Sorry, only JPG, JPEG, PNG & GIF files are allowed.";
+  $uploadOk = 0;
+}
+
+// Check if $uploadOk is set to 0 by an error
+if ($uploadOk == 0) {
+  echo "Sorry, your file was not uploaded.";
+// if everything is ok, try to upload file
+} else {
+  if (move_uploaded_file($_FILES["fileToUpload"]["tmp_name"], $target_file)) {
+    echo "The file ". htmlspecialchars( basename( $_FILES["fileToUpload"]["name"])). " has been uploaded.";
+  } else {
+    echo "Sorry, there was an error uploading your file.";
+  }
+}
+?>
+

srcs/cgi-bin/upload_file1.php

@@ -0,0 +1,67 @@
+#! /usr/bin/php
+
+
+
+
+<?php
+//	$FileName=$_FILES['myFile']['filename'];
+#	$TmpName=$_FILES['myFile']['tmp_name'];
+#	move_uploaded_file($TmpName, $FileName);
+#	echo("File was uploaded successfully!");
+
+	// this part needs to be grabed from POST uri
+$target_dir = "./www/user_files/";
+//$target_dir = $_POST["upload_dir"];
+//$target_dir = $_GET["upload_dir"];
+//$target_file = $target_dir . basename($_FILES["fileToUpload"]["name"]);
+$target_file = $target_dir . basename($_FILES["myFile"]["name"]);
+
+$uploadOk = 1;
+$fileType = strtolower(pathinfo($target_file,PATHINFO_EXTENSION));
+//$imageFileType = strtolower(pathinfo($target_file,PATHINFO_EXTENSION));
+
+// Check if image file is a actual image or fake image
+//if(isset($_POST["submit"])) {
+//  $check = getimagesize($_FILES["fileToUpload"]["tmp_name"]);
+//  if($check !== false) {
+//    echo "File is an image - " . $check["mime"] . ".";
+//    $uploadOk = 1;
+//  } else {
+//    echo "File is not an image.";
+//    $uploadOk = 0;
+//  }
+//}
+
+// Check if file already exists
+if (file_exists($target_file)) {
+  echo "Sorry, file already exists.";
+  $uploadOk = 0;
+}
+
+// Check file size
+if ($_FILES["fileToUpload"]["size"] > 500000) {
+  echo "Sorry, your file is too large.";
+  $uploadOk = 0;
+}
+
+// Allow certain file formats
+//if($imageFileType != "jpg" && $imageFileType != "png" && $imageFileType != "jpeg"
+//&& $imageFileType != "gif" ) {
+ // echo "Sorry, only JPG, JPEG, PNG & GIF files are allowed.";
+ // $uploadOk = 0;
+//}
+
+// Check if $uploadOk is set to 0 by an error
+if ($uploadOk == 0) {
+  echo "Sorry, your file was not uploaded.";
+// if everything is ok, try to upload file
+} else {
+  if (move_uploaded_file($_FILES["fileToUpload"]["tmp_name"], $target_file)) {
+    echo "The file ". htmlspecialchars( basename( $_FILES["fileToUpload"]["name"])). " has been uploaded.";
+  } else {
+    echo "Sorry, there was an error uploading your file.";
+  }
+}
+?>
+
+

srcs/config/ConfigParser.hpp

@@ -14,8 +14,6 @@
 # include <cstdlib>			// strtol, stroul
 # include <iostream>		// cout, cin
 # include <fstream>			// ifstream
-//# include <unistd.h>		// access()
-# include <dirent.h>		// opendir(), doesn't work...
 # include <sys/stat.h>		// stat(), replaces opendir() don't bother with ERRNO ?
 # include <algorithm>		// sort() in Post
 
@@ -23,51 +21,41 @@ class ConfigParser {
 
 public:
 
-	// canonical?
-
-	ConfigParser(const char* path);	// a string?
+	ConfigParser();
 	~ConfigParser();
+	ConfigParser(const std::string &config_file);
 
-	// ideally i wouldn't have one cuz it makes no sense, when would i use it?
-//	ConfigParser & operator=(const ConfigParser& rhs);
+	void	read_config(const std::string &config_file);
 
 	std::vector<ServerConfig> *	parse(); // const?
-//	std::vector<ServerConfig>	parse(); // const?
 
 
 // i thought if it were an instance of this class you could call
-// private member functions from anywhere...
-	void	_print_content() const;
+// private member functions from anywhere... // QUESTION : Wut ?
+	void	print_content() const;
 
 
 private:
 	std::string		_content;
 
-	// explicit?
-		// what exaclty does explicit do again?
-	ConfigParser();	// might need a path as arg?
-	// should this be in private since it always needs a path?
-
 
 	ServerConfig	_parse_server(size_t *start);
 	LocationConfig	_parse_location(size_t *start);
 
-
 	void	_set_server_values(ServerConfig *server, const std::string key, std::string value);
 	void	_set_location_values(LocationConfig *location, const std::string key, std::string value);
 
-
-	std::string _pre_set_val_check(const std::string key, \
+/*	Extra */
+	std::string _pre_set_val_check(const std::string key,
 					const std::string value);
 
 	std::string	_get_first_word(size_t *curr);	// const?
 	std::string	_get_rest_of_line(size_t *curr);	// const?
 
-
 /*	Post Processing */
 	void	_post_processing(std::vector<ServerConfig> *servers);
-	bool	_find_root_path_location(std::vector<LocationConfig> locations); // const?
+	bool	_find_root_path_location(std::vector<LocationConfig> locations) const;
 
 };
 
-#endif
+#endif

srcs/config/LocationConfig.hpp

@@ -1,32 +1,14 @@
-/* ************************************************************************** */
-/*                                                                            */
-/*                                                        :::      ::::::::   */
-/*   LocationConfig.hpp                                 :+:      :+:    :+:   */
-/*                                                    +:+ +:+         +:+     */
-/*   By: lperrey <lperrey@student.42.fr>            +#+  +:+       +#+        */
-/*                                                +#+#+#+#+#+   +#+           */
-/*   Created: 2022/07/23 16:08:00 by me                #+#    #+#             */
-/*   Updated: 2022/08/12 18:12:23 by lperrey          ###   ########.fr       */
-/*                                                                            */
-/* ************************************************************************** */
 
 #ifndef LOCATIONCONFIG_HPP
 # define LOCATIONCONFIG_HPP
 
-# include <map>
 # include <vector>
 # include <string>
-# include <iostream>
-# include <sys/stat.h>	// stat()
 
 # include "utils.hpp"
 
-// again, struct instead?
-class LocationConfig
+struct LocationConfig
 {
-public:
-	// canonic stuff?
-
 	std::string					path;	// /path and /path/ are fine
 									// i remove trailing / systematically
 	std::string					root;
@@ -40,7 +22,7 @@ public:
 	int							redirect_status; // only in location
 	std::string					redirect_uri;	// only 1 per location
 
-	void	print_all()	// const?
+	void	print_all() const
 	{
 		std::cout << "\nPRINTING A LOCATION\n";
 
@@ -58,8 +40,7 @@ public:
 		std::cout << "------\n";
 	}
 
-// works a lot better than  using a compare function...
-	bool operator<(const LocationConfig& rhs) const
+	bool	operator<(const LocationConfig& rhs) const
     {
         int	comp_lhs = 0;
 		int	comp_rhs = 0;
@@ -81,19 +62,16 @@ public:
 		if (rhs.path[rhs.path.find_last_of("/") + 1] != '\0')
 			++comp_rhs; 
 
-		return (comp_lhs < comp_rhs);	// right comparison ? not <= ?
+		return (comp_lhs < comp_rhs);
     };
 
+	bool	operator==(const LocationConfig& rhs) const
+	{
+		if (path.compare(rhs.path) == 0)
+			return true;
+		return false;
+	}
 
 };
 
-
 #endif
-
-
-
-
-
-
-
-

srcs/config/ServerConfig.hpp

@@ -10,14 +10,9 @@
 # include <string>			// string
 # include <iostream>		// cout, cin
 
-// a class that's all public? just so we have options?
-class ServerConfig
+struct ServerConfig
 {
-public:
-	// do i need some canonic stuff?
-
 	std::vector<std::string>	server_name;
-	// we could shove default in here if we wanted to...
 
 	std::string	host;
 	std::string	port;
@@ -25,9 +20,7 @@ public:
 	std::string	root;	// ./www/ or www work www/ and www work
 		// i do remove trailing / tho
 	
-	size_t	client_body_limit;	// set to default max if none set
-	// 413 (Request Entity Too Large) if exceeded
-	// default is 1m 1 000 000 ?
+	size_t	client_body_limit;
 
 	std::vector<std::string>	index;
 	std::map<int, std::string>	error_pages;
@@ -35,7 +28,7 @@ public:
 	std::vector<LocationConfig>		locations;
 
 
-	void	print_all()	// const?
+	void	print_all() const
 	{
 		std::cout << "PRINTING A FULL SERVER CONFIG\n\n";
 
@@ -46,14 +39,14 @@ public:
 		for (size_t i = 0; i < index.size(); i++)
 			std::cout << index[i] << " ";
 		std::cout << "\nerror_pages: ";
-		for(std::map<int, std::string>::iterator it = error_pages.begin(); \
+		for(std::map<int, std::string>::const_iterator it = error_pages.begin(); \
 			it != error_pages.end(); it++)
 			std::cout << it->first << "--" << it->second << " ";
 //		for (size_t i = 0; i < error_pages.size(); i++)
 //			std::cout << error_pages->first << "--" << error_pages->second << " ";
 
 //		std::cout << "skiping Locations for now...\n";
-		for (std::vector<LocationConfig>::iterator it = locations.begin(); it < locations.end(); it++)
+		for (std::vector<LocationConfig>::const_iterator it = locations.begin(); it < locations.end(); it++)
 			it->print_all();
 
 		std::cout << "client_body_limit: " << client_body_limit << '\n';

srcs/config/extraConfig.cpp

@@ -1,11 +1,9 @@
 
-
-
 #include "ConfigParser.hpp"
 
 // should i be sending & references?
 // const?
-std::string	ConfigParser::_pre_set_val_check(const std::string key, \
+std::string	ConfigParser::_pre_set_val_check(const std::string key,
 			const std::string value)
 {
 
@@ -14,7 +12,7 @@ std::string	ConfigParser::_pre_set_val_check(const std::string key, \
 
 //	std::cout << "pre check\n";
 	if (key.find_first_of(";") != NPOS)
-		throw std::invalid_argument("bad config file arguments 2");
+		throw std::invalid_argument("bad config file arguments");
 
 	if (value.find_first_of("\t") != NPOS)
 		throw std::invalid_argument("why would you put tabs between values");
@@ -26,7 +24,7 @@ std::string	ConfigParser::_pre_set_val_check(const std::string key, \
 	// in theory value_find_last_of should find the only ;
 	if (i == NPOS || (value.find_last_not_of(" \n")) != i \
 		|| value.compare(";") == 0)
-		throw std::invalid_argument("bad config file arguments 4");
+		throw std::invalid_argument("bad config file arguments");
 
 
 	return (value.substr(0, i));
@@ -63,7 +61,7 @@ std::string	ConfigParser::_get_rest_of_line(size_t *curr)
 }
 
 
-void	ConfigParser::_print_content() const
+void	ConfigParser::print_content() const
 {
 	std::cout << _content;
 }

srcs/config/parser.cpp

@@ -1,25 +1,26 @@
 
 #include "ConfigParser.hpp"
 
-// Default
-ConfigParser::ConfigParser()
+ConfigParser::ConfigParser() {};
+ConfigParser::~ConfigParser() {};
+
+ConfigParser::ConfigParser(const std::string &config_file)
 {
-	std::cout << "Default Constructor\n";
-	// don't use yet, you have no idea what the defaults are
+	read_config(config_file);
 }
 
-ConfigParser::ConfigParser(const char*  path)
+void	ConfigParser::read_config(const std::string &config_file)
 {
-//	std::cout << "Param Constructor\n";
-
 	std::ifstream	file;
 	std::string		buf;
 	size_t 			comment;
 
-	_content.clear();
-	file.open(path);
-	if (file.is_open())
+	file.open(config_file.c_str());
+	if (!file)
+		throw std::invalid_argument("failed to open config");
+	else
 	{
+		_content.clear();
 		while (!file.eof())
 		{
 			getline(file, buf);
@@ -37,28 +38,9 @@ ConfigParser::ConfigParser(const char*  path)
 				_content.append(tmp + '\n');
 			}
 		}
-		file.close();
 	}
-	else
-		throw std::invalid_argument("failed to open config");
 }
 
-ConfigParser::~ConfigParser()
-{
-	// do i need to destroy anything, won't it handle itself?
-}
-
-/*
-ConfigParser &	ConfigParser::operator=(const ConfigParser& rhs)
-{
-	if (this == rhs)	// * & ?
-		return (*this);	// * ?
-
-	// make some stuff equal
-	return (*this);
-}
-*/
-
 // const?
 std::vector<ServerConfig> *	ConfigParser::parse()
 {
@@ -78,7 +60,7 @@ std::vector<ServerConfig> *	ConfigParser::parse()
 			throw std::invalid_argument("empty config file");
 		std::string	key = _content.substr(start, curr - start);
 		if (key != "server")
-			throw std::invalid_argument("bad config file arguments 1");
+			throw std::invalid_argument("bad config file arguments");
 		ret->push_back(_parse_server(&curr));
 	}
 	_post_processing(ret);
@@ -92,7 +74,7 @@ ServerConfig	ConfigParser::_parse_server(size_t *start)
 
 	ret.client_body_limit = 0;
 	if (curr == NPOS || _content[curr] != '{')
-		throw std::invalid_argument("bad config file syntax 1");
+		throw std::invalid_argument("bad config file syntax");
 
 	if ((curr = _content.find_first_of(" \t\n", curr + 1)) == NPOS)
 		throw std::invalid_argument("bad config file syntax");
@@ -135,13 +117,12 @@ LocationConfig	ConfigParser::_parse_location(size_t *start)
 	ret.path = _get_first_word(&curr);
 	if (ret.path[0] != '/')
 		throw std::invalid_argument("Location path require a leading /");
-	//	ret.path.insert(0, "/");
 	// in theory now curr should be right after the "path"
 
 	curr = _content.find_first_not_of(" \t\n", curr);
 
 	if (curr == NPOS || _content[curr] != '{')
-		throw std::invalid_argument("bad config file syntax 2");
+		throw std::invalid_argument("bad config file syntax");
 
 	if ((curr = _content.find_first_of(" \t\n", curr + 1)) == NPOS)
 		throw std::invalid_argument("bad config file syntax");
@@ -168,7 +149,7 @@ LocationConfig	ConfigParser::_parse_location(size_t *start)
 
 
 // should i be sending pointers or references?
-void	ConfigParser::_set_server_values(ServerConfig *server, \
+void	ConfigParser::_set_server_values(ServerConfig *server,
 			const std::string key, std::string value)
 {
 // should i be sending pointers or references?
@@ -181,20 +162,23 @@ void	ConfigParser::_set_server_values(ServerConfig *server, \
 		throw std::invalid_argument("missing value");
 	else if (key == "server_name" && server->server_name.empty())
 	{
-		for (std::vector<std::string>::iterator it = server->server_name.begin(); \
-				it < server->server_name.end(); it++)
+		for (size_t i = 0; i < size; i++)
 		{
-			if (it->compare(tmp_val[0]) == 0)
-				throw std::invalid_argument("server_name already exists");
+			for (std::vector<std::string>::const_iterator it = server->server_name.begin();
+					it < server->server_name.end(); it++)
+			{
+				if (it->compare(tmp_val[i]) == 0)
+					throw std::invalid_argument("server_name already exists");
+			}
+			server->server_name.push_back(tmp_val[i]);
 		}
-		server->server_name.push_back(tmp_val[0]);
 	}
-	else if (key == "listen" && size == 1 && server->host == "" \
-			&& server->port == "")
+	else if (key == "listen" && size == 1 && server->host == ""
+			&& server->port == "") // QUESTION LUKE : C'est quoi cette condition ? Si listen est vide ? Je comprends pas trop.
 	{
 		if (tmp_val[0].find_first_of(":") == NPOS)
 		{
-			if (!::isNumeric(tmp_val[0]))
+			if (!::isNumeric_btw(0, 65535, tmp_val[0]))
 				throw std::invalid_argument("bad port number");
 			server->host = "0.0.0.0";
 			server->port = tmp_val[0];
@@ -211,7 +195,7 @@ void	ConfigParser::_set_server_values(ServerConfig *server, \
 				if (!::isNumeric_btw(0, 255, ip[i]))
 					throw std::invalid_argument("bad host ip");
 			}
-			if (!::isNumeric(tmp2[1]))
+			if (!::isNumeric_btw(0, 65535, tmp2[1]))
 				throw std::invalid_argument("bad port number");
 			server->host = tmp2[0];
 			server->port = tmp2[1];
@@ -225,22 +209,25 @@ void	ConfigParser::_set_server_values(ServerConfig *server, \
 	//		tmp_val[0].push_back('/');
 		server->root = tmp_val[0];
 	}
-	else if (key == "client_body_limit" && size == 1 \
+	else if (key == "client_body_limit" && size == 1
 			&& server->client_body_limit == 0)
 	{
 		if (!::isNumeric(tmp_val[0]))
 			throw std::invalid_argument("client_body_limit not a number");
 		server->client_body_limit = std::strtoul(tmp_val[0].c_str(), NULL, 10);
+		if (errno == ERANGE || server->client_body_limit > (ULONG_MAX / KB) )
+			throw std::invalid_argument("client_body_limit too big");
+		server->client_body_limit = server->client_body_limit * KB;
 	}
 	else if (key == "index")
 	{
-		for (unsigned long i = 0; i != tmp_val.size(); i++)
+		for (size_t i = 0; i != tmp_val.size(); i++)
 			server->index.push_back(tmp_val[i]);
 	}
 	else if (key == "error_page")
 	{
-		std::string	path = tmp_val[tmp_val.size() - 1];
-		for (unsigned long i = 0; i != tmp_val.size() - 1; i++)
+		std::string	path = tmp_val[size - 1];
+		for (size_t i = 0; i < size - 1; i++)
 		{
 			if (!(isNumeric_btw(400, 599, tmp_val[i])))
 				throw std::invalid_argument("invalid error code");
@@ -256,7 +243,7 @@ void	ConfigParser::_set_server_values(ServerConfig *server, \
 
 
 // should i be sending pointers or references?
-void	ConfigParser::_set_location_values(LocationConfig *location, \
+void	ConfigParser::_set_location_values(LocationConfig *location,
 			const std::string key, std::string value)
 {
 // should i be sending pointers or references?
@@ -279,12 +266,12 @@ void	ConfigParser::_set_location_values(LocationConfig *location, \
 		location->autoindex = (tmp_val[0] == "on" ? true : false);
 	else if (key == "index")
 	{
-		for (unsigned long i = 0; i != tmp_val.size(); i++)
+		for (size_t i = 0; i < size; i++)
 			location->index.push_back(tmp_val[i]);
 	}
 	else if (key == "allow_methods" && location->allow_methods == 0)
 	{
-		for (unsigned long i = 0; i != tmp_val.size(); i++)
+		for (size_t i = 0; i < size; i++)
 		{
 			http_method	m = ::str_to_http_method(tmp_val[i]);
 			if (m == UNKNOWN)
@@ -294,23 +281,22 @@ void	ConfigParser::_set_location_values(LocationConfig *location, \
 	}
 	else if (key == "cgi_ext")
 	{
-		for (size_t i = 0; i < tmp_val.size(); i++)
+		for (size_t i = 0; i < size; i++)
 		{
 			if (tmp_val[i][0] == '.')
 				throw std::invalid_argument("cgi_ext should not have a leading '.'");
 			location->cgi_ext.push_back(tmp_val[i]);
 		}
 	}
-	else if (key == "redirect" && location->redirect_status == 0 \
+	else if (key == "redirect" && location->redirect_status == 0
 			&& location->redirect_uri == "")
 	{
-		if (tmp_val.size() != 2)
+		if (size != 2)
 			throw std::invalid_argument("wrong number of values");
 		if (tmp_val[0] != "301" && tmp_val[0] != "302" 
 			&& tmp_val[0] != "303" && tmp_val[0] != "307"
 			&& tmp_val[0] != "308")
 			throw std::invalid_argument("bad redirect status");
-		// std::cout << tmp_val[1] << '\n';
 		if (tmp_val[1].compare(0, 7, "http://")
 			&& tmp_val[1].compare(0, 8, "https://"))
 			throw std::invalid_argument("bad redirect uri");

srcs/config/postProcessing.cpp

@@ -22,12 +22,7 @@ void	ConfigParser::_post_processing(std::vector<ServerConfig> *servers)
 			throw std::invalid_argument("Config file needs an Index");
 
 		if (it->client_body_limit == 0)
-			it->client_body_limit = 5000;	// what is the recomended size?
-		
-
-		// if error_pages is left empty, we'll use the defaults which
-			// i believe are set elsewhere...
-
+			it->client_body_limit = 1 * MB;
 
 		if (!_find_root_path_location(it->locations))
 		{
@@ -46,6 +41,10 @@ void	ConfigParser::_post_processing(std::vector<ServerConfig> *servers)
 
 		while (it_l != it->locations.end())
 		{
+			if (it_l->path[it_l->path.size() - 1] == '/'
+				&& it_l->path.size() > 1)
+				it_l->path.erase(it_l->path.size() - 1);
+
 			if (it_l->root == "")
 				it_l->root = it->root;
 
@@ -57,15 +56,22 @@ void	ConfigParser::_post_processing(std::vector<ServerConfig> *servers)
 
 	// nothing to be done for cgi_ext, error_pages, redirect
 
-//			if (eval_file_type(it_l->root) == IS_DIR
-//				&& it_l->path[it_l->path.size() - 1] != '/')
-//				it_l->path.push_back('/');
-			if (it_l->path[it_l->path.size() - 1] == '/'
-				&& it_l->path.size() > 1)
-				it_l->path.erase(it_l->path.size() - 1);
-
 			++it_l;
 		}
+
+		it_l = it->locations.begin();
+		while (it_l != it->locations.end())
+		{
+			std::vector<LocationConfig>::const_iterator	tmp = it_l + 1;
+			while (tmp != it->locations.end())
+			{
+				if (it_l->path == tmp->path)
+					throw std::invalid_argument("Duplicate locations in config file");
+				++tmp;
+			}
+			++it_l;
+		}
+
 		std::sort(it->locations.begin(), it->locations.end());
 		std::reverse(it->locations.begin(), it->locations.end());
 
@@ -73,8 +79,7 @@ void	ConfigParser::_post_processing(std::vector<ServerConfig> *servers)
 	}
 }
 
-// const?
-bool	ConfigParser::_find_root_path_location(std::vector<LocationConfig> locations)
+bool	ConfigParser::_find_root_path_location(std::vector<LocationConfig> locations) const
 {
 	std::vector<LocationConfig>::const_iterator it = locations.begin();
 
@@ -85,5 +90,4 @@ bool	ConfigParser::_find_root_path_location(std::vector<LocationConfig> location
 		++it;
 	}
 	return false;
-}
-
+}

srcs/main.cpp

@@ -11,15 +11,9 @@ int	main(int ac, char **av)
 	{
 		std::string	config = (ac == 2 ? av[1] : "./default.config");
 
-	// like this just looks kinda gross, why bother creating an instance
-	// and not immediately parsing? like it servers no other purpose...
-	// what if i call parse directly in the constructor?
-	// oh because the constructor has no return, but still
-	// is there a better way?
 
-		ConfigParser	configParser(config.c_str());
-
-		// configParser._print_content();
+		ConfigParser	configParser(config);
+		// configParser.print_content();
 
 		// i don't love that servers has to be a pointer...
 		std::vector<ServerConfig>* servers = configParser.parse();
@@ -28,8 +22,8 @@ int	main(int ac, char **av)
 		for (std::vector<ServerConfig>::iterator it = servers->begin(); it < servers->end(); it++)
 		{
 			(void)0;
-		//	std::cout << it->server_name << " ";
-			// it->print_all();
+	//		std::cout << it->server_name << " ";
+	//		 it->print_all();
 		}
 
 

srcs/utils.cpp

@@ -153,10 +153,9 @@ std::string	http_methods_to_str(unsigned int methods)
 
 file_type	eval_file_type(const std::string &path)
 {
-	const char *tmp_path = path.c_str();
 	struct stat s;
 
-	if (stat(tmp_path, &s) != -1)
+	if (stat(path.c_str(), &s) != -1)
 	{
 		if (S_ISREG(s.st_mode))
 			return (IS_FILE);
@@ -171,18 +170,18 @@ file_type	eval_file_type(const std::string &path)
 	return (IS_OTHER);
 }
 
-size_t	eval_file_mode(std::string path, int mode)
+size_t	eval_file_access(const std::string &path, int mode)
 {
-	if (access(path.c_str(), F_OK) == -1)
+	if (::access(path.c_str(), F_OK) == -1)
 	{
 		std::perror("err access()");
 		return 404; // NOT_FOUND, file doesn't exist
 	}
 
-	if (access(path.c_str(), mode) == -1)
+	if (::access(path.c_str(), mode) == -1)
 	{
 		std::perror("err access()");
-		return 403; // FORBIDDEN, file doesn't have execution permission
+		return 403; // FORBIDDEN, file doesn't have access permission
 	}
 	return 0;
 }
@@ -257,7 +256,7 @@ size_t
 	std::vector<std::string>			list;
 	std::vector<std::string>::iterator	it;
 	std::vector<std::string>::iterator	it_end;
-	size_t								err = 0;
+	size_t								err_count = 0;
 	size_t								pos;
 	std::string							key;
 	std::string							val;
@@ -270,13 +269,13 @@ size_t
 		pos = (*it).find(':');
 		if (pos == NPOS)
 		{
-			err++;
+			err_count++;
 			continue;
 		}
 		key = (*it).substr(0, pos);
 		if ( key.find(' ') != NPOS )
 		{
-			err++;
+			err_count++;
 			continue;
 		}
 //		bad idea, in cgi we need to have the original value
@@ -285,7 +284,7 @@ size_t
 		val = ::trim(val, ' ');
 		fields.insert( std::pair<std::string, std::string>(key, val) );
 	}
-	return err;
+	return err_count;
 }
 
 void	str_map_key_tolower(std::map<std::string, std::string> & mp)

srcs/utils.hpp

@@ -6,6 +6,7 @@
 # include <map>
 # include <string>
 # include <sstream>
+# include <iostream>
 # include <cstdlib>		// strtol, strtoul
 # include <climits>		// LONG_MAX
 # include <cerrno>		// errno
@@ -21,6 +22,8 @@
 # define CRLF CR LF
 # define CRLF_SIZE 2
 # define NPOS std::string::npos
+# define KB 1024
+# define MB 1048576
 
 /* Equivalent for end of http header size :
 **	std::string(CRLF CRLF).size();
@@ -64,15 +67,41 @@ std::string					trim(std::string str, char del);
 http_method					str_to_http_method(std::string &str);
 std::string					http_methods_to_str(unsigned int methods);
 file_type					eval_file_type(const std::string &path);
-size_t						eval_file_mode(std::string path, int mode);
+size_t						eval_file_access(const std::string &path, int mode);
 void						replace_all_substr(std::string &str, const std::string &ori_substr, const std::string &new_substr);
 std::string					str_tolower(std::string str);
 std::string					extract_line(std::string & str, size_t pos = 0, std::string delim = "\n");
 std::string					get_line    (std::string   str, size_t pos = 0, std::string delim = "\n");
 size_t						parse_http_headers (std::string headers, std::map<std::string, std::string> & fields );
 void						str_map_key_tolower(std::map<std::string, std::string> & mp);
-void						throw_test();
 // debug
+void						throw_test();
 void	print_special(std::string str);
 
+
+/* Template */
+
+template <typename T1, typename T2 >
+void	print_pair(const std::pair<T1,T2> p)
+{
+	std::cout << p.first << ": ";
+	std::cout << p.second << "\n";
+}
+
+template <typename Key, typename T >
+void	print_map(const std::map<Key,T>& c)
+{
+	typename std::map<Key,T>::const_iterator	it = c.begin();
+	typename std::map<Key,T>::const_iterator	it_end = c.end();
+
+	std::cout << "  --print_map():\n";
+	std::cout << "map.size() = " << c.size() << "\n";
+	while (it != it_end)
+	{
+		print_pair(*it);
+		++it;
+	}
+	std::cout << "  --\n";
+}
+
 #endif

srcs/webserv/Webserv.hpp

@@ -71,71 +71,99 @@ class Webserv
 		std::map<std::string, std::string>	_mime_types;
 
 	// accept.cpp
-		void	_accept_connection(listen_socket &lsocket);
+		void		_accept_connection(listen_socket &lsocket);
 		std::map<std::string, std::string>
-				_extract_infos(struct sockaddr_in addr);
+					_extract_infos(struct sockaddr_in addr);
 	// request.cpp
-		void	_request(Client *client);
-		int		_read_request(Client *client);
+		void		_request(Client *client);
+		int			_read_request(Client *client);
 	// response.cpp
-		void	_response(Client *client);
-		int		_send_response(Client *client);
-		void	_append_base_headers(Client *client);
-		void	_construct_response(Client *client);
-		void	_process_method(Client *client, std::string &path);
-		void	_insert_status_line(Client *client);
-		void	_error_html_response(Client *client);
-		void	_append_body(Client *client, const std::string &body, const std::string &file_extension = "");
-		// ServerConfig			*_determine_process_server(Client *client); // cant be const cause of error_pages.operator[]
-		// const LocationConfig	*_determine_location(const ServerConfig &server, const std::string &path) const;
-		std::string				_determine_file_extension(const std::string &path) const;
-	// method_get.cpp
-
-// move later
+		void		_response(Client *client);
+		int			_send_response(Client *client);
+		void		_append_base_headers(Client *client);
+		void		_construct_response(Client *client);
+		void		_process_method(Client *client, std::string &path);
 		std::string	_replace_url_root(Client *client, std::string path);
-
-		void	_get(Client *client, std::string &path);
-		void	_get_file(Client *client, const std::string &path);
-		void	_autoindex(Client *client, const std::string &path);
+		void		_insert_status_line(Client *client);
+		void		_error_html_response(Client *client);
+		void		_append_body(Client *client, const std::string &body, const std::string &file_extension = "");
+	// method_get.cpp
+		void		_get(Client *client, std::string &path);
+		void		_get_file(Client *client, const std::string &path);
+		void		_autoindex(Client *client, const std::string &path);
+		std::string	_determine_file_extension(const std::string &path) const;
 	// method_post.cpp
-		void	_post(Client *client, const std::string &path);
-		void	_post_file(Client *client, const std::string &path);
+		void		_post(Client *client, const std::string &path);
+		void		_upload_files(Client *client);
 	// method_delete.cpp
-		void	_delete(Client *client, const std::string &path);
-		void	_delete_file(Client *client, const std::string &path);
-	// cgi_script.cpp
+		void		_delete(Client *client, const std::string &path);
+		void		_delete_file(Client *client, const std::string &path);
+	// epoll_update.cpp
+		int			_epoll_update(int fd, uint32_t events, int op);
+		int			_epoll_update(int fd, uint32_t events, int op, void *ptr);
+	// signal.cpp
+		void		_handle_last_signal();
+	// close.cpp
+		void		_close_client(int fd);
+		void		_close_all_clients();
+		void		_close_all_clients_fd();
+		void		_close_all_clients_cgi_fd();
+		void		_close_cgi_pipe_rfd(int fd);
+		void		_close_all_cgi_pipe_rfd();
+		void		_close_all_listen_sockets();
+		void		_reopen_lsocket(std::vector<listen_socket>::iterator it);
+		void		_handle_epoll_error_lsocket(uint32_t events, std::vector<listen_socket>::iterator it);
+		void		_handle_epoll_error_client(uint32_t events, int fd);
+	// init.cpp
+		void		_bind(int socket_fd, in_port_t port, std::string host);
+		void		_listen(int socket_fd, unsigned int max_connections);
+		void		_init_http_status_map();
+		void		_init_mime_types_map();
+	// timeout.cpp
+		void		_timeout();
+	// cgi.cpp
 		bool		_is_cgi(Client *client, std::string path);
 		size_t		_cgi_pos(Client *client, std::string &path, size_t pos);
-		std::string	_exec_cgi(Client *client);
-		char**		_set_env(Client *client);
-		char*		_dup_env(std::string var, std::string val);
-		char*		_dup_env(std::string var, int i);
-		std::string	_exec_script(Client *client, char **env);
+		void		_exec_cgi(Client *client);
+		void		_set_env_vector(Client *client, std::vector<std::string> &env_vector);
+		void		_set_env_cstr(char *env_cstr[], std::vector<std::string> &env_vector);
+		std::string	_dup_env(std::string var, std::string val);
+		std::string	_dup_env(std::string var, int i);
+		void		_exec_script(Client *client, char *env[]);
 		void		_check_script_output(Client *client, std::string & output);
 		void		_check_script_status(Client *client, std::string & output);
-		void		_check_script_fields(Client *client, std::string & output);
+		size_t		_check_script_fields(const std::string & output, size_t status);
+		void		_check_fields_duplicates(Client *client, std::string & output);
 		void		_add_script_body_length_header(std::string & output);
 		void		_remove_body_leading_empty_lines(std::string & output);
-	// epoll_update.cpp
-		int		_epoll_update(int fd, uint32_t events, int op);
-		int		_epoll_update(int fd, uint32_t events, int op, void *ptr);
-	// signal.cpp
-		void	_handle_last_signal();
-	// close.cpp
-		void	_close_client(int fd);
-		void	_close_all_clients();
-		void	_close_all_listen_sockets();
-		void	_reopen_lsocket(std::vector<listen_socket>::iterator it);
-		void	_handle_epoll_error_lsocket(uint32_t events, std::vector<listen_socket>::iterator it);
-		void	_handle_epoll_error_client(uint32_t events, int fd);
-	// init.cpp
-		void	_bind(int socket_fd, in_port_t port, std::string host);
-		void	_listen(int socket_fd, unsigned int max_connections);
-		void	_init_http_status_map();
-		void	_init_mime_types_map();
-	// timeout.cpp
-		void	_timeout();
+
+		Client	*_find_cgi_fd(int cgi_fd);
+		void	_read_cgi_output(Client *client);
+		void	_handle_epoll_error_cgi_fd(uint32_t events, Client *client);
+		void	_cgi_epollhup(uint32_t events, Client *client);
+
+
+	///////////////////////
+		class ExecFail : public std::exception
+		{
+			public :
+				virtual const char*	what() const throw() {
+					return ("Exec CGI fail");
+				};
+		};
+
 };
 
 #endif
 
+/*
+	HTTP Semantics:
+		https://www.rfc-editor.org/rfc/rfc9110.html
+		https://www.bortzmeyer.org/9110.html
+		https://www.bortzmeyer.org/cours-http-cnam.html
+	HTTP/1.1:
+		https://www.rfc-editor.org/rfc/rfc9112.html
+		https://www.bortzmeyer.org/9112.html
+	CGI:
+		https://www.rfc-editor.org/rfc/rfc3875.html
+*/

srcs/webserv/accept.cpp

@@ -21,8 +21,8 @@ void	Webserv::_accept_connection(listen_socket &lsocket)
 	::fcntl(accepted_fd, F_SETFL, O_NONBLOCK);
 
 	infos = _extract_infos(addr);
-	Client	client(accepted_fd, &lsocket, infos["port"], infos["ip"]);
-	_clients.push_back(client);
+	Client	new_client(accepted_fd, &lsocket, infos["port"], infos["ip"]);
+	_clients.push_back(new_client);
 	_epoll_update(accepted_fd, EPOLLIN, EPOLL_CTL_ADD);
 }
 

srcs/webserv/cgi.cpp

@@ -0,0 +1,387 @@
+
+#include "Webserv.hpp"
+/*
+	CGI RFC:
+	https://www.rfc-editor.org/rfc/rfc3875.html
+*/
+
+bool	Webserv::_is_cgi(Client *client, std::string path)
+{
+	std::string	script_path;
+	size_t		file_type;
+	size_t		file_mode = client->status;
+	size_t		pos = 0;
+
+	while (pos != NPOS)
+	{
+		pos = _cgi_pos(client, path, pos);
+		if (pos == NPOS)
+			break;
+		client->fill_script_path(path, pos);
+		script_path = client->get_rq_script_path();
+		file_type = ::eval_file_type(script_path);
+		if (file_type == IS_DIR) // but what if it's a symlink ?
+			continue;
+		if (file_type == IS_FILE)
+		{
+			file_mode = ::eval_file_access( script_path, X_OK );
+			if (!file_mode)
+				return true;
+		}
+	}
+	client->clear_script();
+	client->status = file_mode; // 404 not_found OR 403 forbidden
+	return false;
+}
+
+size_t	Webserv::_cgi_pos(Client *client, std::string &path, size_t pos)
+{
+	std::vector<std::string>					v_ext;
+	std::vector<std::string>::const_iterator	it;
+	std::vector<std::string>::const_iterator	it_end;
+	size_t										len;
+	std::locale									loc; // for isalpha()
+
+	v_ext = client->assigned_location->cgi_ext;
+	if (v_ext.empty())
+		return NPOS;
+	it_end = client->assigned_location->cgi_ext.end();
+	while (pos < path.size())
+	{
+		if (path.compare(pos, 2, "./") == 0)
+			pos += 2;
+		pos = path.find('.', pos);
+		if (pos == NPOS)
+			return pos;
+		it = client->assigned_location->cgi_ext.begin();
+		for ( ; it != it_end; ++it)
+		{
+			len = (*it).size();
+			if (path.compare(pos + 1, len, *it) == 0)
+				if ( !std::isalpha(path[pos + 1 + len], loc) )
+					return pos + 1 + len;
+		}
+		pos++;
+	}
+	return NPOS;
+}
+
+void	Webserv::_exec_cgi(Client *client)
+{
+	char*						env_cstr[19] = {NULL};
+	std::vector<std::string>	env_vector;
+	env_vector.reserve(18);
+	int i = 0;
+
+	_set_env_vector(client, env_vector);
+	try {
+		_set_env_cstr(env_cstr, env_vector);
+		_exec_script(client, env_cstr);
+
+		while (env_cstr[i] != NULL)
+			delete[] env_cstr[i++];
+		return;
+	}
+	catch (const Webserv::ExecFail& e)
+	{
+		while (env_cstr[i] != NULL)
+			delete[] env_cstr[i++];
+		throw;
+	}
+}
+
+std::string	Webserv::_dup_env(std::string var, std::string val = "")
+{
+	std::string str;
+
+	str = var + "=" + val;
+	return (str);
+}
+
+std::string	Webserv::_dup_env(std::string var, int i)
+{
+	std::string str;
+	std::string val;
+
+	val = ::itos(i);
+	str = var + "=" + val;
+	return (str);
+}
+
+// TODO : verifier que les variables sont corrects
+/*
+	https://www.rfc-editor.org/rfc/rfc3875#section-4.1
+*/
+void	Webserv::_set_env_vector(Client *client, std::vector<std::string> &env_vector)
+{
+	env_vector.push_back(_dup_env("AUTH_TYPE"));		// authentification not supporte
+	env_vector.push_back(_dup_env("CONTENT_LENGTH"		, client->get_rq_body().size()));
+	env_vector.push_back(_dup_env("CONTENT_TYPE"		, client->get_rq_headers("Content-Type")));
+	env_vector.push_back(_dup_env("GATEWAY_INTERFACE"	, "CGI/1.1")); // https://www.rfc-editor.org/rfc/rfc3875#section-4.1.4
+	env_vector.push_back(_dup_env("PATH_INFO"			, client->get_rq_script_info())); // LUKE: To Check
+	env_vector.push_back(_dup_env("PATH_TRANSLATED"));	// not supported // LUKE: Why not supported ?
+	env_vector.push_back(_dup_env("QUERY_STRING"		, client->get_rq_query()));
+	env_vector.push_back(_dup_env("REMOTE_ADDR"			, client->get_cl_ip()));
+	env_vector.push_back(_dup_env("REMOTE_HOST"			, client->get_cl_ip())); // equal to REMOTE_ADDR or empty
+	env_vector.push_back(_dup_env("REMOTE_IDENT"));		// authentification not supported
+	env_vector.push_back(_dup_env("REMOTE_USER"));		// authentification not supported
+	env_vector.push_back(_dup_env("REQUEST_METHOD"		, client->get_rq_method_str()));
+	env_vector.push_back(_dup_env("SCRIPT_NAME"			, "/" + client->get_rq_script_path())); // LUKE: To Check
+	env_vector.push_back(_dup_env("SERVER_NAME"			, client->get_cl_lsocket()->host));
+	env_vector.push_back(_dup_env("SERVER_PORT"			, client->get_cl_lsocket()->port));
+	env_vector.push_back(_dup_env("SERVER_PROTOCOL"		, "HTTP/1.1"));
+	env_vector.push_back(_dup_env("SERVER_SOFTWARE"		, "Webserv/0.1"));
+	env_vector.push_back(_dup_env("REDIRECT_STATUS"		, "200"));
+}
+
+void	Webserv::_set_env_cstr(char *env_cstr[], std::vector<std::string> &env_vector)
+{
+	std::vector<std::string>::const_iterator it = env_vector.begin();
+	std::vector<std::string>::const_iterator it_end = env_vector.end();
+	int i = 0;
+	while (it != it_end)
+	{
+		env_cstr[i] = new char[it->size()+1]; 
+		std::strcpy(env_cstr[i], it->c_str());
+		++it;
+		++i;
+	}
+	env_cstr[i] = NULL;
+}
+
+
+void	Webserv::_exec_script(Client *client, char *env[])
+{
+	#define RD 0
+	#define WR 1
+	#define FD_WR_TO_CHLD fd_in[WR]
+	#define FD_WR_TO_PRNT fd_out[WR]
+	#define FD_RD_FR_CHLD fd_out[RD]
+	#define FD_RD_FR_PRNT fd_in[RD]
+
+	pid_t			pid;
+	char * const nll[1]	= {NULL};
+	std::string		script_output;
+	std::string		body = client->get_rq_body();
+	int				fd_in[2];
+	int				fd_out[2];
+	std::string		path;
+
+	::pipe(fd_in);
+	::pipe(fd_out);
+
+	pid = fork();
+	if (pid == -1)
+		std::perror("err fork()");
+	else if (pid == 0) // child
+	{
+		std::signal(SIGPIPE, SIG_DFL);
+		std::signal(SIGINT, SIG_DFL);
+
+		_close_all_clients_fd();
+		::close(_epfd);
+		::close(FD_WR_TO_CHLD);
+		::close(FD_RD_FR_CHLD);
+		if (dup2(FD_RD_FR_PRNT, STDIN_FILENO) == -1)
+		{
+			std::perror("err dup2()");
+			::close(FD_RD_FR_PRNT); // Valgind debug, not essential
+			::close(FD_WR_TO_PRNT); // Valgind debug, not essential
+			throw ExecFail();
+		}
+		if (dup2(FD_WR_TO_PRNT, STDOUT_FILENO) == -1)
+		{
+			std::perror("err dup2()");
+			::close(FD_RD_FR_PRNT); // Valgind debug, not essential
+			::close(FD_WR_TO_PRNT); // Valgind debug, not essential
+			throw ExecFail();
+		}
+
+		::close(FD_RD_FR_PRNT);
+		::close(FD_WR_TO_PRNT);
+
+		path = client->get_rq_script_path(); // Wut ? Only relative path ?
+/*DEBUG*/std::cerr << "execve:[" << path << "]\n";
+		if (::execve(path.c_str(), nll, env) == -1) // replace path for debug error forcing
+		{
+			std::perror("err execve()");
+			::close(STDIN_FILENO); // Valgind debug, not essential
+			::close(STDOUT_FILENO); // Valgind debug, not essential
+			throw ExecFail();
+		}
+	}
+	else //parent
+	{
+		::close(FD_RD_FR_PRNT);
+		::close(FD_WR_TO_PRNT);
+		::write(FD_WR_TO_CHLD, body.c_str(), body.size()); // move this before the fork ?
+		::close(FD_WR_TO_CHLD);
+
+		// add FD_RD_FR_CHLD to epoll,
+		_epoll_update(FD_RD_FR_CHLD, EPOLLIN, EPOLL_CTL_ADD);
+		// stop monitoring client->fd until the cgi-script as done is job
+		_epoll_update(client->get_cl_fd(), 0, EPOLL_CTL_DEL);
+
+		client->cgi_pipe_rfd = FD_RD_FR_CHLD;
+		client->cgi_pid = pid;
+	}
+}
+
+#define STATUS_500 std::string("Status: 500" CRLF CRLF);
+
+void	Webserv::_check_script_output(Client *client, std::string & output)
+{
+	size_t pos;
+	pos = client->cgi_output.find(CRLF CRLF);
+	if (pos == 0 || pos == NPOS)
+	{
+		client->status = 500;;
+		return;
+	}
+	_check_script_status(client, output);
+	if (client->status >= 400 && client->status < 600)
+		return;
+//*DEBUG*/ std::cout << "\n" B_PURPLE "[script status]:" RESET "\n"; ::print_special(output); std::cout << B_PURPLE "-----------" RESET "\n\n";
+	client->status = _check_script_fields(output, client->status);
+	_check_fields_duplicates(client, output);
+//*DEBUG*/ std::cout << "\n" B_PURPLE "[script fields]:" RESET "\n"; ::print_special(output); std::cout << B_PURPLE "-----------" RESET "\n\n";
+	_remove_body_leading_empty_lines(output);
+//*DEBUG*/ std::cout << "\n" B_PURPLE "[script empty lines]:" RESET "\n"; ::print_special(output); std::cout << B_PURPLE "-----------" RESET "\n\n";
+	_add_script_body_length_header(output);
+//*DEBUG*/ std::cout << "\n" B_PURPLE "[script content length]:" RESET "\n"; ::print_special(output); std::cout << B_PURPLE "-----------" RESET "\n\n";
+}
+
+void	Webserv::_check_script_status(Client *client, std::string & output)
+{
+	size_t	pos;
+	int		status_pos;
+
+	pos = output.find("Status:");
+	if (pos != NPOS)
+	{
+		status_pos = pos + std::string("Status:").size();
+		client->status = std::strtoul(output.c_str() + status_pos, NULL, 10);
+		::extract_line(output, pos, CRLF);
+	}
+	else
+		client->status = 200;
+}
+
+size_t	Webserv::_check_script_fields(const std::string & output, size_t status)
+{
+	std::string	headers;
+	std::string body;
+	size_t		pos;
+
+	pos = output.find(CRLF CRLF);
+	if (pos == NPOS) // there is not empty line
+		return 500;
+	headers = output.substr(0, pos);
+	body = output.substr(pos + CRLF_SIZE * 2);
+	headers = str_tolower(headers);
+	pos = headers.find("content-type");
+	if (pos == NPOS) // there is no content-type field
+	{
+		if (!body.empty()) // there is body
+			return 500;
+		if (headers.find("location") == NPOS) // there is no location field
+			return 500;
+	}
+	else if (headers.find("location") != NPOS) // there is a location field
+	{
+		if (body.empty()) // there is no body
+			return 500;
+	}
+	return status;
+}
+
+void	Webserv::_check_fields_duplicates(Client *client, std::string & output)
+{
+	std::map<std::string, std::string>				srv_fld;	// server_field
+	std::map<std::string, std::string>				scr_fld;	// script_field
+	std::map<std::string, std::string>::iterator	it_srv;
+	std::map<std::string, std::string>::iterator	it_scr;
+	std::string										tmp;
+	size_t											pos;
+
+	// put server headers in map
+	tmp = client->response;
+	pos = tmp.find(CRLF CRLF);
+	if (pos != NPOS)
+		tmp.erase(pos);
+	::parse_http_headers(tmp, srv_fld);
+	// put script headers in map
+	tmp = output;
+	pos = tmp.find(CRLF CRLF);
+	if (pos != NPOS)
+		tmp.erase(pos);
+	::parse_http_headers(tmp, scr_fld);
+	// compare both map to supress duplicates
+	for (it_srv = srv_fld.begin(); it_srv != srv_fld.end(); it_srv++)
+	{
+		for (it_scr = scr_fld.begin(); it_scr != scr_fld.end(); it_scr++)
+		{
+			if (str_tolower(it_srv->first) == str_tolower(it_scr->first))
+			{
+				pos = client->response.find(it_srv->first);
+				::extract_line(client->response, pos, CRLF);
+			}
+		}
+	}
+}
+
+void	Webserv::_remove_body_leading_empty_lines(std::string & output)
+{
+	size_t		pos;
+	size_t		pos_empty;
+
+	pos = output.find(CRLF CRLF);
+	if (pos == NPOS)
+		return;
+	pos += CRLF_SIZE * 2;
+	pos_empty = pos;
+	while (pos_empty == pos)
+	{
+		pos = output.find(CRLF, pos);
+		if (pos == pos_empty)
+			extract_line(output, pos, CRLF);
+	}
+}
+
+void	Webserv::_add_script_body_length_header(std::string & output)
+{
+	std::map<std::string, std::string>				field;
+	std::map<std::string, std::string>::iterator	it;
+	std::stringstream								str_len;
+	std::string										tmp;
+	size_t											pos;
+	size_t											len;
+
+	pos = output.find(CRLF CRLF);
+	if (pos != NPOS)
+		tmp = output.substr(pos + (CRLF_SIZE * 2));
+	len = tmp.size();
+	str_len << len;
+	
+	// put script headers in map
+	tmp = output;
+	pos = tmp.find(CRLF CRLF);
+	if (pos != NPOS)
+		tmp.erase(pos);
+	::parse_http_headers(tmp, field);
+	// case insensitive search in map for "Content-Length"
+	tmp = "Content-Length";
+	for (it = field.begin(); it != field.end(); ++it)
+	{
+		if (str_tolower(it->first) == str_tolower(tmp))
+		{
+			pos = output.find(it->first);
+			::extract_line(output, pos, CRLF);
+		}
+	}
+	tmp += ": ";
+	tmp += str_len.str();
+	tmp += CRLF;
+	output.insert(0, tmp);
+}
+

srcs/webserv/cgi_script.cpp

@@ -1,304 +0,0 @@
-
-#include "Webserv.hpp"
-
-bool	Webserv::_is_cgi(Client *client, std::string path)
-{
-	std::string	script_path;
-	size_t		file_type;
-	size_t		file_mode;
-	size_t		pos = 0;
-
-	while (pos != NPOS)
-	{
-		pos = _cgi_pos(client, path, pos);
-		if (pos == NPOS)
-			break;
-		client->fill_script_path(path, pos);
-		script_path = "." + client->get_rq_script_path();
-		file_type = ::eval_file_type(script_path);
-		if (file_type == IS_DIR) // but what if it's a symlink ?
-			continue;
-		if (file_type == IS_FILE)
-		{
-			file_mode = ::eval_file_mode( script_path, X_OK );
-			if (!file_mode)
-				return true;
-		}
-	}
-	client->clear_script();
-	client->status = file_mode; // 404 not_found OR 403 forbidden
-	return false;
-}
-
-size_t	Webserv::_cgi_pos(Client *client, std::string &path, size_t pos)
-{
-	std::vector<std::string>					v_ext;
-	std::vector<std::string>::const_iterator	it;
-	std::vector<std::string>::const_iterator	it_end;
-	size_t										len;
-	std::locale									loc; // for isalpha()
-
-	v_ext = client->assigned_location->cgi_ext;
-	if (v_ext.empty())
-		return NPOS;
-	it_end = client->assigned_location->cgi_ext.end();
-	while (pos < path.size())
-	{
-		if (path.compare(pos, 2, "./") == 0)
-			pos += 2;
-		pos = path.find('.', pos);
-		if (pos == NPOS)
-			return pos;
-		it = client->assigned_location->cgi_ext.begin();
-		for ( ; it != it_end; ++it)
-		{
-			len = (*it).size();
-			if (path.compare(pos + 1, len, *it) == 0)
-				if ( !std::isalpha(path[pos + 1 + len], loc) )
-					return pos + 1 + len;
-		}
-		pos++;
-	}
-	return NPOS;
-}
-
-std::string	Webserv::_exec_cgi(Client *client)
-{
-	char**			env;
-	std::string		script_output;
-
-	env = _set_env(client);
-	script_output = _exec_script(client, env);
-
-	for (int i = 0; env[i]; i++)
-		delete[] env[i];
-	delete[] env;
-
-	return script_output;
-}
-
-char*	Webserv::_dup_env(std::string var, std::string val = "")
-{
-	std::string str;
-
-	str = var + "=" + val;
-	return ( strdup(str.c_str()) );
-}
-
-char*	Webserv::_dup_env(std::string var, int i)
-{
-	std::string str;
-	std::string val;
-
-	val = ::itos(i);
-	str = var + "=" + val;
-	// TODO change strdup for something with new
-	return ( strdup(str.c_str()) );
-}
-
-char**	Webserv::_set_env(Client *client)
-{
-	char**	env = new char*[19];
-
-	env[0]  = _dup_env("AUTH_TYPE");		// authentification not supported
-	env[1]  = _dup_env("CONTENT_LENGTH"		, client->get_rq_body().size());
-	env[2]  = _dup_env("CONTENT_TYPE"		, client->get_rq_headers("Content-Type"));
-	env[3]  = _dup_env("GATEWAY_INTERFACE"	, "CGI/1.1"); // https://www.rfc-editor.org/rfc/rfc3875
-	env[4]  = _dup_env("PATH_INFO"			, client->get_rq_script_info());
-	env[5]  = _dup_env("PATH_TRANSLATED");	// not supported
-	env[6]  = _dup_env("QUERY_STRING"		, client->get_rq_query());
-	env[7]  = _dup_env("REMOTE_ADDR"		, client->get_cl_ip());
-	env[8]  = _dup_env("REMOTE_HOST"		, client->get_rq_headers("Host")); // just test
-	env[9]  = _dup_env("REMOTE_IDENT");		// authentification not supported
-	env[10] = _dup_env("REMOTE_USER");		// authentification not supported
-	env[11] = _dup_env("REQUEST_METHOD"		, client->get_rq_method_str());
-	env[12] = _dup_env("SCRIPT_NAME"		, client->get_rq_script_path());
-	env[13] = _dup_env("SERVER_NAME"		, client->get_rq_hostname());
-	env[14] = _dup_env("SERVER_PORT"		, client->get_rq_port());
-	env[15] = _dup_env("SERVER_PROTOCOL"	, client->get_rq_version());
-	env[16] = _dup_env("SERVER_SOFTWARE"	, "webser/1.0");
-	env[17] = _dup_env("REDIRECT_STATUS"	, "200");
-	env[18] = NULL;
-
-	return env;
-}
-
-std::string	Webserv::_exec_script(Client *client, char **env)
-{
-	#define RD 0
-	#define WR 1
-	#define CGI_BUF_SIZE 10
-	#define FD_WR_TO_CHLD fd_in[WR]
-	#define FD_WR_TO_PRNT fd_out[WR]
-	#define FD_RD_FR_CHLD fd_out[RD]
-	#define FD_RD_FR_PRNT fd_in[RD]
-
-	pid_t			pid;
-	char			buf[CGI_BUF_SIZE]; // WIP define buffer
-	char * const *	nll = NULL;
-	std::string		script_output;
-	std::string		body = client->get_rq_body();
-	int				fd_in[2];
-	int				fd_out[2];
-	std::string		path;
-
-	pipe(fd_in);
-	pipe(fd_out);
-
-	pid = fork();
-	if (pid == -1)
-		std::cerr << "fork crashed" << std::endl;
-	else if (pid == 0) // child
-	{   
-		close(FD_WR_TO_CHLD);
-		close(FD_RD_FR_CHLD);
-		dup2(FD_RD_FR_PRNT, STDIN_FILENO);
-		dup2(FD_WR_TO_PRNT, STDOUT_FILENO);
-		path = "." + client->get_rq_script_path();
-/*DEBUG*/std::cerr << "execve:[" << path << "]\n";
-		execve(path.c_str(), nll, env);
-		// for tests execve crash :
-		//execve("wrong", nll, env);
-		std::cerr << "execve crashed.\n";
-
-		// TODO HUGO : check errno
-	}
-	else //parent
-	{
-		close(FD_RD_FR_PRNT);
-		close(FD_WR_TO_PRNT);
-		write(FD_WR_TO_CHLD, body.c_str(), body.size());
-		close(FD_WR_TO_CHLD);
-		waitpid(-1, NULL, 0);
-
-		memset(buf, '\0', CGI_BUF_SIZE);
-		while (read(FD_RD_FR_CHLD, buf, CGI_BUF_SIZE - 1) > 0)
-		{
-			script_output += buf;
-			memset(buf, '\0', CGI_BUF_SIZE);
-		}
-		close(FD_RD_FR_CHLD);
-	}
-	if (script_output.empty())
-		script_output = "Status: 500\r\n\r\n";
-
-	return script_output;
-}
-
-void	Webserv::_check_script_output(Client *client, std::string & output)
-{
-	_check_script_status(client, output);
-	_check_script_fields(client, output);
-	_add_script_body_length_header(output);
-	_remove_body_leading_empty_lines(output);
-	// _check_script_empty_lines(client, output);
-	// _check_script_space_colons(client, output);
-	// _check_script_new_lines(client, output);
-}
-
-void	Webserv::_check_script_status(Client *client, std::string & output)
-{
-	size_t	pos;
-	int		status_pos;
-
-	pos = output.find("Status:");
-	if (pos != NPOS)
-	{
-		status_pos = pos + std::string("Status:").size();
-		client->status = std::strtoul(output.c_str() + status_pos, NULL, 10);
-		::extract_line(output, pos, CRLF);
-	}
-	else
-		client->status = 200;
-}
-
-void	Webserv::_check_script_fields(Client *client, std::string & output)
-{
-	std::map<std::string, std::string>				srv_fld;	// server_field
-	std::map<std::string, std::string>				scr_fld;	// script_field
-	std::map<std::string, std::string>::iterator	it_srv;
-	std::map<std::string, std::string>::iterator	it_scr;
-	std::string										tmp;
-	size_t											pos;
-
-	// put server headers in map
-	tmp = client->response;
-	pos = tmp.find(CRLF CRLF);
-	if (pos != NPOS)
-		tmp.erase(pos);
-	::parse_http_headers(tmp, srv_fld);
-	// put script headers in map
-	tmp = output;
-	pos = tmp.find(CRLF CRLF);
-	if (pos != NPOS)
-		tmp.erase(pos);
-	::parse_http_headers(tmp, scr_fld);
-	// compare both map to supress duplicates
-	for (it_srv = srv_fld.begin(); it_srv != srv_fld.end(); it_srv++)
-	{
-		for (it_scr = scr_fld.begin(); it_scr != scr_fld.end(); it_scr++)
-		{
-			if (str_tolower(it_srv->first) == str_tolower(it_scr->first))
-			{
-				pos = client->response.find(it_srv->first);
-				::extract_line(client->response, pos, CRLF);
-			}
-		}
-	}
-}
-
-void	Webserv::_remove_body_leading_empty_lines(std::string & output)
-{
-	size_t		pos;
-	size_t		pos_empty;
-
-	pos = output.find(CRLF CRLF);
-	if (pos == NPOS)
-		return;
-	pos += CRLF_SIZE * 2;
-	pos_empty = pos;
-	while (pos_empty == pos)
-	{
-		pos = output.find(CRLF, pos);
-		if (pos == pos_empty)
-			extract_line(output, pos, CRLF);
-	}
-}
-
-void	Webserv::_add_script_body_length_header(std::string & output)
-{
-	std::map<std::string, std::string>				field;
-	std::map<std::string, std::string>::iterator	it;
-	std::stringstream								str_len;
-	std::string										tmp;
-	size_t											pos;
-	size_t											len;
-
-	pos = output.find(CRLF CRLF);
-	if (pos != NPOS)
-		tmp = output.substr(pos + CRLF_SIZE);
-	len = tmp.size();
-	str_len << len;
-	
-	// put script headers in map
-	tmp = output;
-	pos = tmp.find(CRLF CRLF);
-	if (pos != NPOS)
-		tmp.erase(pos);
-	::parse_http_headers(tmp, field);
-	// case insensitive search in map for "Content-Length"
-	tmp = "Content-Length";
-	for (it = field.begin(); it != field.end(); ++it)
-	{
-		if (str_tolower(it->first) == str_tolower(tmp))
-		{
-			pos = output.find(it->first);
-			::extract_line(output, pos, CRLF);
-		}
-	}
-	tmp += ": ";
-	tmp += str_len.str();
-	tmp += CRLF;
-	output.insert(0, tmp);
-}
-

srcs/webserv/close.cpp

@@ -4,7 +4,8 @@
 void	Webserv::_close_client(int fd)
 {
 	std::vector<Client>::iterator	it = _clients.begin();
-	while (it != _clients.end())
+	std::vector<Client>::iterator	it_end = _clients.end();
+	while (it != it_end)
 	{
 		if (*it == fd)
 		{
@@ -21,18 +22,45 @@ void	Webserv::_close_client(int fd)
 
 void	Webserv::_close_all_clients()
 {
-	while (!_clients.empty())
+	_close_all_clients_fd();
+	_clients.clear();
+}
+
+void	Webserv::_close_all_clients_fd()
+{
+	_close_all_clients_cgi_fd();
+	std::vector<Client>::iterator	it = _clients.begin();
+	std::vector<Client>::iterator	it_end = _clients.end();
+	while (it != it_end)
 	{
 		// _epoll_update(_clients.back().fd, 0, EPOLL_CTL_DEL); // normalement superflu, DEBUG
-		std::cerr << "close fd " << _clients.back().get_cl_fd() << "\n";
-		if (::close(_clients.back().get_cl_fd()) == -1)
+		std::cerr << "close fd " << it->get_cl_fd() << "\n";
+		if (::close(it->get_cl_fd()) == -1)
 			std::perror("err close()");
-		_clients.pop_back();
+		++it;
 	}
 }
 
-void	Webserv::_close_all_listen_sockets()
+void	Webserv::_close_all_clients_cgi_fd()
 {
+	std::vector<Client>::iterator	it = _clients.begin();
+	std::vector<Client>::iterator	it_end = _clients.end();
+	while (it != it_end)
+	{
+		// _epoll_update(_clients.back().fd, 0, EPOLL_CTL_DEL); // normalement superflu, DEBUG
+		if (it->cgi_pipe_rfd)
+		{
+			std::cerr << "close cgi-fd " << it->cgi_pipe_rfd << "\n";
+			if (::close(it->cgi_pipe_rfd) == -1)
+				std::perror("err close()");
+		}
+		++it;
+	}
+}
+
+
+void	Webserv::_close_all_listen_sockets()
+{ // TODO : change like clients (clear in place of pop_back)
 	while (!_listen_sockets.empty())
 	{
 		// _epoll_update(_listen_sockets.back().fd, 0, EPOLL_CTL_DEL); // normalement superflu, DEBUG
@@ -54,7 +82,7 @@ void	Webserv::_reopen_lsocket(std::vector<listen_socket>::iterator it)
 	if (::close(it->fd) == -1)
 		std::perror("err close()");
 	std::cerr << "try to reopen lsocket\n";
-	ret = ::socket(AF_INET, SOCK_STREAM | SOCK_NONBLOCK, 0); // (SOCK_CLOEXEC) for CGI fork ?
+	ret = ::socket(AF_INET, SOCK_STREAM | SOCK_NONBLOCK | SOCK_CLOEXEC, 0);
 	if (ret == -1)
 	{
 		std::perror("err socket()");
@@ -108,5 +136,8 @@ void	Webserv::_handle_epoll_error_client(uint32_t events, int fd)
 		std::cerr << "EPOLLERR on client fd " << fd << "\n"; // DEBUG
 	if (events & EPOLLHUP)
 		std::cerr << "EPOLLHUP on client fd " << fd << "\n"; // DEBUG
-	_close_client(fd);
+	if (std::find(_clients.begin(), _clients.end(), fd) != _clients.end())
+		std::cerr << "Found the client in _clients" << "\n"; // DEBUG
+
+	_close_client(fd); // " EPOLLHUP on client fd 8 " en boucle quand on mattraque un peu les CGI, donc il ne le trouve pas dans _clients. Etrange.
 }

srcs/webserv/http_status.hpp

@@ -2,7 +2,9 @@
 #ifndef HTTP_STATUS_HPP
 # define HTTP_STATUS_HPP
 
-// https://www.iana.org/assignments/http-status-codes/http-status-codes.xhtml
+/*
+	https://www.iana.org/assignments/http-status-codes/http-status-codes.xhtml
+*/
 
 /*
 	First version of macro HTML_ERROR(STATUS) dont work with call like this :

srcs/webserv/init.cpp

@@ -24,7 +24,7 @@ void	Webserv::init_virtual_servers(std::vector<ServerConfig>* servers)
 			continue;
 		}
 
-		ret = ::socket(AF_INET, SOCK_STREAM | SOCK_NONBLOCK, 0); // (SOCK_CLOEXEC) for CGI fork ?
+		ret = ::socket(AF_INET, SOCK_STREAM | SOCK_NONBLOCK | SOCK_CLOEXEC, 0);
 		if (ret == -1)
 		{
 			std::perror("err socket()");
@@ -112,9 +112,11 @@ void	Webserv::_init_http_status_map()
 	_http_status.insert(status_pair(405, S405));
 	_http_status.insert(status_pair(408, S408));
 	_http_status.insert(status_pair(413, S413));
+	_http_status.insert(status_pair(415, S415));
 
 	_http_status.insert(status_pair(500, S500));
 	_http_status.insert(status_pair(501, S501));
+	_http_status.insert(status_pair(505, S505));
 }
 
 void	Webserv::_init_mime_types_map()

srcs/webserv/method_delete.cpp

@@ -4,7 +4,6 @@
 void	Webserv::_delete(Client *client, const std::string &path)
 {
 /*
-	WIP
 	https://www.rfc-editor.org/rfc/rfc9110.html#name-delete
 */
 	_delete_file(client, path);
@@ -12,24 +11,21 @@ void	Webserv::_delete(Client *client, const std::string &path)
 
 void	Webserv::_delete_file(Client *client, const std::string &path)
 {
-	if (access(path.c_str(), F_OK) == -1)
-	{
-		std::perror("err access()");
-		client->status = 404;
-		return ;
-	}
+	std::cout << "_delete_file()\n";
+	client->status = ::eval_file_access(path, W_OK);
+	if (client->status)
+		return;
 
-	if (access(path.c_str(), W_OK) == -1)
-	{
-		std::perror("err access()");
-		client->status = 403;
-		return ;
-	}
-
-	if (remove(path.c_str()) == -1)
+	if (std::remove(path.c_str()) == -1)
 	{
 		std::perror("err remove()");
-		client->status = 500;
+		if (errno == ENOTEMPTY || errno == EEXIST)
+			client->status = 403;
+		else
+			client->status = 500;
 		return ;
 	}
+
+	client->status = 204;
+	client->response.append(CRLF);
 }

srcs/webserv/method_get.cpp

@@ -1,24 +1,14 @@
 
 #include "Webserv.hpp"
-
-std::string	Webserv::_replace_url_root(Client *client, std::string path)
-{
-	std::cerr << "assigned_location->path =	" << client->assigned_location->path << "\n"; // debug
-	std::cerr << "path before =			" << path << "\n"; // DEBUG
-	if (client->assigned_location->path == "/")
-		path.insert(0, client->assigned_location->root);
-	else
-		path.replace(0, client->assigned_location->path.size(), client->assigned_location->root);
-	std::cerr << "path after =			" << path << "\n"; // DEBUG
-	return path;
-}
+#define MAX_FILESIZE 1 * MB // unused
 
 // const?
+/*
+	https://www.rfc-editor.org/rfc/rfc9110.html#name-get
+*/
 void	Webserv::_get(Client *client, std::string &path)
 {
-
-
-// Index/Autoindex block
+	std::cout << "_get()\n";
 	if (eval_file_type(path) == IS_DIR)
 	{
 		if (path[path.size() - 1] != '/')
@@ -29,44 +19,36 @@ void	Webserv::_get(Client *client, std::string &path)
 			{
 				path.append(client->assigned_location->index[i]);
 				_get_file(client, path);
+				std::cerr << "Added an index\n"; //debug
 				return ;
 			}
 		}
 		if (client->assigned_location->autoindex == true)
 			_autoindex(client, path);
+		else
+			client->status = 404;
 	}
 	else
 		_get_file(client, path);
 }
 
-# define MAX_FILESIZE 1000000 // (1Mo)
-void	Webserv::_get_file(Client *client, const std::string &path)
-{
 /*
 	std::ios::binary
 	https://gcc.gnu.org/onlinedocs/libstdc++/manual/fstreams.html#std.io.filestreams.binary
 	tldr : its seems to not be so simple to do read/write of binary file in a portable way.
 */
+void	Webserv::_get_file(Client *client, const std::string &path)
+{
 	std::ifstream		ifd; // For chunk, ifstream directly in struct CLient for multiples read without close() ?
 	std::stringstream	buf;
 
 	std::cout << "_get_file()\n";
 
-	if (access(path.c_str(), F_OK) == -1)
-	{
-		std::perror("err access()");
-		client->status = 404;
-		return ;
-	}
+	client->status = ::eval_file_access(path, R_OK);
+	if (client->status)
+		return;
 
-	if (access(path.c_str(), R_OK) == -1)
-	{
-		std::perror("err access()");
-		client->status = 403;
-		return ;
-	}
-
-	ifd.open(path.c_str(), std::ios::ate);
+	ifd.open(path.c_str());
 	if (!ifd)
 	{
 		std::cerr << path << ": ifd.open fail" << '\n';
@@ -74,17 +56,6 @@ void	Webserv::_get_file(Client *client, const std::string &path)
 	}
 	else
 	{
-		/* std::streampos	size = ifd.tellg();
-		// WIP Low priority : Chunk or not chunk (if filesize too big)
-		if (size > MAX_FILESIZE)
-		{
-			// Then chunk
-			client->status = 500; // WIP temp
-			std::cerr << "File too large for non chunk body\n";
-			return ;
-		} */
-
-		ifd.seekg(0, std::ios::beg);
 		buf << ifd.rdbuf();
 		if (!ifd || !buf)
 		{
@@ -100,6 +71,19 @@ void	Webserv::_get_file(Client *client, const std::string &path)
 	}
 }
 
+/*
+// WIP Low priority : Chunk or not chunk (if filesize > MAX_FILESIZE)
+// WITH flag "std::ios::ate" for open()
+std::streampos	size = ifd.tellg();
+ifd.seekg(0, std::ios::beg);
+if (size > MAX_FILESIZE)
+{
+	// Chunked GET here
+	return ;
+}
+*/
+
+
 // const?
 void	Webserv::_autoindex(Client *client, const std::string &path)
 {
@@ -109,8 +93,6 @@ void	Webserv::_autoindex(Client *client, const std::string &path)
 	DIR				*dir;
 	struct dirent	*ent;
 
-	// std::cout << "location root: " << client->assigned_location->root << " location path: " << client->assigned_location->path << '\n';
-
 	// std::cout << "Path in auto is: " << path << '\n';
 	if ( (dir = opendir(path.c_str()) ) != NULL)
 	{
@@ -119,19 +101,21 @@ void	Webserv::_autoindex(Client *client, const std::string &path)
 		dir_list.append(AUTOINDEX_MID1);
 		dir_list.append(path);
 		dir_list.append(AUTOINDEX_MID2);
-	/* print all the files and directories within directory */
 		while ((ent = readdir (dir)) != NULL)
 		{
-			// std::cout << "ent: " << ent->d_name << '\n';
 			if (strcmp(".", ent->d_name) == 0)
 				continue ;
-			dir_list.append("<a href=\"");
-			dir_list.append(client->get_rq_abs_path() + "/");
+			dir_list.append("<a style=\"font-size:1.5em\" href=\"");
+			dir_list.append(client->get_rq_abs_path());
+			if (dir_list[dir_list.size() - 1] != '/')
+				dir_list.push_back('/');
 			dir_list.append(ent->d_name);
 			dir_list.append("\">");
 			dir_list.append(ent->d_name);
+			if (ent->d_type == DT_DIR)
+				dir_list.append("/");
 			dir_list.append("</a>");
-			dir_list.append("\r\n");	// is this right?
+			dir_list.append("\n");
 		}
 
 // <a href="http://nginx.org/">nginx.org</a>.<br/>
@@ -141,15 +125,22 @@ void	Webserv::_autoindex(Client *client, const std::string &path)
 // <a href="/test/test_deeper/..">..</a>
 
 		dir_list.append(AUTOINDEX_END);
-//		std::cout << "\n\n" << dir_list << '\n';
 		closedir (dir);
+		client->status = 200;
 		_append_body(client, dir_list, "html");
 	}
 	else
 	{
-		// in theory not possible cuz we already checked...
-		std::cerr << "could not open dir\n";
-		// throw?
+		client->status = 500;
+		perror("could not open dir");
 		return ;
 	}
 }
+
+std::string	Webserv::_determine_file_extension(const std::string &path) const
+{
+	size_t	dot_pos = path.rfind(".");
+	if (dot_pos != NPOS && dot_pos + 1 < path.size())
+		return ( path.substr(dot_pos + 1) );
+	return (std::string(""));
+}

srcs/webserv/method_post.cpp

@@ -5,54 +5,130 @@
 void	Webserv::_post(Client *client, const std::string &path)
 {
 /*
-	WIP
 	https://www.rfc-editor.org/rfc/rfc9110.html#name-post
 */
-	_post_file(client, path);
+	(void)path; // unused, we use "assigned_location->upload_dir" instead
+	std::cout << "_post()\n";
+	std::cerr << "upload_dir = " << client->assigned_location->upload_dir << "\n";
+
+
+	if (client->get_rq_abs_path() != client->assigned_location->path)
+		client->status = 404; // 404 ? J'ai un doute.
+	else if (client->assigned_location->upload_dir.empty())
+		client->status = 404; // 404 ? J'ai un doute.
+	else if (client->get_rq_multi_bodys().empty())
+	{
+		client->status = 415;
+		client->response.append("Accept: multipart/form-data"); // empty, no encoding accepted
+		client->response.append(CRLF);
+	}
+	else
+		_upload_files(client);
 }
 
-void	Webserv::_post_file(Client *client, const std::string &path)
+#define	DEFAULT_NAME "unnamed_file"
+// TODO : Loop for multi body
+void	Webserv::_upload_files(Client *client)
 {
+	std::cout << "_upload_files()\n";
 	std::ofstream	ofd;
+	std::vector<MultipartBody>::const_iterator	body_it = client->get_rq_multi_bodys().begin();
+	std::string	path;
+	std::string	filename;
+	size_t	pos;
+	bool	file_existed = false;
 
-	bool	file_existed;
-	if (access(path.c_str(), F_OK) == -1)
-		file_existed = false;
-	else
-		file_existed = true;
+	client->status = ::eval_file_access(client->assigned_location->upload_dir, W_OK);
+	if (client->status)
+		return;
 
-	// How to determine status 403 for file that dont already exist ?
-	if (file_existed && access(path.c_str(), W_OK) == -1)
+	while (body_it != client->get_rq_multi_bodys().end())
 	{
-		std::perror("err access()");
-		client->status = 403;
-		return ;
-	}
+		if (body_it->body.empty())
+		{
+			++body_it;
+			continue;
+		}
+		// Content-Disposition: form-data; name="upload_file"; filename="camion.jpg"
+		::print_map(body_it->headers);
+		filename = client->get_rq_multi_bodys_headers("Content-Disposition", body_it);
+		std::cerr << "filename ="<< filename << "\n";
+		pos = filename.find("filename=");
+		if (pos != NPOS)
+		{
+			filename = filename.substr(pos + sizeof("filename=")-1);
+			std::cerr << "filename ="<< filename << "\n";
+			// A l'arrache pour enlever les "
+			filename.erase(0, 1);
+			std::cerr << "filename ="<< filename << "\n";
+			filename.erase(filename.size()-1, 1);
+			std::cerr << "filename ="<< filename << "\n";
+			std::cerr << "filename ="<< filename << "\n";
+			if (filename.empty())
+				filename = DEFAULT_NAME;
+		}
+		else
+		{
+			filename = DEFAULT_NAME;
+		}
+		std::cerr << "filename ="<< filename << "\n";
+		path = client->assigned_location->upload_dir; // Assume there a final '/'
+		path.append(filename);
 
-	ofd.open(path.c_str(), std::ios::trunc);
-	if (!ofd)
-	{
-		std::cerr << path << ": ofd.open fail" << '\n';
-		client->status = 500;
-	}
-	else
-	{
-		// Content-Length useless at this point ?
-		ofd << client->get_rq_body();
+
+		if (::access(path.c_str(), F_OK) == -1)
+			file_existed = false;
+		else
+			file_existed = true;
+
+		// How to determine status 403 for file that dont already exist ? access() on the upload_dir ?
+		if (file_existed && ::access(path.c_str(), W_OK) == -1)
+		{
+			std::perror("err access()");
+			client->status = 403;
+			return ;
+		}
+
+		ofd.open(path.c_str(), std::ios::trunc);
+		if (!ofd)
+		{
+			std::cerr << path << ": ofd.open fail" << '\n';
+			client->status = 500;
+			return;
+		}
+		ofd << body_it->body;
 		if (!ofd)
 		{
 			std::cerr << path << ": ofd.write fail" << '\n';
 			client->status = 500;
+			return;
 		}
-		else if (file_existed)
-		{
-			client->status = 200;
-			// WIP https://www.rfc-editor.org/rfc/rfc9110.html#name-200-ok
-		}
-		else
-		{
-			client->status = 201;
-			// WIP https://www.rfc-editor.org/rfc/rfc9110.html#section-9.3.3-4
-		}
+		++body_it;
+		ofd.close();
 	}
+
+	client->status = 204;
+	client->response.append(CRLF);
+	// https://www.rfc-editor.org/rfc/rfc9110.html#name-204-no-content
 }
+
+/*
+	if (file_existed) // with multi body it doesn't make much sense
+	{
+		client->status = 200;
+		client->response.append("Location: ");
+		client->response.append("/index.html"); // WIP
+		client->response.append(CRLF);
+		client->response.append(CRLF);
+		// WIP https://www.rfc-editor.org/rfc/rfc9110.html#name-200-ok
+	}
+	else
+	{
+		client->status = 201;
+		client->response.append("Location: ");
+		client->response.append("/index.html"); // WIP
+		client->response.append(CRLF);
+		client->response.append(CRLF);
+		// WIP https://www.rfc-editor.org/rfc/rfc9110.html#section-9.3.3-4
+	}
+*/

srcs/webserv/parsing_request.cpp

@@ -1,2 +0,0 @@
-#include "parsing_request.hpp"
-

srcs/webserv/request.cpp

@@ -25,7 +25,7 @@ void	Webserv::_request(Client *client)
 	}
 	else if (ret == READ_COMPLETE)
 	{
-		if (client->body_complete)
+		if (client->body_complete && client->get_rq_multi_bodys().empty()) // DEBUG
 			std::cerr << "______BODY\n" << client->get_rq_body() << "\n______\n"; // DEBUG
 		_epoll_update(client->get_cl_fd(), EPOLLOUT, EPOLL_CTL_MOD);
 		client->request_complete = true;
@@ -53,8 +53,13 @@ int	Webserv::_read_request(Client *client)
 	// ::print_special(client->raw_request);
 	// std::cerr << "__raw_request__\n" << client->raw_request << "\n______\n"; // DEBUG
 
+	print_special(client->raw_request);
+
+	std::cerr << "client header complete: " << client->header_complete << "\n"; // DEBUG
+
 	if (!client->header_complete)
 	{
+		std::cout << "Header not complete\n"; // debug
 		client->parse_request_headers(_servers);
 		if (client->status)
 			return READ_COMPLETE;
@@ -73,6 +78,7 @@ int	Webserv::_read_request(Client *client)
 	}
 	if (client->header_complete)
 	{
+		std::cerr << "Client header Complete\n";
 		// client->read_body_size += ret; // Not accurate, part of body could have been read with headers, unused for now
 		client->parse_request_body();
 		if (client->status || client->body_complete)

srcs/webserv/response.cpp

@@ -3,7 +3,7 @@
 
 enum send_return
 {
-	SEND_IN_PROGRESS, // unused
+	SEND_IN_PROGRESS,
 	SEND_COMPLETE,
 	SEND_CLOSE,
 };
@@ -21,7 +21,10 @@ void	Webserv::_response(Client *client)
 	}
 	else if (ret == SEND_COMPLETE)
 	{
-		if (client->get_rq_headers("Connection") == "close" || client->status == 408)
+		if (client->get_rq_headers("Connection") == "close"
+		|| client->status == 400 // TODO: Refactoring
+		|| client->status == 408
+		|| client->status == 413)
 			_close_client(client->get_cl_fd());
 		else
 		{
@@ -37,16 +40,33 @@ int	Webserv::_send_response(Client *client)
 
 	std::cerr << "send()\n";
 
-	_append_base_headers(client);
-	if (!client->status)
-		_construct_response(client);
+	if (client->response.empty())
+	{
+		_append_base_headers(client);
+		if (!client->status)
+		{
+			_construct_response(client);
+			if (client->cgi_pipe_rfd)
+				return SEND_IN_PROGRESS;
+		}
+	}
+	else if (!client->cgi_output.empty())
+	{
+		// /*DEBUG*/ std::cout << "\n" B_PURPLE "[response]:" RESET "\n"; ::print_special(client->response); std::cout << B_PURPLE "-----------" RESET "\n\n";
+		// /*DEBUG*/ std::cout << "\n" B_PURPLE "[script output]:" RESET "\n"; ::print_special(script_output); std::cout << B_PURPLE "-----------" RESET "\n\n";
+		_check_script_output(client, client->cgi_output); // FD_CGI : adjust for client->cgi_output;
+		if (client->status < 400)
+			client->response += client->cgi_output;
+		// /*DEBUG*/ std::cout << "\n" B_PURPLE "[response + output]:" RESET "\n"; ::print_special(client->response); std::cout << B_PURPLE "-----------" RESET "\n\n";		
+	}
+
 	_insert_status_line(client);
 	if (client->status >= 400)
 		_error_html_response(client);
 
-/*DEBUG*/ std::cout << "\n" B_PURPLE "[response + output + headers]:" RESET "\n"; ::print_special(client->response); std::cout << B_PURPLE "-----------" RESET "\n\n";
+/*DEBUG*/ std::cout << "\n" B_PURPLE "[response + output + headers]:" RESET "\n"; ::print_special(client->response); std::cout << "\n" B_PURPLE "-----------" RESET "\n\n";
 
-	std::cerr << "client->response.size() = " << client->response.size() << "\n"; // DEBUG
+	// /* Debug */ std::cerr << "client->response.size() = " << client->response.size() << "\n"; // DEBUG
 	ret = ::send(client->get_cl_fd(), client->response.c_str(), client->response.size(), 0);
 	if (ret == -1)
 	{
@@ -54,7 +74,12 @@ int	Webserv::_send_response(Client *client)
 		std::cerr << "client.fd =" << client->get_cl_fd() << "\n"; // DEBUG
 		return SEND_CLOSE;
 	}
-	std::cerr << "ret send() = " << ret << "\n"; // DEBUG
+	if (ret == 0) // actually never happen ?
+	{
+		std::cerr << "SEND RET 0 for client.fd =" << client->get_cl_fd() << "\n"; // DEBUG
+		return SEND_CLOSE;
+	}
+	// /* Debug */ std::cerr << "ret send() = " << ret << "\n"; // DEBUG
 
 	return SEND_COMPLETE;
 }
@@ -63,13 +88,15 @@ void	Webserv::_append_base_headers(Client *client)
 {
 	client->response.append("Server: Webserv/0.1" CRLF);
 
-	if (client->get_rq_headers("Connection") == "close")
+	if (client->get_rq_headers("Connection") == "close"
+	|| client->status == 400 // TODO: Refactoring
+	|| client->status == 408
+	|| client->status == 413)
 		client->response.append("Connection: close" CRLF);
 	else
 		client->response.append("Connection: keep-alive" CRLF);
 }
 
-// TODO HUGO : wip
 void	Webserv::_construct_response(Client *client)
 {
 	std::string	path;
@@ -78,16 +105,7 @@ void	Webserv::_construct_response(Client *client)
 	path = _replace_url_root(client, client->get_rq_abs_path());
 	if (_is_cgi(client, path))
 	{
-		script_output = _exec_cgi(client);
-
-/*DEBUG*/ std::cout << "\n" B_PURPLE "[script output]:" RESET "\n"; ::print_special(script_output); std::cout << B_PURPLE "-----------" RESET "\n\n";
-/*DEBUG*/ std::cout << "\n" B_PURPLE "[response]:" RESET "\n"; ::print_special(client->response); std::cout << B_PURPLE "-----------" RESET "\n\n";
-
-		_check_script_output(client, script_output);
-		client->response += script_output;
-
-/*DEBUG*/ std::cout << "\n" B_PURPLE "[response + output]:" RESET "\n"; ::print_special(client->response); std::cout << B_PURPLE "-----------" RESET "\n\n";
-
+		_exec_cgi(client);
 		return;
 	}
 	_process_method(client, path);
@@ -95,7 +113,8 @@ void	Webserv::_construct_response(Client *client)
 
 void	Webserv::_process_method(Client *client, std::string &path)
 {
-	std::cerr << "allow_methods = " << http_methods_to_str(client->assigned_location->allow_methods) << "\n"; // debug
+//	std::cerr << "allow_methods = " << http_methods_to_str(client->assigned_location->allow_methods) << "\n"; // debug
+	std::cerr << "Path again:			" << path << '\n'; // debug
 
 	switch (client->get_rq_method())
 	{
@@ -110,26 +129,53 @@ void	Webserv::_process_method(Client *client, std::string &path)
 	}
 }
 
+std::string	Webserv::_replace_url_root(Client *client, std::string path)
+{
+	// /* Debug */ std::cerr << "assigned_location->path =	" << client->assigned_location->path << "\n"; // debug
+	// /* Debug */ std::cerr << "path before =			" << path << "\n"; // DEBUG
+	if (client->assigned_location->path == "/")
+		path.insert(0, client->assigned_location->root);
+	else
+		path.replace(0, client->assigned_location->path.size(), client->assigned_location->root);
+	// /* Debug */ std::cerr << "path after =			" << path << "\n"; // DEBUG
+	return path;
+}
+
+/*
+	https://www.rfc-editor.org/rfc/rfc9112.html#name-status-line
+*/
 void	Webserv::_insert_status_line(Client *client)
 {
 	std::string	status_line;
+	std::string	status = _http_status[client->status];
+	if (status.empty())
+		status = ::itos(client->status);
 
 	status_line.append("HTTP/1.1 ");
-	status_line.append(_http_status[client->status]);
+	status_line.append(status);
 	status_line.append(CRLF);
 	client->response.insert(0, status_line);
 }
 
 void	Webserv::_error_html_response(Client *client)
 {
-	if (!client->assigned_server || client->assigned_server->error_pages[client->status].empty())
+	std::cout << "_error_html_response()\n";
+
+	if (client->assigned_server
+	&& !client->assigned_server->error_pages[client->status].empty()
+	&& ::eval_file_access(client->assigned_server->error_pages[client->status], R_OK) == 0 )
 	{
-		std::string	html_page = HTML_ERROR;
-		::replace_all_substr(html_page, STATUS_PLACEHOLDER, _http_status[client->status]);
-		_append_body(client, html_page, "html");
+		_get_file(client, client->assigned_server->error_pages[client->status]);
 	}
 	else
-		_get_file(client, client->assigned_server->error_pages[client->status]);
+	{
+		std::string	status = _http_status[client->status];
+		if (status.empty())
+			status = "Error " + ::itos(client->status);
+		std::string	html_page = HTML_ERROR;
+		::replace_all_substr(html_page, STATUS_PLACEHOLDER, status);
+		_append_body(client, html_page, "html");
+	}
 }
 
 void	Webserv::_append_body(Client *client, const std::string &body, const std::string &file_extension)
@@ -155,105 +201,3 @@ void	Webserv::_append_body(Client *client, const std::string &body, const std::s
 	client->response.append(CRLF);
 	client->response.append(body);
 }
-
-// Temporary Global Scope. Probably move to Client in the future.
-ServerConfig	*_determine_process_server(Client *client, std::vector<ServerConfig> &servers)
-{
-/*
-	http://nginx.org/en/docs/http/request_processing.html
-	_determine_process_server() should be complete.
-	TODO : test it
-*/
-
-	std::string	const &server_name = client->get_rq_headers("Host");
-	std::vector<ServerConfig>::iterator	it = servers.begin();
-	std::vector<ServerConfig>::iterator	default_server = servers.end();
-
-	while (it != servers.end())
-	{
-		if (it->host == client->get_cl_lsocket()->host
-		&&	it->port == client->get_cl_lsocket()->port)
-		{
-			if ( std::find(it->server_name.begin(), it->server_name.end(), server_name) != it->server_name.end() )
-				break;
-			else if (default_server == servers.end())
-				default_server = it;
-		}
-		++it;
-	}
-	if (it != servers.end())
-		return (&(*it));
-	else
-		return (&(*default_server));
-}
-
-// const?
-// Temporary Global Scope. Probably move to Client in the future.
-const LocationConfig	*_determine_location(const ServerConfig &server, const std::string &path)
-{
-/* RULES ***
-
-If a path coresponds exactly to a location, use that one
-if no path coresponds then use the most correct one
-	most correct means the most precise branch that is still above
-	the point we are aiming for
-
-New Rule for location paths, they never end in /
-Sooo
-If we get a url that ends in / ignore the last /
-
-
-*/
-
-	std::string	uri = path;
-	if (uri[uri.size() - 1] == '/')
-		uri.erase(uri.size() - 1);
-
-
-	for (std::vector<LocationConfig>::const_iterator it = server.locations.begin(); it != server.locations.end(); it++)
-	{
-		if (it->path.size() > uri.size())
-			continue ;
-
-		if (uri.compare(0, it->path.size(), it->path) == 0)
-		{
-			if (it->path.size() == uri.size())
-				return (&(*it));
-			else if (uri[it->path.size()] == '/')
-				return (&(*it));
-	// this works cuz only ever looking for a / burried in a longer path
-		}
-	}
-	return (&(server.locations.back()));
-
-
-//	/test/mdr
-//	/test/mdr/
-//	/test/mdrBST
-
-/* More stuff to check this still works with ***
-
-/test/test_
-/test/test_/
-/test/test_deeper
-/test/test_deeper/
-/test/test_deepei
-/test/test_deepei/
-/test/test_deeperi
-/test/test_deeper/super_deep/
-/test/aaaaaaaaaaa/super_deep/
-
-*/
-
-}
-
-
-
-
-std::string	Webserv::_determine_file_extension(const std::string &path) const
-{
-	size_t	dot_pos = path.rfind(".");
-	if (dot_pos != NPOS && dot_pos + 1 < path.size())
-		return ( path.substr(dot_pos + 1) );
-	return (std::string(""));
-}

srcs/webserv/run_loop.cpp

@@ -4,6 +4,98 @@
 #define MAX_EVENTS 42 // arbitrary
 #define TIMEOUT 3000
 
+
+#define BUFSIZE 8192 // (8Ko)
+#define STATUS_500 std::string("Status: 500" CRLF CRLF);
+void	Webserv::_read_cgi_output(Client *client)
+{
+	char	buf[BUFSIZE];
+	ssize_t	ret;
+	std::cerr << "_read_cgi_output()" << "\n";
+	std::cerr << "cgi_pid = " << client->cgi_pid << "\n";
+	std::cerr << "client fd = " << client->get_cl_fd() << "\n";
+	std::cerr << "cgi fd = " << client->cgi_pipe_rfd << "\n";
+
+	ret = ::read(client->cgi_pipe_rfd, buf, BUFSIZE);
+	std::cerr << "cgi read ret = " << ret << "\n";
+	if (ret == -1)
+	{
+		std::perror("err read(cgi_fd)");
+		client->cgi_output = STATUS_500;
+	}
+	else if (ret == 0)
+		std::cerr << "Madame s'il vous plait, du Ketchup pour mon hamburger" << " (AKA:ret=0)" << "\n";
+	else
+	{
+		std::cerr << "NORMAL BEHAVIOR I THINK!\n"; // debug
+		client->cgi_output.append(buf, ret);
+	}
+}
+
+void	Webserv::_handle_epoll_error_cgi_fd(uint32_t events, Client *client)
+{
+	(void)events;
+	std::cerr << "cgi EPOLLERR" << "\n";
+	std::cerr << "cgi_pid = " << client->cgi_pid << "\n";
+	std::cerr << "client fd = " << client->get_cl_fd() << "\n";
+	std::cerr << "cgi fd = " << client->cgi_pipe_rfd << "\n";
+
+	client->cgi_output = STATUS_500;
+
+	pid_t	wait_ret;
+	wait_ret = ::waitpid(client->cgi_pid, NULL, WNOHANG);
+	std::cerr << "cgi EPOLLERR waitpid ret = " << wait_ret << "\n";
+	if (wait_ret == client->cgi_pid)
+	{
+		if (client->cgi_pipe_rfd)
+		{
+			if (::close(client->cgi_pipe_rfd) == -1)
+				std::perror("err close()");
+		}
+		client->cgi_pipe_rfd = 0;
+		_epoll_update(client->get_cl_fd(), EPOLLOUT, EPOLL_CTL_ADD);
+	}
+
+}
+
+void	Webserv::_cgi_epollhup(uint32_t events, Client *client)
+{
+	(void)events;
+	(void)client;
+
+/* 	std::cerr << "cgi EPOLLHUP" << "\n";
+	std::cerr << "cgi_pid = " << client->cgi_pid << "\n";
+	std::cerr << "client fd = " << client->get_cl_fd() << "\n";
+	std::cerr << "cgi fd = " << client->cgi_pipe_rfd << "\n"; */
+
+	pid_t	wait_ret;
+	wait_ret = ::waitpid(client->cgi_pid, NULL, WNOHANG);
+	// std::cerr << "cgi EPOLLHUP waitpid ret = " << wait_ret << "\n";
+	if (wait_ret == client->cgi_pid)
+	{
+		if (client->cgi_pipe_rfd)
+		{
+			if (::close(client->cgi_pipe_rfd) == -1)
+				std::perror("err close()");
+		}
+		client->cgi_pipe_rfd = 0;
+		_epoll_update(client->get_cl_fd(), EPOLLOUT, EPOLL_CTL_ADD);
+	}
+}
+
+Client	*Webserv::_find_cgi_fd(int cgi_fd)
+{
+	std::vector<Client>::iterator	it = _clients.begin();
+	std::vector<Client>::iterator	it_end = _clients.end();
+	while (it != it_end)
+	{
+		if (it->cgi_pipe_rfd == cgi_fd)
+			return (&(*it));
+		++it;
+	}
+	return (NULL);
+}
+
 void	Webserv::run()
 {
 	std::cerr << "Server started\n";
@@ -11,13 +103,15 @@ void	Webserv::run()
 	int	nfds;
 	int	i;
 	int	count_loop = 0;
-	std::vector<listen_socket>::iterator	it_socket;
+	std::vector<listen_socket>::iterator	it_lsocket;
+	Client	*client_cgi = NULL;
 
 	g_run = true;
 	while (g_run)
 	{
-		std::cerr << ++count_loop << "----loop epoll()\n";
+		std::cerr << ++count_loop << "----loop epoll() ";
 		nfds = ::epoll_wait(_epfd, events, MAX_EVENTS, TIMEOUT);
+		std::cerr << "(nfds=" << nfds << ")\n";
 		if (nfds == -1)
 		{
 			int errno_copy = errno;
@@ -33,15 +127,25 @@ void	Webserv::run()
 		while (i < nfds)
 		{
 			try {
-				it_socket = std::find(_listen_sockets.begin(), _listen_sockets.end(), events[i].data.fd);
-				if (it_socket != _listen_sockets.end())
+				it_lsocket = std::find(_listen_sockets.begin(), _listen_sockets.end(), events[i].data.fd);
+				client_cgi = _find_cgi_fd(events[i].data.fd);
+				if (it_lsocket != _listen_sockets.end())
 				{
 					if (events[i].events & EPOLLERR || events[i].events & EPOLLHUP)
-						_handle_epoll_error_lsocket(events[i].events, it_socket);
+						_handle_epoll_error_lsocket(events[i].events, it_lsocket);
 					else if (events[i].events & EPOLLIN)
-						_accept_connection(*it_socket);
+						_accept_connection(*it_lsocket);
 				}
-				else
+				else if (client_cgi)
+				{
+					if (events[i].events & EPOLLERR)
+						_handle_epoll_error_cgi_fd(events[i].events, client_cgi);
+					else if (events[i].events & EPOLLIN)
+						_read_cgi_output(client_cgi);
+					else if ( (events[i].events & EPOLLHUP) && !(events[i].events & EPOLLIN) )
+						_cgi_epollhup(events[i].events, client_cgi);
+				}
+				else if (std::find(_clients.begin(), _clients.end(), events[i].data.fd) != _clients.end()) // TODO: save the it in var to avoid multiples find()
 				{
 					if (events[i].events & EPOLLERR || events[i].events & EPOLLHUP)
 						_handle_epoll_error_client(events[i].events, events[i].data.fd);
@@ -62,6 +166,11 @@ void	Webserv::run()
 				std::vector<Client>().swap(_clients);
 				break;
 			}
+			catch (const Webserv::ExecFail& e)
+			{
+				std::cerr << e.what() << '\n';
+				throw;
+			}
 			catch (const std::exception& e) {
 				std::cerr << e.what() << '\n';
 				++i;

srcs/webserv/timeout.cpp

@@ -12,7 +12,12 @@ void	Webserv::_timeout()
 			std::cerr << "timeout request fd " << it->get_cl_fd() << "\n";
 			it->status = 408;
 			_epoll_update(it->get_cl_fd(), EPOLLOUT, EPOLL_CTL_MOD);
+
+			// DEBUG, close without repsonse 408
+			/* _close_client(it->get_cl_fd());
+			it = _clients.begin(); */
 		}
+		// else // DEBUG
 		++it;
 	}
 }

test_file_upload.txt

@@ -0,0 +1,6 @@
+START
+http://localhost:4040/test
+http://localhost:4040/test/test_deeper/
+http://localhost:4040/test/test_deeper/super_deep/
+http://localhost:4040/test/index1.html
+STOP