Make :status in requests be a stream error

tests/h2-support/src/frames.rs

@@ -2,7 +2,7 @@ use std::convert::TryInto;
 use std::fmt;
 
 use bytes::Bytes;
-use http::{self, HeaderMap};
+use http::{self, HeaderMap, StatusCode};
 
 use h2::frame::{self, Frame, StreamId};
 
@@ -162,6 +162,14 @@ impl Mock<frame::Headers> {
         Mock(frame)
     }
 
+    pub fn status(self, value: StatusCode) -> Self {
+        let (id, mut pseudo, fields) = self.into_parts();
+
+        pseudo.set_status(value);
+
+        Mock(frame::Headers::new(id, pseudo, fields))
+    }
+
     pub fn scheme(self, value: &str) -> Self {
         let (id, mut pseudo, fields) = self.into_parts();
         let value = value.parse().unwrap();

tests/h2-tests/tests/server.rs

@@ -1025,6 +1025,30 @@ async fn server_error_on_unclean_shutdown() {
     srv.await.expect_err("should error");
 }
 
+#[tokio::test]
+async fn server_error_on_status_in_request() {
+    h2_support::trace_init!();
+
+    let (io, mut client) = mock::new();
+
+    let client = async move {
+        let settings = client.assert_server_handshake().await;
+        assert_default_settings!(settings);
+        client
+            .send_frame(frames::headers(1).status(StatusCode::OK))
+            .await;
+        client.recv_frame(frames::reset(1).protocol_error()).await;
+    };
+
+    let srv = async move {
+        let mut srv = server::handshake(io).await.expect("handshake");
+
+        assert!(srv.next().await.is_none());
+    };
+
+    join(client, srv).await;
+}
+
 #[tokio::test]
 async fn request_without_authority() {
     h2_support::trace_init!();