From a71627e61ef7a44a6ed6e6724caaa221447c5fa2 Mon Sep 17 00:00:00 2001 From: Rob Ede Date: Thu, 18 Nov 2021 16:40:54 +0000 Subject: [PATCH] update changelog --- actix-files/CHANGES.md | 4 ++-- actix-http/tests/test_server.rs | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/actix-files/CHANGES.md b/actix-files/CHANGES.md index 23accca40..41336c21c 100644 --- a/actix-files/CHANGES.md +++ b/actix-files/CHANGES.md @@ -1,9 +1,9 @@ # Changes ## Unreleased - 2021-xx-xx -* Fix 304 Not Modified responses to omit the Content-Length header, as per the spec. [#????] +* Fix 304 Not Modified responses to omit the Content-Length header, as per the spec. [#2453] -[#????]: https://github.com/actix/actix-web/pull/???? +[#2453]: https://github.com/actix/actix-web/pull/2453 ## 0.6.0-beta.8 - 2021-10-20 diff --git a/actix-http/tests/test_server.rs b/actix-http/tests/test_server.rs index eb8348080..11bc8e939 100644 --- a/actix-http/tests/test_server.rs +++ b/actix-http/tests/test_server.rs @@ -838,8 +838,8 @@ async fn test_not_modified_spec_h1() { res.headers().get(&CL), Some(&header::HeaderValue::from_static("4")), ); - // server does not prevent payload from being sent but clients will likely choose not to read it - // TODO: this is probably a bug + // server does not prevent payload from being sent but clients may choose not to read it + // TODO: this is probably a bug, especially since CL header can differ in length from the body assert!(!srv.load_body(res).await.unwrap().is_empty()); // TODO: add stream response tests