Adding default-tls and rustls-tls features (#44)

* WIP creting rustls feature to replace OpenSSL

* edited README.md to reflect the new feature available on the crate

* implementing some validation to avoid enabling default-tls and rustls-tls feature

* ensuring that rustls-tls its used when feature is enabled

* Little tweaks

---------

Co-authored-by: Celeo <[email protected]>

Cargo.toml

@@ -19,7 +19,7 @@ http = "1.1.0"
 jsonwebtoken = { version = "9.1.0", optional = true }
 log = "0.4.20"
 rand = "0.8.5"
-reqwest = { version = "0.12.5", features = ["json"] }
+reqwest = { version = "0.12.5", default-features = false, features = ["json", "charset", "http2", "macos-system-configuration"] }
 serde = { version = "1.0.188", features = ["derive"] }
 serde_json = "1.0.107"
 sha2 = "0.10.8"
@@ -33,6 +33,8 @@ chrono = "0.4.31"
 uuid = { version = "1.5.0", features = ["v4", "fast-rng"] }
 
 [features]
-default = ["random_state", "validate_jwt"]
+default = ["random_state", "validate_jwt", "default-tls"]
 random_state = []
 validate_jwt = ["dep:jsonwebtoken"]
+default-tls = ["reqwest/default-tls"]
+rustls-tls = ["reqwest/rustls-tls"]

README.md

@@ -13,9 +13,9 @@ Add the latest version to your `Cargo.toml`.
 
 This crate has several features that are enabled by default.
 
-If you don't want or need random SSO state string generation, you can disable the "random_state" feature.
-
-If you don't want or need SSO token verification, you can disable the "validate_jwt" feature.
+- If you don't want or need random SSO state string generation, you can disable the "random_state" feature.
+- If you don't want or need SSO token verification, you can disable the "validate_jwt" feature.
+- If you prefer to use [rustls](https://crates.io/crates/rustls) instead of your system's TLS implementation ([more info here](https://docs.rs/reqwest/latest/reqwest/tls/)) to make requests, you can disable the default features and add the "rustls-tls" feature.
 
 ## Using
 

src/builders.rs

@@ -219,10 +219,18 @@ impl EsiBuilder {
             );
             map
         };
+        #[cfg(not(feature = "rustls-tls"))]
         let client = Client::builder()
             .timeout(http_timeout)
             .default_headers(headers)
             .build()?;
+
+        #[cfg(feature = "rustls-tls")]
+        let client = Client::builder()
+            .timeout(http_timeout)
+            .default_headers(headers)
+            .use_rustls_tls()
+            .build()?;
         Ok(client)
     }
 

src/lib.rs

@@ -98,6 +98,11 @@
 #![deny(clippy::all)]
 #![deny(missing_docs)]
 
+#[cfg(all(feature = "default-tls", feature = "rustls-tls"))]
+compile_error!(
+    "feature \"default-tls\" and feature \"rustls-tls\" cannot be enabled at the same time"
+);
+
 #[macro_use]
 mod macros;