fix(server): do not automatically set ContentLength for 204 and 304 R…

…esponses
hyperium · Jul 4, 2017 · c4c89a2 · c4c89a2
1 parent 81c0d18
commit c4c89a2
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/http/h1/parse.rs b/src/http/h1/parse.rs
@@ -131,9 +131,16 @@ impl Http1Transaction for ServerTransaction {
  body
  }
 
- fn should_set_length(_head: &MessageHead<Self::Outgoing>) -> bool {
+ fn should_set_length(head: &MessageHead<Self::Outgoing>) -> bool {
  //TODO: pass method, check if method == HEAD
- true
+
+ match head.subject {
+ // TODO: support for 1xx codes needs improvement everywhere
+ // would be 100...199 => false
+ StatusCode::NoContent |
+ StatusCode::NotModified => false,
+ _ => true,
+ }
  }
 }