Always send Content-Length, even if body is None

src/dialog/dialog.rs

@@ -361,9 +361,9 @@ impl DialogInner {
         }
         headers.push(Header::MaxForwards(70.into()));
 
-        body.as_ref().map(|b| {
-            headers.push(Header::ContentLength((b.len() as u32).into()));
-        });
+        headers.push(Header::ContentLength(
+            body.as_ref().map_or(0u32, |b| b.len() as u32).into(),
+        ));
 
         let req = rsip::Request {
             method,
@@ -454,9 +454,9 @@ impl DialogInner {
             .as_ref()
             .map(|c| resp_headers.push(Contact::from(c.clone()).into()));
 
-        body.as_ref().map(|b| {
-            resp_headers.push(Header::ContentLength((b.len() as u32).into()));
-        });
+        resp_headers.push(Header::ContentLength(
+            body.as_ref().map_or(0u32, |b| b.len() as u32).into(),
+        ));
 
         resp_headers.push(Header::UserAgent(
             self.endpoint_inner.user_agent.clone().into(),

src/transaction/message.rs

@@ -226,10 +226,12 @@ impl EndpointInner {
                     | Header::CallId(_)
                     | Header::From(_)
                     | Header::To(_)
-                    | Header::MaxForwards(_)
                     | Header::CSeq(_)
             )
         });
+        headers.push(Header::ContentLength(
+            body.as_ref().map_or(0u32, |b| b.len() as u32).into(),
+        ));
         headers.unique_push(Header::UserAgent(self.user_agent.clone().into()));
         Response {
             status_code,

src/transport/tests/test_stream_encoding.rs

@@ -319,12 +319,12 @@ fn test_sip_codec_multiple_messages_with_bodies() {
             assert_eq!(req.call_id_header().unwrap().value(), "test-call-id-2");
         }
         _ => panic!("Expected request message"),
-    } 
+    }
 
     // Buffer should be empty after consuming both messages
     assert_eq!(
         buffer.len(),
         0,
         "Buffer should be empty after consuming all messages"
     );
-}
+}