feat: delegate vendoring of OpenSSL to git2 dependency tree

libgit2 is the only reason we rely on OpenSSL instead of using rust-native TLS solutions.

Author: 2bndy5

Cargo.lock

@@ -23,8 +23,6 @@ futures = "0.3.31"
 git2 = "0.20.2"
 lenient_semver = "0.4.2"
 log = { version = "0.4.28", features = ["std"] }
-openssl = { version = "0.10", features = ["vendored"], optional = true }
-openssl-probe = { version = "0.1", optional = true }
 quick-xml = { version = "0.38.3", features = ["serialize"] }
 regex = "1.11.2"
 reqwest = "0.12.23"
@@ -39,9 +37,10 @@ which = "8.0.0"
 [dev-dependencies]
 mockito = "1.7.0"
 tempfile = "3.22.0"
+git2 = { version = "0.20.2", features = ["https"]}
 
 [features]
-openssl-vendored = ["dep:openssl", "dep:openssl-probe"]
+openssl-vendored = ["git2/vendored-openssl"]
 
 [lib]
 bench = false

cpp-linter/Cargo.toml

@@ -22,13 +22,6 @@ use crate::rest_api::{github::GithubApiClient, RestApiClient};
 
 const VERSION: &str = env!("CARGO_PKG_VERSION");
 
-fn probe_ssl_certs() {
-    #[cfg(feature = "openssl-vendored")]
-    unsafe {
-        openssl_probe::init_openssl_env_vars();
-    }
-}
-
 /// This is the backend entry point for console applications.
 ///
 /// The idea here is that all functionality is implemented in Rust. However, passing
@@ -49,8 +42,6 @@ fn probe_ssl_certs() {
 /// alias ("path/to/cpp-linter.exe"). Thus, the parser in [`crate::cli`] will halt on an error
 /// because it is not configured to handle positional arguments.
 pub async fn run_main(args: Vec<String>) -> Result<()> {
-    probe_ssl_certs();
-
     let arg_parser = get_arg_parser();
     let args = arg_parser.get_matches_from(args);
     let cli = Cli::from(&args);