difftreelog
style fix clippy warnings
in: trunk
5 files changed
crates/fleet-base/src/deploy.rsdiffbeforeafterboth--- a/crates/fleet-base/src/deploy.rs
+++ b/crates/fleet-base/src/deploy.rs
@@ -235,17 +235,12 @@
error!("failed to disarm rollback run: {e}");
}
}
- } else {
- match host
- .rm_file("/etc/fleet_rollback_marker", true)
- .in_current_span()
- .await
- {
- Err(_e) => {
- // Marker might not exist, yet better try to remove it.
- }
- _ => {}
- }
+ } else if let Err(_e) = host
+ .rm_file("/etc/fleet_rollback_marker", true)
+ .in_current_span()
+ .await
+ {
+ // Marker might not exist, yet better try to remove it.
}
}
Ok(())
crates/fleet-base/src/host.rsdiffbeforeafterboth--- a/crates/fleet-base/src/host.rs
+++ b/crates/fleet-base/src/host.rs
@@ -214,13 +214,11 @@
pub fn set_session_destination(&self, dest: String) {
self.session_destination
.set(dest)
- .ok()
.expect("session destination is already set")
}
pub fn set_deploy_kind(&self, kind: DeployKind) {
self.deploy_kind
.set(kind)
- .ok()
.expect("deploy kind is already set");
}
pub async fn deploy_kind(&self) -> Result<DeployKind> {
@@ -568,6 +566,7 @@
local: true,
session: OnceLock::new(),
deploy_kind: OnceCell::new(),
+ session_destination: OnceCell::new(),
}
}
@@ -588,6 +587,7 @@
local: self.localhost == name,
session: OnceLock::new(),
deploy_kind: OnceCell::new(),
+ session_destination: OnceCell::new(),
})
}
pub async fn list_hosts(&self) -> Result<Vec<ConfigHost>> {
crates/fleet-shared/src/encoding.rsdiffbeforeafterboth--- a/crates/fleet-shared/src/encoding.rs
+++ b/crates/fleet-shared/src/encoding.rs
@@ -51,10 +51,10 @@
};
let data = if let Some(unprefixed) = string.strip_prefix(BASE64_ENCODED_PREFIX) {
STANDARD_NO_PAD
- .decode(unprefixed.replace(|v| matches!(v, '\n' | '\t' | ' '), ""))
+ .decode(unprefixed.replace(['\n', '\t', ' '], ""))
.map_err(|e| format!("base64-encoded failed: {e}"))?
} else if let Some(unprefixed) = string.strip_prefix(Z85_ENCODED_PREFIX) {
- z85::decode(unprefixed.replace(|v| matches!(v, '\n' | '\t' | ' '), ""))
+ z85::decode(unprefixed.replace(['\n', '\t', ' '], ""))
.map_err(|e| format!("z85-encoded failed: {e}"))?
} else if let Some(unprefixed) = string.strip_prefix(PLAINTEXT_NEWLINE_PREFIX) {
unprefixed.as_bytes().to_owned()
crates/nixlike/src/lib.rsdiffbeforeafterboth--- a/crates/nixlike/src/lib.rs
+++ b/crates/nixlike/src/lib.rs
@@ -138,7 +138,7 @@
}
}
- let key = k.into_iter().last().unwrap();
+ let key = k.into_iter().next_back().unwrap();
if map.contains_key(&key) {
return Err("can't override object");
}
crates/nixlike/src/se_impl.rsdiffbeforeafterboth272728 type Error = Error;28 type Error = Error;292930 fn serialize_element<T: ?Sized>(&mut self, value: &T) -> Result<(), Self::Error>30 fn serialize_element<T>(&mut self, value: &T) -> Result<(), Self::Error>31 where31 where32 T: serde::Serialize,32 T: serde::Serialize,33 T: ?Sized,33 {34 {34 self.0.push(value.serialize(MySerialize)?);35 self.0.push(value.serialize(MySerialize)?);35 Ok(())36 Ok(())444545 type Error = Error;46 type Error = Error;464747 fn serialize_element<T: ?Sized>(&mut self, value: &T) -> Result<(), Self::Error>48 fn serialize_element<T>(&mut self, value: &T) -> Result<(), Self::Error>48 where49 where49 T: serde::Serialize,50 T: serde::Serialize,51 T: ?Sized,50 {52 {51 self.0.push(value.serialize(MySerialize)?);53 self.0.push(value.serialize(MySerialize)?);52 Ok(())54 Ok(())616362 type Error = Error;64 type Error = Error;636564 fn serialize_field<T: ?Sized>(&mut self, value: &T) -> Result<(), Self::Error>66 fn serialize_field<T>(&mut self, value: &T) -> Result<(), Self::Error>65 where67 where66 T: serde::Serialize,68 T: serde::Serialize,69 T: ?Sized,67 {70 {68 self.0.push(value.serialize(MySerialize)?);71 self.0.push(value.serialize(MySerialize)?);69 Ok(())72 Ok(())818482 type Error = Error;85 type Error = Error;838684 fn serialize_field<T: ?Sized>(&mut self, value: &T) -> Result<(), Self::Error>87 fn serialize_field<T>(&mut self, value: &T) -> Result<(), Self::Error>85 where88 where86 T: serde::Serialize,89 T: serde::Serialize,90 T: ?Sized,87 {91 {88 self.1.serialize_field(value)92 self.1.serialize_field(value)89 }93 }102106103 type Error = Error;107 type Error = Error;104108105 fn serialize_key<T: ?Sized>(&mut self, key: &T) -> Result<(), Self::Error>109 fn serialize_key<T>(&mut self, key: &T) -> Result<(), Self::Error>106 where110 where107 T: serde::Serialize,111 T: serde::Serialize,112 T: ?Sized,108 {113 {109 let _ = self114 let _ = self110 .1115 .1111 .insert(key.serialize(MySerialize)?.parse_string()?.to_owned());116 .insert(key.serialize(MySerialize)?.parse_string()?.to_owned());112 Ok(())117 Ok(())113 }118 }114119115 fn serialize_value<T: ?Sized>(&mut self, value: &T) -> Result<(), Self::Error>120 fn serialize_value<T>(&mut self, value: &T) -> Result<(), Self::Error>116 where121 where117 T: serde::Serialize,122 T: serde::Serialize,123 T: ?Sized,118 {124 {119 self.0125 self.0120 .insert(self.1.take().unwrap(), value.serialize(MySerialize)?);126 .insert(self.1.take().unwrap(), value.serialize(MySerialize)?);133139134 type Error = Error;140 type Error = Error;135141136 fn serialize_field<T: ?Sized>(&mut self, key: &str, value: &T) -> Result<(), Self::Error>142 fn serialize_field<T>(&mut self, key: &str, value: &T) -> Result<(), Self::Error>137 where143 where138 T: serde::Serialize,144 T: serde::Serialize,145 T: ?Sized,139 {146 {140 self.0.insert(key.to_owned(), value.serialize(MySerialize)?);147 self.0.insert(key.to_owned(), value.serialize(MySerialize)?);141 Ok(())148 Ok(())153160154 type Error = Error;161 type Error = Error;155162156 fn serialize_field<T: ?Sized>(163 fn serialize_field<T>(&mut self, key: &'static str, value: &T) -> Result<(), Self::Error>157 &mut self,158 key: &'static str,159 value: &T,160 ) -> Result<(), Self::Error>161 where164 where162 T: serde::Serialize,165 T: serde::Serialize,166 T: ?Sized,163 {167 {164 self.1.insert(key.to_owned(), value.serialize(MySerialize)?);168 self.1.insert(key.to_owned(), value.serialize(MySerialize)?);165 Ok(())169 Ok(())253 Ok(Value::Null)257 Ok(Value::Null)254 }258 }255259256 fn serialize_some<T: ?Sized>(self, value: &T) -> Result<Self::Ok, Self::Error>260 fn serialize_some<T>(self, value: &T) -> Result<Self::Ok, Self::Error>257 where261 where258 T: serde::Serialize,262 T: serde::Serialize,263 T: ?Sized,259 {264 {260 value.serialize(self)265 value.serialize(self)261 }266 }277 Ok(Value::String(variant.to_string()))282 Ok(Value::String(variant.to_string()))278 }283 }279284280 fn serialize_newtype_struct<T: ?Sized>(285 fn serialize_newtype_struct<T>(281 self,286 self,282 _name: &'static str,287 _name: &'static str,283 value: &T,288 value: &T,284 ) -> Result<Self::Ok, Self::Error>289 ) -> Result<Self::Ok, Self::Error>285 where290 where286 T: serde::Serialize,291 T: serde::Serialize,292 T: ?Sized,287 {293 {288 value.serialize(self)294 value.serialize(self)289 }295 }290296291 fn serialize_newtype_variant<T: ?Sized>(297 fn serialize_newtype_variant<T>(292 self,298 self,293 _name: &'static str,299 _name: &'static str,294 _variant_index: u32,300 _variant_index: u32,297 ) -> Result<Self::Ok, Self::Error>303 ) -> Result<Self::Ok, Self::Error>298 where304 where299 T: serde::Serialize,305 T: serde::Serialize,306 T: ?Sized,300 {307 {301 Ok(Value::Object(308 Ok(Value::Object(302 vec![(variant.to_string(), value.serialize(self)?)]309 vec![(variant.to_string(), value.serialize(self)?)]