From 7f56e0774c1aba59fa3926faef4c98a2993682b5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabian=20Gr=C3=BCnbichler?= Date: Fri, 14 May 2021 15:44:44 +0200 Subject: [PATCH] http: takeover ProxyConfig from proxmox_backup MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit adapted to use moved build_authority helper. Signed-off-by: Fabian Grünbichler --- proxmox-http/Cargo.toml | 2 +- proxmox-http/src/http/mod.rs | 4 +- proxmox-http/src/http/proxy_config.rs | 84 +++++++++++++++++++++++++++ 3 files changed, 88 insertions(+), 2 deletions(-) create mode 100644 proxmox-http/src/http/proxy_config.rs diff --git a/proxmox-http/Cargo.toml b/proxmox-http/Cargo.toml index 6b2d8db8..9c6fc35a 100644 --- a/proxmox-http/Cargo.toml +++ b/proxmox-http/Cargo.toml @@ -27,5 +27,5 @@ proxmox = { path = "../proxmox", optional = true, version = "0.11.3", default-fe default = [] client = [ "http-helpers" ] -http-helpers = [ "http", "hyper", "tokio/io-util", "tokio-openssl" ] +http-helpers = [ "base64", "http", "hyper", "tokio/io-util", "tokio-openssl", "proxmox" ] websocket = [ "base64", "futures", "hyper", "openssl", "proxmox/tokio", "tokio/io-util", "tokio/sync" ] diff --git a/proxmox-http/src/http/mod.rs b/proxmox-http/src/http/mod.rs index 49602465..055648ee 100644 --- a/proxmox-http/src/http/mod.rs +++ b/proxmox-http/src/http/mod.rs @@ -1,5 +1,7 @@ mod wrapper; - pub use wrapper::MaybeTlsStream; pub mod helpers; + +mod proxy_config; +pub use proxy_config::ProxyConfig; diff --git a/proxmox-http/src/http/proxy_config.rs b/proxmox-http/src/http/proxy_config.rs new file mode 100644 index 00000000..7e93a477 --- /dev/null +++ b/proxmox-http/src/http/proxy_config.rs @@ -0,0 +1,84 @@ +use anyhow::{Error, format_err, bail}; + +use http::Uri; + +use crate::http::helpers; + +/// HTTP Proxy Configuration +#[derive(Clone)] +pub struct ProxyConfig { + pub host: String, + pub port: u16, + pub authorization: Option, // user:pass + pub force_connect: bool, +} + +impl ProxyConfig { + /// Parse proxy config from ALL_PROXY environment var + pub fn from_proxy_env() -> Result, Error> { + + // We only support/use ALL_PROXY environment + + match std::env::var_os("ALL_PROXY") { + None => return Ok(None), + Some(all_proxy) => { + let all_proxy = match all_proxy.to_str() { + Some(s) => String::from(s), + None => bail!("non UTF-8 content in env ALL_PROXY"), + }; + if all_proxy.is_empty() { + return Ok(None); + } + let config = Self::parse_proxy_url(&all_proxy)?; + Ok(Some(config)) + } + } + } + + /// Parse proxy configuration string [http://][:port] + /// + /// Default port is 1080 (like curl) + pub fn parse_proxy_url(http_proxy: &str) -> Result { + proxmox::try_block!({ + let proxy_uri: Uri = http_proxy.parse()?; + let proxy_authority = match proxy_uri.authority() { + Some(authority) => authority, + None => bail!("missing proxy authority"), + }; + let host = proxy_authority.host().to_owned(); + let port = match proxy_uri.port() { + Some(port) => port.as_u16(), + None => 1080, // CURL default port + }; + + match proxy_uri.scheme_str() { + Some("http") => { /* Ok */ } + Some(scheme) => bail!("unsupported proxy scheme '{}'", scheme), + None => { /* assume HTTP */ } + } + + let authority_vec: Vec<&str> = proxy_authority.as_str().rsplitn(2, '@').collect(); + let authorization = if authority_vec.len() == 2 { + Some(authority_vec[1].to_string()) + } else { + None + }; + + Ok(ProxyConfig { + host, + port, + authorization, + force_connect: false, + }) + }).map_err(|err| format_err!("parse_proxy_url failed: {}", err)) + } + + /// Assemble canonical proxy string (including scheme and port) + pub fn to_proxy_string(&self) -> Result { + let authority = helpers::build_authority(&self.host, self.port)?; + Ok(match self.authorization { + None => format!("http://{}", authority), + Some(ref authorization) => format!("http://{}@{}", authorization, authority) + }) + } +}