Because `send_reset` called `recv_err`, which calls `reclaim_all_capacity`, which eventually calls `transition(stream, ..)` -- all of which happens _before_ the RESET frame is enqueued -- it was possible for the stream to get unlinked from the store (if there was any connection-level capacity to reassign). This could then cause the stream to get "leaked" on drop/EOF since it would no longer be iterated. Fix this by delaying the call to `reclaim_all_capacity` _after_ enqueueing the RESET frame. A test demonstrating the issue is included.
		
			
				
	
	
		
			1153 lines
		
	
	
		
			36 KiB
		
	
	
	
		
			Rust
		
	
	
	
	
	
			
		
		
	
	
			1153 lines
		
	
	
		
			36 KiB
		
	
	
	
		
			Rust
		
	
	
	
	
	
| #[macro_use]
 | |
| extern crate log;
 | |
| extern crate h2_support;
 | |
| 
 | |
| use h2_support::prelude::*;
 | |
| 
 | |
| #[test]
 | |
| fn handshake() {
 | |
|     let _ = ::env_logger::try_init();
 | |
| 
 | |
|     let mock = mock_io::Builder::new()
 | |
|         .handshake()
 | |
|         .write(SETTINGS_ACK)
 | |
|         .build();
 | |
| 
 | |
|     let (_client, h2) = client::handshake(mock).wait().unwrap();
 | |
| 
 | |
|     trace!("hands have been shook");
 | |
| 
 | |
|     // At this point, the connection should be closed
 | |
|     h2.wait().unwrap();
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn client_other_thread() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://http2.akamai.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .close();
 | |
| 
 | |
|     let h2 = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             ::std::thread::spawn(move || {
 | |
|                 let request = Request::builder()
 | |
|                     .uri("https://http2.akamai.com/")
 | |
|                     .body(())
 | |
|                     .unwrap();
 | |
|                 let res = client
 | |
|                     .send_request(request, true)
 | |
|                     .unwrap().0
 | |
|                     .wait()
 | |
|                     .expect("request");
 | |
|                 assert_eq!(res.status(), StatusCode::OK);
 | |
|             });
 | |
| 
 | |
|             h2.expect("h2")
 | |
|         });
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn recv_invalid_server_stream_id() {
 | |
|     let _ = ::env_logger::try_init();
 | |
| 
 | |
|     let mock = mock_io::Builder::new()
 | |
|         .handshake()
 | |
|         // Write GET /
 | |
|         .write(&[
 | |
|             0, 0, 0x10, 1, 5, 0, 0, 0, 1, 0x82, 0x87, 0x41, 0x8B, 0x9D, 0x29,
 | |
|                 0xAC, 0x4B, 0x8F, 0xA8, 0xE9, 0x19, 0x97, 0x21, 0xE9, 0x84,
 | |
|         ])
 | |
|         .write(SETTINGS_ACK)
 | |
|         // Read response
 | |
|         .read(&[0, 0, 1, 1, 5, 0, 0, 0, 2, 137])
 | |
|         // Write GO_AWAY
 | |
|         .write(&[0, 0, 8, 7, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1])
 | |
|         .build();
 | |
| 
 | |
|     let (mut client, h2) = client::handshake(mock).wait().unwrap();
 | |
| 
 | |
|     // Send the request
 | |
|     let request = Request::builder()
 | |
|         .uri("https://http2.akamai.com/")
 | |
|         .body(())
 | |
|         .unwrap();
 | |
| 
 | |
|     info!("sending request");
 | |
|     let (response, _) = client.send_request(request, true).unwrap();
 | |
| 
 | |
|     // The connection errors
 | |
|     assert!(h2.wait().is_err());
 | |
| 
 | |
|     // The stream errors
 | |
|     assert!(response.wait().is_err());
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn request_stream_id_overflows() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
| 
 | |
|     let h2 = client::Builder::new()
 | |
|         .initial_stream_id(::std::u32::MAX >> 1)
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::GET)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
| 
 | |
