update feature setup to allow to use with http only

Author: jayjamesjay

README.md

@@ -16,7 +16,7 @@ http_req by default uses [rust-native-tls](https://crates.io/crates/native-tls),
 which relies on TLS framework provided by OS on Windows and macOS, and OpenSSL
 on all other platforms. But it also supports [rustls](https://crates.io/crates/rustls).
 
-## Features
+## All features
 
 - Support for both HTTP and HTTPS protocols via [rust-native-tls](https://crates.io/crates/native-tls) (or optionally [rustls](https://crates.io/crates/rustls))
 - Creating and sending HTTP requests using the `Request` type (with extended capabilities provided via `RequestMessage` and `Stream`)
@@ -50,6 +50,15 @@ In order to use `http_req` with `rustls` in your project, add the following line
 http_req = { version="^0.14", default-features = false, features = ["rust-tls"] }
 ```
 
+### HTTP only
+
+In order to use `http_req` without any additional features in your project (no HTTPS, no Authentication), add the following lines to `Cargo.toml`:
+
+```toml
+[dependencies]
+http_req = { version="^0.14", default-features = false }
+```
+
 ## Example
 
 Basic HTTP GET request

examples/authentication.rs

@@ -7,7 +7,7 @@ fn main() {
     // Container for body of a response.
     let mut body = Vec::new();
     // URL of the website.
-    let uri = Uri::try_from("http://httpbin.org/basic-auth/foo/bar").unwrap();
+    let uri = Uri::try_from("https://httpbin.org/basic-auth/foo/bar").unwrap();
     // Authentication details: username and password.
     let auth = Authentication::basic("foo", "bar");
 

src/lib.rs

@@ -23,6 +23,7 @@ pub mod error;
 pub mod request;
 pub mod response;
 pub mod stream;
+#[cfg(any(feature = "native-tls", feature = "rust-tls"))]
 pub mod tls;
 pub mod uri;
 

src/request.rs

@@ -832,7 +832,11 @@ impl<'a> Request<'a> {
         let mut stream = Stream::connect(self.message.uri, self.connect_timeout)?;
         stream.set_read_timeout(self.read_timeout)?;
         stream.set_write_timeout(self.write_timeout)?;
-        stream = Stream::try_to_https(stream, self.message.uri, self.root_cert_file_pem)?;
+
+        #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
+        {
+            stream = Stream::try_to_https(stream, self.message.uri, self.root_cert_file_pem)?;
+        }
 
         // Send the request message to the stream.
         let request_msg = self.message.parse();

src/stream.rs

@@ -1,15 +1,17 @@
 //! TCP stream
 
+#[cfg(any(feature = "native-tls", feature = "rust-tls"))]
+use crate::tls::{self, Conn};
 use crate::{
     error::{Error, ParseErr},
-    tls::{self, Conn},
     uri::Uri,
     CR_LF, LF,
 };
+#[cfg(any(feature = "native-tls", feature = "rust-tls"))]
+use std::path::Path;
 use std::{
     io::{self, BufRead, Read, Write},
     net::{TcpStream, ToSocketAddrs},
-    path::Path,
     sync::mpsc::{Receiver, RecvTimeoutError, Sender},
     time::{Duration, Instant},
 };
@@ -21,6 +23,7 @@ const BUF_SIZE: usize = 16 * 1000;
 #[derive(Debug)]
 pub enum Stream {
     Http(TcpStream),
+    #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
     Https(Conn<TcpStream>),
 }
 
@@ -43,6 +46,7 @@ impl Stream {
     /// Checks if `uri` scheme denotes a HTTPS protocol:
     /// - If yes, attempts to establish a secure connection
     /// - Otherwise, returns the `stream` without any modification
+    #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
     pub fn try_to_https(
         stream: Stream,
         uri: &Uri,
@@ -73,6 +77,7 @@ impl Stream {
     pub fn set_read_timeout(&mut self, dur: Option<Duration>) -> Result<(), Error> {
         match self {
             Stream::Http(stream) => Ok(stream.set_read_timeout(dur)?),
+            #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
             Stream::Https(conn) => Ok(conn.get_mut().set_read_timeout(dur)?),
         }
     }
@@ -81,6 +86,7 @@ impl Stream {
     pub fn set_write_timeout(&mut self, dur: Option<Duration>) -> Result<(), Error> {
         match self {
             Stream::Http(stream) => Ok(stream.set_write_timeout(dur)?),
+            #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
             Stream::Https(conn) => Ok(conn.get_mut().set_write_timeout(dur)?),
         }
     }
@@ -90,6 +96,7 @@ impl Read for Stream {
     fn read(&mut self, buf: &mut [u8]) -> Result<usize, io::Error> {
         match self {
             Stream::Http(stream) => stream.read(buf),
+            #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
             Stream::Https(stream) => stream.read(buf),
         }
     }
@@ -99,13 +106,15 @@ impl Write for Stream {
     fn write(&mut self, buf: &[u8]) -> Result<usize, io::Error> {
         match self {
             Stream::Http(stream) => stream.write(buf),
+            #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
             Stream::Https(stream) => stream.write(buf),
         }
     }
 
     fn flush(&mut self) -> Result<(), io::Error> {
         match self {
             Stream::Http(stream) => stream.flush(),
+            #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
             Stream::Https(stream) => stream.flush(),
         }
     }
@@ -318,6 +327,7 @@ mod tests {
     }
 
     #[test]
+    #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
     fn stream_try_to_https() {
         {
             let uri = Uri::try_from(URI_S).unwrap();
@@ -362,6 +372,7 @@ mod tests {
 
             assert_eq!(inner_read_timeout, Some(TIMEOUT));
         }
+        #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
         {
             let uri = Uri::try_from(URI_S).unwrap();
             let mut stream = Stream::connect(&uri, None).unwrap();
@@ -393,6 +404,7 @@ mod tests {
 
             assert_eq!(inner_read_timeout, Some(TIMEOUT));
         }
+        #[cfg(any(feature = "native-tls", feature = "rust-tls"))]
         {
             let uri = Uri::try_from(URI_S).unwrap();
             let mut stream = Stream::connect(&uri, None).unwrap();