From 6078787daae8bd05510bbd18e782e81fdf2c5c40 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jon=20H=C3=A4ggblad?= Date: Wed, 3 Jul 2024 12:26:47 +0200 Subject: [PATCH] Remove constructor since it's weakly typed (#4689) --- common/http-api-client/src/user_agent.rs | 23 ++++++----------------- 1 file changed, 6 insertions(+), 17 deletions(-) diff --git a/common/http-api-client/src/user_agent.rs b/common/http-api-client/src/user_agent.rs index 6ca8375056..1cdbf45825 100644 --- a/common/http-api-client/src/user_agent.rs +++ b/common/http-api-client/src/user_agent.rs @@ -8,21 +8,10 @@ use nym_bin_common::build_information::{BinaryBuildInformation, BinaryBuildInfor #[derive(Clone, Debug)] pub struct UserAgent { - application: String, - platform: String, - version: String, - git_commit: String, -} - -impl UserAgent { - pub fn new(application: String, platform: String, version: String, git_commit: String) -> Self { - UserAgent { - application, - platform, - version, - git_commit, - } - } + pub application: String, + pub version: String, + pub platform: String, + pub git_commit: String, } impl fmt::Display for UserAgent { @@ -48,8 +37,8 @@ impl From for UserAgent { fn from(build_info: BinaryBuildInformation) -> Self { UserAgent { application: build_info.binary_name.to_string(), - platform: build_info.cargo_triple.to_string(), version: build_info.build_version.to_string(), + platform: build_info.cargo_triple.to_string(), git_commit: build_info.commit_sha.to_string(), } } @@ -59,8 +48,8 @@ impl From for UserAgent { fn from(build_info: BinaryBuildInformationOwned) -> Self { UserAgent { application: build_info.binary_name, - platform: build_info.cargo_triple, version: build_info.build_version, + platform: build_info.cargo_triple, git_commit: build_info.commit_sha, } }