|             h2.drive(response).and_then(move |(h2, _)| {
 | |
|                 let request = Request::builder()
 | |
|                     .method(Method::GET)
 | |
|                     .uri("https://example.com/")
 | |
|                     .body(())
 | |
|                     .unwrap();
 | |
| 
 | |
|                 // second cannot use the next stream id, it's over
 | |
| 
 | |
|                 let poll_err = client.poll_ready().unwrap_err();
 | |
|                 assert_eq!(poll_err.to_string(), "user error: stream ID overflowed");
 | |
| 
 | |
|                 let err = client.send_request(request, true).unwrap_err();
 | |
|                 assert_eq!(err.to_string(), "user error: stream ID overflowed");
 | |
| 
 | |
|                 h2.expect("h2").map(|ret| {
 | |
|                     // Hold on to the `client` handle to avoid sending a GO_AWAY
 | |
|                     // frame.
 | |
|                     drop(client);
 | |
|                     ret
 | |
|                 })
 | |
|             })
 | |
|         });
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(::std::u32::MAX >> 1)
 | |
|                 .request("GET", "https://example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(
 | |
|             frames::headers(::std::u32::MAX >> 1)
 | |
|                 .response(200)
 | |
|                 .eos()
 | |
|         )
 | |
|         .idle_ms(10)
 | |
|         .close();
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn client_builder_max_concurrent_streams() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let mut settings = frame::Settings::default();
 | |
|     settings.set_max_concurrent_streams(Some(1));
 | |
| 
 | |
|     let srv = srv
 | |
|         .assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_custom_settings(settings)
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://example.com/")
 | |
|                 .eos()
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .close();
 | |
| 
 | |
|     let mut builder = client::Builder::new();
 | |
|     builder.max_concurrent_streams(1);
 | |
| 
 | |
|     let h2 = builder
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::GET)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
|             h2.drive(response).map(move |(h2, _)| (client, h2))
 | |
|         });
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn request_over_max_concurrent_streams_errors() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
| 
 | |
|     let srv = srv.assert_client_handshake_with_settings(frames::settings()
 | |
|                 // super tiny server
 | |
|                 .max_concurrent_streams(1))
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("POST", "https://example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .recv_frame(frames::headers(3).request("POST", "https://example.com/"))
 | |
|         .send_frame(frames::headers(3).response(200))
 | |
|         .recv_frame(frames::data(3, "hello").eos())
 | |
|         .send_frame(frames::data(3, "").eos())
 | |
|         .recv_frame(frames::headers(5).request("POST", "https://example.com/"))
 | |
|         .send_frame(frames::headers(5).response(200))
 | |
|         .recv_frame(frames::data(5, "hello").eos())
 | |
|         .send_frame(frames::data(5, "").eos())
 | |
|         .close();
 | |
| 
 | |
|     let h2 = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             // we send a simple req here just to drive the connection so we can
 | |
|             // receive the server settings.
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
|             h2.drive(response).map(move |(h2, _)| (client, h2))
 | |
|         })
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             let (resp1, mut stream1) = client.send_request(request, false).unwrap();
 | |
| 
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // second request is put into pending_open
 | |
|             let (resp2, mut stream2) = client.send_request(request, false).unwrap();
 | |
| 
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::GET)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // third stream is over max concurrent
 | |
|             assert!(client.poll_ready().expect("poll_ready").is_not_ready());
 | |
| 
 | |
|             let err = client.send_request(request, true).unwrap_err();
 | |
|             assert_eq!(err.to_string(), "user error: rejected");
 | |
| 
 | |
|             stream1.send_data("hello".into(), true).expect("req send_data");
 | |
| 
 | |
|             h2.drive(resp1.expect("req")).and_then(move |(h2, _)| {
 | |
|                 stream2.send_data("hello".into(), true)
 | |
|                     .expect("req2 send_data");
 | |
|                 h2.expect("h2").join(resp2.expect("req2"))
 | |
|             })
 | |
|         });
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn send_request_poll_ready_when_connection_error() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
| 
 | |
