LeaYeh / webserver

MIT License
3 stars 0 forks source link

[FEAT] Add implementation of connection handler #40

Closed LeaYeh closed 2 months ago

LeaYeh commented 2 months ago

@yenthing check the webshell interface and naming convention in commit https://github.com/LeaYeh/webserver/pull/40/commits/e9f8efc9d9152d7839ccfbb9b92622b5f233cafb

LeaYeh commented 2 months ago

Now we have dummy behavior 😎👌

But we need to figure out what is GET /favicon.ico HTTP/1.1 and how to re-connect after timeout correctly.

...
2024-08-29 20:11:13 [DEBUG] Reactor received 1 events                                                                           
2024-08-29 20:11:13 [DEBUG] Read 711 bytes from fd: 7                                                                           
2024-08-29 20:11:13 [DEBUG] Buffer: 
GET / HTTP/1.1
Host: 127.0.0.1:8888
Connection: keep-alive
Cache-Control: max-age=0
sec-ch-ua: "Not)A;Brand";v="99", "Google Chrome";v="127", "Chromium";v="127"
sec-ch-ua-mobile: ?0
sec-ch-ua-platform: "Linux"
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/127.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.7
Sec-Fetch-Site: none
Sec-Fetch-Mode: navigate
Sec-Fetch-User: ?1
Sec-Fetch-Dest: document
Accept-Encoding: gzip, deflate, br, zstd
Accept-Language: zh-TW,zh;q=0.9,en-US;q=0.8,en;q=0.7,zh-CN;q=0.6

2024-08-29 20:11:13 [INFO] Request is complete                                                                                 
2024-08-29 20:11:13 [DEBUG] Processing request                                                                                  
2024-08-29 20:11:13 [DEBUG] Responding to request                                                                               
2024-08-29 20:11:13 [DEBUG] Elapsed time: 189                                                                                   
2024-08-29 20:11:13 [INFO] Keep-alive timeout or close request                                                                 
2024-08-29 20:11:13 [DEBUG] Kernel::~Kernel()                                                                                   
2024-08-29 20:11:13 [DEBUG] Reactor::~Reactor()                                                                                 
2024-08-29 20:11:13 [DEBUG] Acceptor destroyed with server_id: 0                                                                
2024-08-29 20:11:13 [DEBUG] ConnectionHandler destroyed with server_id: 0                                                       
2024-08-29 20:11:13 [INFO] Connection closed by server                                                                         
terminate called after throwing an instance of 'std::runtime_error'
  what():  epoll_ctl failed
[1]    3029197 IOT instruction (core dumped)  ./webserv configs/default.conf
yenthing commented 2 months ago

Can you please add Chore into [verb] for commit message, because its in the collaborate docs but not in the test. Upsi

yenthing commented 2 months ago

Can you please add Chore into [verb] for commit message, because its in the collaborate docs but not in the test. Upsi

[FIXED] We did it! Muahahahaha