Mirror/stealth
1
0
Fork 0
mirror of https://github.com/LORDBABUINO/stealth.git synced 2026-06-09 22:13:29 -07:00
Code Issues Packages Projects Releases Wiki Activity

refactor(api): use match for scan input validation

Browse Source 
The if-chain and counter pattern is a 5-arm tuple match in disguise.
Pattern match on (descriptor, descriptors, utxos) directly: every
valid case picks exactly one variant, the empty tuple is a clear
missing-input error, and the wildcard catches mutual-exclusion
violations.
This commit is contained in:
LORDBABUINO
2026-05-11 20:18:52 -03:00
parent ca827db53e
commit 446fb9eb45
1 changed files with 7 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files
api/src/routes/wallet.rs
+7 -31
View File
@@ -41,39 +41,15 @@ async fn scan_post(
impl ScanRequestBody {
fn into_scan_target(self) -> Result<ScanTarget, ApiError> {
let mut selected_sources = 0usize;
if self.descriptor.is_some() {
selected_sources += 1;
}
if self.descriptors.is_some() {
selected_sources += 1;
}
if self.utxos.is_some() {
selected_sources += 1;
}
if selected_sources == 0 {
return Err(ApiError::bad_request(
match (self.descriptor, self.descriptors, self.utxos) {
(Some(d), None, None) => Ok(ScanTarget::Descriptor(d)),
(None, Some(ds), None) => Ok(ScanTarget::Descriptors(ds)),
(None, None, Some(utxos)) => Ok(ScanTarget::Utxos(utxos)),
(None, None, None) => Err(ApiError::bad_request(
"one input source is required: descriptor, descriptors, or utxos",
));
)),
_ => Err(ApiError::bad_request("provide exactly one input source")),
}
if selected_sources > 1 {
return Err(ApiError::bad_request(
"descriptor, descriptors, and utxos are mutually exclusive",
));
}
if let Some(descriptor) = self.descriptor {
return Ok(ScanTarget::Descriptor(descriptor));
}
if let Some(descriptors) = self.descriptors {
return Ok(ScanTarget::Descriptors(descriptors));
}
if let Some(utxos) = self.utxos {
return Ok(ScanTarget::Utxos(utxos));
}
Err(ApiError::bad_request("invalid scan request body"))
}
}