Skip to content

Commit e48519a

Browse files
committed
refactor(http1): adjust a few tracing event outputs
1 parent 55a61c0 commit e48519a

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

src/proto/h1/io.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,7 @@ where
223223
match Pin::new(&mut self.io).poll_read(cx, &mut buf) {
224224
Poll::Ready(Ok(_)) => {
225225
let n = buf.filled().len();
226+
trace!("received {} bytes", n);
226227
unsafe {
227228
// Safety: we just read that many bytes into the
228229
// uninitialized part of the buffer, so this is okay.

src/proto/h1/role.rs

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -120,11 +120,7 @@ impl Http1Transaction for Server {
120120
/* SAFETY: it is safe to go from MaybeUninit array to array of MaybeUninit */
121121
let mut headers: [MaybeUninit<httparse::Header<'_>>; MAX_HEADERS] =
122122
unsafe { MaybeUninit::uninit().assume_init() };
123-
trace!(
124-
"Request.parse([Header; {}], [u8; {}])",
125-
headers.len(),
126-
buf.len()
127-
);
123+
trace!(bytes = buf.len(), "Request.parse");
128124
let mut req = httparse::Request::new(&mut []);
129125
let bytes = buf.as_ref();
130126
match req.parse_with_uninit_headers(bytes, &mut headers) {
@@ -144,7 +140,6 @@ impl Http1Transaction for Server {
144140
is_http_11 = false;
145141
Version::HTTP_10
146142
};
147-
trace!("headers: {:?}", &req.headers);
148143

149144
record_header_indices(bytes, &req.headers, &mut headers_indices)?;
150145
headers_len = req.headers.len();
@@ -879,11 +874,7 @@ impl Http1Transaction for Client {
879874
// SAFETY: We can go safely from MaybeUninit array to array of MaybeUninit
880875
let mut headers: [MaybeUninit<httparse::Header<'_>>; MAX_HEADERS] =
881876
unsafe { MaybeUninit::uninit().assume_init() };
882-
trace!(
883-
"Response.parse([Header; {}], [u8; {}])",
884-
headers.len(),
885-
buf.len()
886-
);
877+
trace!(bytes = buf.len(), "Response.parse");
887878
let mut res = httparse::Response::new(&mut []);
888879
let bytes = buf.as_ref();
889880
match ctx.h1_parser_config.parse_response_with_uninit_headers(

0 commit comments

Comments
 (0)