Proxmox/proxmox
5
0
Fork 1
Code Issues Pull Requests Releases Activity

http: sync: remove redundant calls for setting User-Agent

Browse Source 
the requests are all created via the agent that already contains the user
agent, so this internal helper isn't needed anymore.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
Fabian Grünbichler 2023-03-03 12:39:22 +01:00 committed by Wolfgang Bumiller
parent d69fee254a
commit 5ba9d9b2c2
1 changed files with 0 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
proxmox-http/src/client/sync.rs
View File

@ -37,19 +37,7 @@ impl Client {
Ok(builder.build()) Ok(builder.build())
} }
fn add_user_agent(&self, req: ureq::Request) -> ureq::Request {
req.set(
"User-Agent",
self.options
.user_agent
.as_deref()
.unwrap_or(DEFAULT_USER_AGENT_STRING),
)
}
fn call(&self, req: ureq::Request) -> Result<ureq::Response, Error> { fn call(&self, req: ureq::Request) -> Result<ureq::Response, Error> {
let req = self.add_user_agent(req);
req.call().map_err(Into::into) req.call().map_err(Into::into)
} }
@ -57,8 +45,6 @@ impl Client {
where where
R: Read, R: Read,
{ {
let req = self.add_user_agent(req);
req.send(body).map_err(Into::into) req.send(body).map_err(Into::into)
} }
@ -147,7 +133,6 @@ impl HttpClient<String, String> for Client {
let mut req = self let mut req = self
.agent()? .agent()?
.request(request.method().as_str(), &request.uri().to_string()); .request(request.method().as_str(), &request.uri().to_string());
req = self.add_user_agent(req);
let orig_headers = request.headers(); let orig_headers = request.headers();
@ -195,7 +180,6 @@ impl HttpClient<&[u8], Vec<u8>> for Client {
let mut req = self let mut req = self
.agent()? .agent()?
.request(request.method().as_str(), &request.uri().to_string()); .request(request.method().as_str(), &request.uri().to_string());
req = self.add_user_agent(req);
let orig_headers = request.headers(); let orig_headers = request.headers();