|     let srv = srv.assert_client_handshake_with_settings(frames::settings()
 | |
|                 // super tiny server
 | |
|                 .max_concurrent_streams(1))
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("POST", "https://example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .recv_frame(frames::headers(3).request("POST", "https://example.com/").eos())
 | |
|         .send_frame(frames::headers(8).response(200).eos())
 | |
|         //.recv_frame(frames::headers(5).request("POST", "https://example.com/").eos())
 | |
|         .close();
 | |
| 
 | |
|     let h2 = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             // we send a simple req here just to drive the connection so we can
 | |
|             // receive the server settings.
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
|             h2.drive(response).map(move |(h2, _)| (client, h2))
 | |
|         })
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             let (resp1, _) = client.send_request(request, true).unwrap();
 | |
| 
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // second request is put into pending_open
 | |
|             let (resp2, _) = client.send_request(request, true).unwrap();
 | |
| 
 | |
|             // third stream is over max concurrent
 | |
|             let until_ready = futures::future::poll_fn(move || {
 | |
|                 client.poll_ready()
 | |
|             }).expect_err("client poll_ready").then(|_| Ok(()));
 | |
| 
 | |
|             // a FuturesUnordered is used on purpose!
 | |
|             //
 | |
|             // We don't want a join, since any of the other futures notifying
 | |
|             // will make the until_ready future polled again, but we are
 | |
|             // specifically testing that until_ready gets notified on its own.
 | |
|             let mut unordered = futures::stream::FuturesUnordered::<Box<Future<Item=(), Error=()>>>::new();
 | |
|             unordered.push(Box::new(until_ready));
 | |
|             unordered.push(Box::new(h2.expect_err("client conn").then(|_| Ok(()))));
 | |
|             unordered.push(Box::new(resp1.expect_err("req1").then(|_| Ok(()))));
 | |
|             unordered.push(Box::new(resp2.expect_err("req2").then(|_| Ok(()))));
 | |
| 
 | |
|             unordered.for_each(|_| Ok(()))
 | |
|         });
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn http_11_request_without_scheme_or_authority() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "/")
 | |
|                 .scheme("http")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .close();
 | |
| 
 | |
|     let h2 = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             // we send a simple req here just to drive the connection so we can
 | |
|             // receive the server settings.
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::GET)
 | |
|                 .uri("/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
|             h2.drive(response)
 | |
|                 .map(move |(h2, _)| (client, h2))
 | |
|         });
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn http_2_request_without_scheme_or_authority() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .close();
 | |
| 
 | |
|     let h2 = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             // we send a simple req here just to drive the connection so we can
 | |
|             // receive the server settings.
 | |
|             let request = Request::builder()
 | |
|                 .version(Version::HTTP_2)
 | |
|                 .method(Method::GET)
 | |
|                 .uri("/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request is allowed
 | |
|             assert!(client.send_request(request, true).is_err());
 | |
|             h2.expect("h2").map(|ret| {
 | |
|                 // Hold on to the `client` handle to avoid sending a GO_AWAY frame.
 | |
|                 drop(client);
 | |
|                 ret
 | |
|             })
 | |
|         });
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| #[ignore]
 | |
| fn request_with_h1_version() {}
 | |
| 
 | |
| #[test]
 | |
| fn request_with_connection_headers() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     // can't assert full handshake, since client never sends a request, and
 | |
|     // thus never bothers to ack the settings...
 | |
|     let srv = srv.read_preface()
 | |
|         .unwrap()
 | |
|         .recv_frame(frames::settings())
 | |
|         // goaway is required to make sure the connection closes because
 | |
|         // of no active streams
 | |
|         .recv_frame(frames::go_away(0))
 | |
|         .close();
 | |
| 
 | |
|     let headers = vec![
 | |
|         ("connection", "foo"),
 | |
|         ("keep-alive", "5"),
 | |
|         ("proxy-connection", "bar"),
 | |
|         ("transfer-encoding", "chunked"),
 | |
|         ("upgrade", "HTTP/2.0"),
 | |
|         ("te", "boom"),
 | |
|     ];
 | |
| 
 | |
