Skip to content

Commit 4687324

Browse files
committed
refactor: rename read_header to find_message_head_end
1 parent bc3d203 commit 4687324

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

body-plz/src/reader/chunked_reader.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ use crate::body_struct::ChunkedBody;
99
use super::content_length_reader::read_content_length;
1010
use header_plz::abnf::CRLF;
1111
use header_plz::header_map::HeaderMap;
12-
use header_plz::reader::read_header;
12+
use header_plz::reader::find_message_head_end;
1313

1414
// Enum to represent chunked reader errors
1515
#[cfg_attr(any(debug_assertions, test), derive(Eq, PartialEq))]
@@ -124,7 +124,7 @@ impl ChunkReader {
124124
return Some(ChunkedBody::EndCRLF(buf.split_at_current_pos()));
125125
}
126126
// 4.b. Actual Headers
127-
if read_header(buf) {
127+
if find_message_head_end(buf) {
128128
*self = Self::End;
129129
let header_map = HeaderMap::new(buf.split_at_current_pos());
130130
Some(ChunkedBody::Trailers(header_map))

header-plz/src/reader.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ use crate::abnf::HEADER_DELIMITER;
1010
* false.
1111
*/
1212

13-
pub fn read_header(buf: &mut Cursor) -> bool {
13+
pub fn find_message_head_end(buf: &mut Cursor) -> bool {
1414
// 1. Find HEADER_DELIMITER
1515
if let Some(index) = buf
1616
.as_ref()
@@ -41,7 +41,7 @@ mod tests {
4141
let mut buf = BytesMut::from(req);
4242
let mut cur = Cursor::new(&mut buf);
4343
let verify = BytesMut::from(req);
44-
let status: bool = read_header(&mut cur);
44+
let status: bool = find_message_head_end(&mut cur);
4545
assert!(status);
4646
assert_eq!(cur.position(), verify.len());
4747
}
@@ -52,7 +52,7 @@ mod tests {
5252
Host: reqbin.com\r\n";
5353
let mut buf = BytesMut::from(req);
5454
let mut cur = Cursor::new(&mut buf);
55-
let status: bool = read_header(&mut cur);
55+
let status: bool = find_message_head_end(&mut cur);
5656
assert!(!status);
5757
assert_eq!(cur.position(), req.len() - 3);
5858
}
@@ -62,17 +62,17 @@ mod tests {
6262
let req = "GET /echo HTTP/1.1\r\n";
6363
let mut buf = BytesMut::from(req);
6464
let mut cur = Cursor::new(&mut buf);
65-
let mut status = read_header(&mut cur);
65+
let mut status = find_message_head_end(&mut cur);
6666
assert!(!status);
6767
assert_eq!(cur.position(), req.len() - 3);
6868
let toadd = b"Host: reqbin.com\r";
6969
cur.as_mut().extend_from_slice(toadd);
70-
status = read_header(&mut cur);
70+
status = find_message_head_end(&mut cur);
7171
assert!(!status);
7272
assert_eq!(cur.position(), cur.as_ref().len() - 3);
7373
let toadd = b"\n\r\n";
7474
cur.as_mut().extend_from_slice(toadd);
75-
status = read_header(&mut cur);
75+
status = find_message_head_end(&mut cur);
7676
assert!(status);
7777
assert_eq!(cur.position(), cur.as_ref().len());
7878
}

0 commit comments

Comments
 (0)