Skip to content

Commit 5462a73

Browse files
committed
fix: add logs when processing TCP stream
1 parent cb399bb commit 5462a73

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

src/http/request_raw.rs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
use anyhow::Result;
2+
use log::trace;
23
use std::{
34
io::{BufRead, BufReader, Read},
45
net::TcpStream,
@@ -14,15 +15,18 @@ pub struct HttpRequestRaw {
1415

1516
impl HttpRequestRaw {
1617
pub fn from_tcp(stream: &TcpStream) -> Result<HttpRequestRaw> {
18+
trace!("trying to convert TCP message into HTTP request");
1719
let mut buf_reader = BufReader::new(stream);
1820

1921
let mut request_line = String::new();
2022
let mut headers = Vec::new();
2123
let mut body = Vec::new();
2224

25+
trace!("read request line");
2326
buf_reader.read_line(&mut request_line)?;
2427

2528
let mut line = String::new();
29+
trace!("proceed to read read headers");
2630
while buf_reader.read_line(&mut line)? > 0 {
2731
if line.trim().is_empty() {
2832
break;
@@ -43,13 +47,16 @@ impl HttpRequestRaw {
4347
.iter()
4448
.find(|header| header.name == "Content-Length")
4549
{
50+
trace!("found Content-Length header, using value to read body");
4651
let content_len: usize = content_len.value.parse()?;
4752
if content_len > 0 {
53+
trace!("read body ({} bytes)", content_len);
4854
body = vec![0; content_len];
4955
buf_reader.read_exact(&mut body)?;
5056
}
5157
}
5258

59+
trace!("finish processing TCP stream");
5360
Ok(HttpRequestRaw {
5461
request_line,
5562
headers,

src/web_server.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,7 @@ impl WebServer {
4646
self.pool.execute(move || {
4747
let result = handle_connection(router_clone, stream);
4848
if let Err(result) = result {
49-
let error = format!("error: {}", result);
50-
error!("{}", error);
49+
error!("handle_connection failed: {}", result);
5150
}
5251
});
5352
}

0 commit comments

Comments
 (0)