|     let client = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(move |(mut client, conn)| {
 | |
|             for (name, val) in headers {
 | |
|                 let req = Request::builder()
 | |
|                     .uri("https://http2.akamai.com/")
 | |
|                     .header(name, val)
 | |
|                     .body(())
 | |
|                     .unwrap();
 | |
|                 let err = client.send_request(req, true).expect_err(name);
 | |
| 
 | |
|                 assert_eq!(err.to_string(), "user error: malformed headers");
 | |
|             }
 | |
|             conn.unwrap()
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn connection_close_notifies_response_future() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://http2.akamai.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         // don't send any response, just close
 | |
|         .close();
 | |
| 
 | |
|     let client = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, conn)| {
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let req = client
 | |
|                 .send_request(request, true)
 | |
|                 .expect("send_request1")
 | |
|                 .0
 | |
|                 .then(|res| {
 | |
|                     let err = res.expect_err("response");
 | |
|                     assert_eq!(
 | |
|                         err.to_string(),
 | |
|                         "broken pipe"
 | |
|                     );
 | |
|                     Ok(())
 | |
|                 });
 | |
| 
 | |
|             conn.expect("conn").join(req)
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn connection_close_notifies_client_poll_ready() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://http2.akamai.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .close();
 | |
| 
 | |
|     let client = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, conn)| {
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let req = client
 | |
|                 .send_request(request, true)
 | |
|                 .expect("send_request1")
 | |
|                 .0
 | |
|                 .then(|res| {
 | |
|                     let err = res.expect_err("response");
 | |
|                     assert_eq!(
 | |
|                         err.to_string(),
 | |
|                         "broken pipe"
 | |
|                     );
 | |
|                     Ok::<_, ()>(())
 | |
|                 });
 | |
| 
 | |
|             conn.drive(req)
 | |
|                 .and_then(move |(_conn, _)| {
 | |
|                     let err = client.poll_ready().expect_err("poll_ready");
 | |
|                     assert_eq!(
 | |
|                         err.to_string(),
 | |
|                         "broken pipe"
 | |
|                     );
 | |
|                     Ok(())
 | |
|                 })
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn sending_request_on_closed_connection() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://http2.akamai.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         // a bad frame!
 | |
|         .send_frame(frames::headers(0).response(200).eos())
 | |
|         .close();
 | |
| 
 | |
|     let h2 = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, h2)| {
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             // first request works
 | |
|             let req = client
 | |
|                 .send_request(request, true)
 | |
|                 .expect("send_request1")
 | |
|                 .0
 | |
|                 .expect("response1")
 | |
|                 .map(|_| ());
 | |
| 
 | |
|             // after finish request1, there should be a conn error
 | |
|             let h2 = h2.then(|res| {
 | |
|                 res.expect_err("h2 error");
 | |
|                 Ok::<(), ()>(())
 | |
|             });
 | |
| 
 | |
|             h2.select(req)
 | |
|                 .then(|res| match res {
 | |
|                     Ok((_, next)) => next,
 | |
|                     Err(_) => unreachable!("both selected futures cannot error"),
 | |
|                 })
 | |
|                 .map(move |_| client)
 | |
|         })
 | |
|         .and_then(|mut client| {
 | |
|             let poll_err = client.poll_ready().unwrap_err();
 | |
|             let msg = "protocol error: unspecific protocol error detected";
 | |
|             assert_eq!(poll_err.to_string(), msg);
 | |
| 
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
|             let send_err = client.send_request(request, true).unwrap_err();
 | |
|             assert_eq!(send_err.to_string(), msg);
 | |
| 
 | |
|             Ok(())
 | |
|         });
 | |
| 
 | |
|     h2.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn recv_too_big_headers() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_custom_settings(
 | |
|             frames::settings()
 | |
|                 .max_header_list_size(10)
 | |
|         )
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://http2.akamai.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .recv_frame(
 | |
|             frames::headers(3)
 | |
