add enabled argument to api

bindings/rust/s2n-tls-hyper/src/connector.rs

@@ -120,8 +120,8 @@ pub struct Builder<Http, ConnBuilder> {
 
 impl<Http, ConnBuilder> Builder<Http, ConnBuilder> {
     /// Allows communication with insecure HTTP endpoints in addition to secure HTTPS endpoints.
-    pub fn with_insecure_http(&mut self) -> &mut Self {
-        self.insecure_http = true;
+    pub fn with_insecure_http(&mut self, enabled: bool) -> &mut Self {
+        self.insecure_http = enabled;
         self
     }
 
@@ -274,4 +274,13 @@ mod tests {
 
         Ok(())
     }
+
+    #[tokio::test]
+    async fn default_builder() -> Result<(), Box<dyn StdError>> {
+        // Ensure that insecure HTTP is disabled by default.
+        let connector = HttpsConnector::builder(Config::default()).build();
+        assert!(!connector.insecure_http);
+
+        Ok(())
+    }
 }

bindings/rust/s2n-tls-hyper/tests/http.rs

@@ -360,9 +360,7 @@ async fn insecure_http() -> Result<(), Box<dyn Error + Send + Sync>> {
             let connector = {
                 let config = common::config()?.build()?;
                 let mut builder = HttpsConnector::builder(config);
-                if enable_insecure_http {
-                    builder.with_insecure_http();
-                }
+                builder.with_insecure_http(enable_insecure_http);
                 builder.build()
             };
 
@@ -376,7 +374,7 @@ async fn insecure_http() -> Result<(), Box<dyn Error + Send + Sync>> {
                 let response = response.unwrap();
                 assert_eq!(response.status(), 200);
             } else {
-                // By default, insecure HTTP is disabled, and the request should error.
+                // If insecure HTTP is disabled, the request should error.
                 let error = response.unwrap_err();
 
                 // Ensure an InvalidScheme error is produced.