lywsvip/rustfs
1
0
Fork 0
mirror of https://github.com/rustfs/rustfs.git synced 2026-05-06 22:28:16 +08:00
Code Issues Packages Projects Releases Wiki Activity

test(admin): cover empty kms key aliases when merging #2331

Browse Source
This commit is contained in:
overtrue
2026-03-30 20:14:51 +08:00
parent 7d97f4f66f
commit 0b840dcc86
1 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
rustfs/src/admin/handlers/kms_keys.rs
View File

@@ -314,6 +314,18 @@ mod tests {
assert_eq!(extract_key_id(&uri).as_deref(), expected);
}
}
#[test]
fn test_extract_key_id_skips_empty_aliases() {
for (uri, expected) in [
("/rustfs/admin/v3/kms/key/status?keyId=&key-id=minio-key", Some("minio-key")),
("/rustfs/admin/v3/kms/key/status?keyId=&key-id=&key=fallback-key", Some("fallback-key")),
("/rustfs/admin/v3/kms/key/status?keyId=&key-id=&key=", None),
] {
let uri: Uri = uri.parse().expect("uri should parse");
assert_eq!(extract_key_id(&uri).as_deref(), expected);
}
}
}
/// List KMS keys (legacy endpoint)