|                 .request("GET", "https://http2.akamai.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .send_frame(frames::headers(3).response(200))
 | |
|         // no reset for 1, since it's closed anyways
 | |
|         // but reset for 3, since server hasn't closed stream
 | |
|         .recv_frame(frames::reset(3).refused())
 | |
|         .idle_ms(10)
 | |
|         .close();
 | |
| 
 | |
|     let client = client::Builder::new()
 | |
|         .max_header_list_size(10)
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, conn)| {
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let req1 = client
 | |
|                 .send_request(request, true)
 | |
|                 .expect("send_request")
 | |
|                 .0
 | |
|                 .expect_err("response1")
 | |
|                 .map(|err| {
 | |
|                     assert_eq!(
 | |
|                         err.reason(),
 | |
|                         Some(Reason::REFUSED_STREAM)
 | |
|                     );
 | |
|                 });
 | |
| 
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let req2 = client
 | |
|                 .send_request(request, true)
 | |
|                 .expect("send_request")
 | |
|                 .0
 | |
|                 .expect_err("response2")
 | |
|                 .map(|err| {
 | |
|                     assert_eq!(
 | |
|                         err.reason(),
 | |
|                         Some(Reason::REFUSED_STREAM)
 | |
|                     );
 | |
|                 });
 | |
| 
 | |
|             conn.drive(req1.join(req2))
 | |
|                 .and_then(|(conn, _)| conn.expect("client"))
 | |
|                 .map(|c| (c, client))
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().expect("wait");
 | |
| 
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn pending_send_request_gets_reset_by_peer_properly() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let payload = [0; (frame::DEFAULT_INITIAL_WINDOW_SIZE * 2) as usize];
 | |
|     let max_frame_size = frame::DEFAULT_MAX_FRAME_SIZE as usize;
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://http2.akamai.com/"),
 | |
|         )
 | |
|         // Note that we can only send up to ~4 frames of data by default
 | |
|         .recv_frame(frames::data(1, &payload[0..max_frame_size]))
 | |
|         .recv_frame(frames::data(1, &payload[max_frame_size..(max_frame_size*2)]))
 | |
|         .recv_frame(frames::data(1, &payload[(max_frame_size*2)..(max_frame_size*3)]))
 | |
|         .recv_frame(frames::data(1, &payload[(max_frame_size*3)..(max_frame_size*4-1)]))
 | |
| 
 | |
|         .idle_ms(100)
 | |
| 
 | |
|         .send_frame(frames::reset(1).refused())
 | |
|         // Because all active requests are finished, connection should shutdown
 | |
|         // and send a GO_AWAY frame. If the reset stream is bugged (and doesn't
 | |
|         // count towards concurrency limit), then connection will not send
 | |
|         // a GO_AWAY and this test will fail.
 | |
|         .recv_frame(frames::go_away(0))
 | |
| 
 | |
|         .close();
 | |
| 
 | |
|     let client = client::Builder::new()
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, conn)| {
 | |
|             let request = Request::builder()
 | |
|                 .uri("https://http2.akamai.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let (response, mut stream) = client
 | |
|                 .send_request(request, false)
 | |
|                 .expect("send_request");
 | |
| 
 | |
|             let response = response.expect_err("response")
 | |
|                 .map(|err| {
 | |
|                     assert_eq!(
 | |
|                         err.reason(),
 | |
|                         Some(Reason::REFUSED_STREAM)
 | |
|                     );
 | |
|                 });
 | |
| 
 | |
|             // Send the data
 | |
|             stream.send_data(payload[..].into(), true).unwrap();
 | |
| 
 | |
|             conn.drive(response)
 | |
|                 .and_then(|(conn, _)| conn.expect("client"))
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn request_without_path() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(frames::headers(1).request("GET", "http://example.com/").eos())
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .close();
 | |
| 
 | |
|     let client = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(move |(mut client, conn)| {
 | |
|             // Note the lack of trailing slash.
 | |
|             let request = Request::get("http://example.com")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
| 
 | |
|             conn.drive(response)
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().unwrap();
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn request_options_with_star() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     // Note the lack of trailing slash.
 | |
|     let uri = uri::Uri::from_parts({
 | |
|         let mut parts = uri::Parts::default();
 | |
|         parts.scheme = Some(uri::Scheme::HTTP);
 | |
|         parts.authority = Some(uri::Authority::from_shared("example.com".into()).unwrap());
 | |
|         parts.path_and_query = Some(uri::PathAndQuery::from_static("*"));
 | |
|         parts
 | |
|     }).unwrap();
 | |
| 
 | |
|     let srv = srv.assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(frames::headers(1).request("OPTIONS", uri.clone()).eos())
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .close();
 | |
| 
 | |
|     let client = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(move |(mut client, conn)| {
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::OPTIONS)
 | |
|                 .uri(uri)
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let (response, _) = client.send_request(request, true).unwrap();
 | |
| 
 | |
|             conn.drive(response)
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().unwrap();
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn notify_on_send_capacity() {
 | |
|     // This test ensures that the client gets notified when there is additional
 | |
|     // send capacity. In other words, when the server is ready to accept a new
 | |
|     // stream, the client is notified.
 | |
|     use std::sync::mpsc;
 | |
| 
 | |
|     let _ = ::env_logger::try_init();
 | |
| 
 | |
|     let (io, srv) = mock::new();
 | |
|     let (done_tx, done_rx) = futures::sync::oneshot::channel();
 | |
|     let (tx, rx) = mpsc::channel();
 | |
| 
 | |
|     let mut settings = frame::Settings::default();
 | |
|     settings.set_max_concurrent_streams(Some(1));
 | |
| 
 | |
|     let srv = srv
 | |
|         .assert_client_handshake_with_settings(settings)
 | |
|         .unwrap()
 | |
|         // This is the ACK
 | |
|         .recv_settings()
 | |
|         .map(move |h| {
 | |
|             tx.send(()).unwrap();
 | |
|             h
 | |
|         })
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://www.example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .recv_frame(
 | |
|             frames::headers(3)
 | |
|                 .request("GET", "https://www.example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(3).response(200).eos())
 | |
|         .recv_frame(
 | |
|             frames::headers(5)
 | |
|                 .request("GET", "https://www.example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(5).response(200).eos())
 | |
|         // Don't close the connection until the client is done doing its
 | |
|         // checks.
 | |
|         .wait_for(done_rx)
 | |
|         .close()
 | |
|         ;
 | |
| 
 | |
|     let client = client::handshake(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(move |(mut client, conn)| {
 | |
|             ::std::thread::spawn(move || {
 | |
|                 rx.recv().unwrap();
 | |
| 
 | |
|                 let mut responses = vec![];
 | |
| 
 | |
|                 for _ in 0..3 {
 | |
|                     // Wait for capacity. If the client is **not** notified,
 | |
|                     // this hangs.
 | |
|                     poll_fn(|| client.poll_ready()).wait().unwrap();
 | |
| 
 | |
|                     let request = Request::builder()
 | |
|                         .uri("https://www.example.com/")
 | |
|                         .body(())
 | |
|                         .unwrap();
 | |
| 
 | |
|                     let response = client.send_request(request, true)
 | |
|                         .unwrap().0;
 | |
| 
 | |
|                     responses.push(response);
 | |
|                 }
 | |
| 
 | |
|                 for response in responses {
 | |
|                     let response = response.wait().unwrap();
 | |
|                     assert_eq!(response.status(), StatusCode::OK);
 | |
|                 }
 | |
| 
 | |
|                 poll_fn(|| client.poll_ready()).wait().unwrap();
 | |
| 
 | |
|                 done_tx.send(()).unwrap();
 | |
|             });
 | |
| 
 | |
|             conn.expect("h2")
 | |
|         })
 | |
|         .expect("client");
 | |
| 
 | |
| 
 | |
|     client.join(srv).wait().unwrap();
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn send_stream_poll_reset() {
 | |
|     let _ = ::env_logger::try_init();
 | |
|     let (io, srv) = mock::new();
 | |
| 
 | |
|     let srv = srv
 | |
|         .assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("POST", "https://example.com/")
 | |
|         )
 | |
|         .send_frame(frames::reset(1).refused())
 | |
|         .close();
 | |
| 
 | |
|     let client = client::Builder::new()
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(|(mut client, conn)| {
 | |
|             let request = Request::builder()
 | |
|                 .method(Method::POST)
 | |
|                 .uri("https://example.com/")
 | |
|                 .body(())
 | |
|                 .unwrap();
 | |
| 
 | |
|             let (_response, mut tx) = client.send_request(request, false).unwrap();
 | |
|             conn.drive(futures::future::poll_fn(move || {
 | |
|                 tx.poll_reset()
 | |
|             }))
 | |
|                 .map(|(_, reason)| {
 | |
|                     assert_eq!(reason, Reason::REFUSED_STREAM);
 | |
|                 })
 | |
|         });
 | |
| 
 | |
|     client.join(srv).wait().expect("wait");
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn drop_pending_open() {
 | |
|     // This test checks that a stream queued for pending open behaves correctly when its
 | |
|     // client drops.
 | |
|     let _ = ::env_logger::try_init();
 | |
| 
 | |
|     let (io, srv) = mock::new();
 | |
|     let (init_tx, init_rx) = futures::sync::oneshot::channel();
 | |
|     let (trigger_go_away_tx, trigger_go_away_rx) = futures::sync::oneshot::channel();
 | |
|     let (sent_go_away_tx, sent_go_away_rx) = futures::sync::oneshot::channel();
 | |
|     let (drop_tx, drop_rx) = futures::sync::oneshot::channel();
 | |
| 
 | |
|     let mut settings = frame::Settings::default();
 | |
|     settings.set_max_concurrent_streams(Some(2));
 | |
| 
 | |
|     let srv = srv
 | |
|         .assert_client_handshake_with_settings(settings)
 | |
|         .unwrap()
 | |
|     // This is the ACK
 | |
|         .recv_settings()
 | |
|         .map(move |h| {
 | |
|             init_tx.send(()).unwrap();
 | |
|             h
 | |
|         })
 | |
|         .recv_frame(
 | |
|             frames::headers(1)
 | |
|                 .request("GET", "https://www.example.com/"),
 | |
|         )
 | |
|         .recv_frame(
 | |
|             frames::headers(3)
 | |
|                 .request("GET", "https://www.example.com/")
 | |
|                 .eos(),
 | |
|         )
 | |
|         .wait_for(trigger_go_away_rx)
 | |
|         .send_frame(frames::go_away(3))
 | |
|         .map(move |h| {
 | |
|             sent_go_away_tx.send(()).unwrap();
 | |
|             h
 | |
|         })
 | |
|         .wait_for(drop_rx)
 | |
|         .send_frame(frames::headers(3).response(200).eos())
 | |
|         .recv_frame(
 | |
|             frames::data(1, vec![]).eos(),
 | |
|         )
 | |
|         .send_frame(frames::headers(1).response(200).eos())
 | |
|         .close()
 | |
|         ;
 | |
| 
 | |
|     fn request() -> Request<()> {
 | |
|         Request::builder()
 | |
|             .uri("https://www.example.com/")
 | |
|             .body(())
 | |
|             .unwrap()
 | |
|     }
 | |
| 
 | |
|     let client = client::Builder::new()
 | |
|         .max_concurrent_reset_streams(0)
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(move |(mut client, conn)| {
 | |
|             conn.expect("h2").join(init_rx.expect("init_rx").and_then(move |()| {
 | |
|                 // Fill up the concurrent stream limit.
 | |
|                 assert!(client.poll_ready().unwrap().is_ready());
 | |
|                 let mut response1 = client.send_request(request(), false).unwrap();
 | |
|                 assert!(client.poll_ready().unwrap().is_ready());
 | |
|                 let response2 = client.send_request(request(), true).unwrap();
 | |
|                 assert!(client.poll_ready().unwrap().is_ready());
 | |
|                 let response3 = client.send_request(request(), true).unwrap();
 | |
| 
 | |
|                 // Trigger a GOAWAY frame to invalidate our third request.
 | |
|                 trigger_go_away_tx.send(()).unwrap();
 | |
|                 sent_go_away_rx.expect("sent_go_away_rx").and_then(move |_| {
 | |
|                     // Now drop all the references to that stream.
 | |
|                     drop(response3);
 | |
|                     drop(client);
 | |
|                     drop_tx.send(()).unwrap();
 | |
| 
 | |
|                     // Complete the second request, freeing up a stream.
 | |
|                     response2.0.expect("resp2")
 | |
|                 }).and_then(move |_| {
 | |
|                     response1.1.send_data(Default::default(), true).unwrap();
 | |
|                     response1.0.expect("resp1")
 | |
|                 })
 | |
|             }))
 | |
|         });
 | |
| 
 | |
| 
 | |
|     client.join(srv).wait().unwrap();
 | |
| }
 | |
| 
 | |
| #[test]
 | |
| fn malformed_response_headers_dont_unlink_stream() {
 | |
|     // This test checks that receiving malformed headers frame on a stream with
 | |
|     // no remaining references correctly resets the stream, without prematurely
 | |
|     // unlinking it.
 | |
|     let _ = ::env_logger::try_init();
 | |
| 
 | |
|     let (io, srv) = mock::new();
 | |
|     let (drop_tx, drop_rx) = futures::sync::oneshot::channel();
 | |
|     let (queued_tx, queued_rx) = futures::sync::oneshot::channel();
 | |
| 
 | |
|     let srv = srv
 | |
|         .assert_client_handshake()
 | |
|         .unwrap()
 | |
|         .recv_settings()
 | |
|         .recv_frame(frames::headers(1).request("GET", "http://example.com/"))
 | |
|         .recv_frame(frames::headers(3).request("GET", "http://example.com/"))
 | |
|         .recv_frame(frames::headers(5).request("GET", "http://example.com/"))
 | |
|         .map(move |h| {
 | |
|             drop_tx.send(()).unwrap();
 | |
|             h
 | |
|         })
 | |
|         .wait_for(queued_rx)
 | |
|         .send_bytes(&[
 | |
|             // 2 byte frame
 | |
|             0, 0, 2,
 | |
|             // type: HEADERS
 | |
|             1,
 | |
|             // flags: END_STREAM | END_HEADERS
 | |
|             5,
 | |
|             // stream identifier: 3
 | |
|             0, 0, 0, 3,
 | |
|             // data - invalid (pseudo not at end of block)
 | |
|             144, 135
 | |
|             // Per the spec, this frame should cause a stream error of type
 | |
|             // PROTOCOL_ERROR.
 | |
|         ])
 | |
|         .close()
 | |
|         ;
 | |
| 
 | |
|     fn request() -> Request<()> {
 | |
|         Request::builder()
 | |
|             .uri("http://example.com/")
 | |
|             .body(())
 | |
|             .unwrap()
 | |
|     }
 | |
| 
 | |
|     let client = client::Builder::new()
 | |
|         .handshake::<_, Bytes>(io)
 | |
|         .expect("handshake")
 | |
|         .and_then(move |(mut client, conn)| {
 | |
|             let (_req1, mut send1) = client.send_request(
 | |
|                 request(), false).unwrap();
 | |
|             // Use up most of the connection window.
 | |
|             send1.send_data(vec![0; 65534].into(), true).unwrap();
 | |
|             let (req2, mut send2) = client.send_request(
 | |
|                 request(), false).unwrap();
 | |
|             let (req3, mut send3) = client.send_request(
 | |
|                 request(), false).unwrap();
 | |
|             conn.expect("h2").join(drop_rx.then(move |_| {
 | |
|                 // Use up the remainder of the connection window.
 | |
|                 send2.send_data(vec![0; 2].into(), true).unwrap();
 | |
|                 // Queue up for more connection window.
 | |
|                 send3.send_data(vec![0; 1].into(), true).unwrap();
 | |
|                 queued_tx.send(()).unwrap();
 | |
|                 Ok((req2, req3))
 | |
|             }))
 | |
|         });
 | |
| 
 | |
| 
 | |
|     client.join(srv).wait().unwrap();
 | |
| }
 | |
| 
 | |
| const SETTINGS: &'static [u8] = &[0, 0, 0, 4, 0, 0, 0, 0, 0];
 | |
| const SETTINGS_ACK: &'static [u8] = &[0, 0, 0, 4, 1, 0, 0, 0, 0];
 | |
| 
 | |
| trait MockH2 {
 | |
|     fn handshake(&mut self) -> &mut Self;
 | |
| }
 | |
| 
 | |
| impl MockH2 for mock_io::Builder {
 | |
|     fn handshake(&mut self) -> &mut Self {
 | |
|         self.write(b"PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n")
 | |
|             // Settings frame
 | |
|             .write(SETTINGS)
 | |
|             .read(SETTINGS)
 | |
|             .read(SETTINGS_ACK)
 | |
|     }
 | |
| }
 |