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

fix(ecstore): replace unsafe k.unwrap() in bucket_target_sys (#729) (#3982)

Browse Source 
fix(ecstore): replace k.unwrap() with safe pattern in bucket_target_sys

Replace unsafe k.unwrap().as_str() with if let Some(key_str) pattern
in 5 locations where HeaderMap iterator yields (Option<HeaderName>, Value).

This prevents potential panics if header names are invalid.

Refs https://github.com/rustfs/backlog/issues/729
This commit is contained in:
Zhengchao An
2026-06-28 10:42:56 +08:00
committed by GitHub
parent 259a99a501
commit 6e72dc3076
1 changed files with 20 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/ecstore/src/bucket/bucket_target_sys.rs
+20 -15
View File
@@ -1525,9 +1525,10 @@ impl TargetClient {
.customize() .customize()
.map_request(move |mut req| { .map_request(move |mut req| {
for (k, v) in headers.clone().into_iter() { for (k, v) in headers.clone().into_iter() {
let key_str = k.unwrap().as_str().to_string(); if let Some(key_str) = k.map(|k| k.as_str().to_string()) {
let value_str = v.to_str().unwrap_or("").to_string(); let value_str = v.to_str().unwrap_or("").to_string();
req.headers_mut().insert(key_str, value_str); req.headers_mut().insert(key_str, value_str);
}
} }
Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req) Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req)
@@ -1586,9 +1587,10 @@ impl TargetClient {
.customize() .customize()
.map_request(move |mut req| { .map_request(move |mut req| {
for (k, v) in headers.clone().into_iter() { for (k, v) in headers.clone().into_iter() {
let key_str = k.unwrap().as_str().to_string(); if let Some(key_str) = k.map(|k| k.as_str().to_string()) {
let value_str = v.to_str().unwrap_or("").to_string(); let value_str = v.to_str().unwrap_or("").to_string();
req.headers_mut().insert(key_str, value_str); req.headers_mut().insert(key_str, value_str);
}
} }
Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req) Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req)
}) })
@@ -1625,9 +1627,10 @@ impl TargetClient {
.customize() .customize()
.map_request(move |mut req| { .map_request(move |mut req| {
for (k, v) in headers.clone().into_iter() { for (k, v) in headers.clone().into_iter() {
let key_str = k.unwrap().as_str().to_string(); if let Some(key_str) = k.map(|k| k.as_str().to_string()) {
let value_str = v.to_str().unwrap_or("").to_string(); let value_str = v.to_str().unwrap_or("").to_string();
req.headers_mut().insert(key_str, value_str); req.headers_mut().insert(key_str, value_str);
}
} }
Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req) Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req)
}) })
@@ -1661,9 +1664,10 @@ impl TargetClient {
.customize() .customize()
.map_request(move |mut req| { .map_request(move |mut req| {
for (k, v) in headers.clone().into_iter() { for (k, v) in headers.clone().into_iter() {
let key_str = k.unwrap().as_str().to_string(); if let Some(key_str) = k.map(|k| k.as_str().to_string()) {
let value_str = v.to_str().unwrap_or("").to_string(); let value_str = v.to_str().unwrap_or("").to_string();
req.headers_mut().insert(key_str, value_str); req.headers_mut().insert(key_str, value_str);
}
} }
Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req) Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req)
}) })
@@ -1694,9 +1698,10 @@ impl TargetClient {
.customize() .customize()
.map_request(move |mut req| { .map_request(move |mut req| {
for (k, v) in headers.clone().into_iter() { for (k, v) in headers.clone().into_iter() {
let key_str = k.unwrap().as_str().to_string(); if let Some(key_str) = k.map(|k| k.as_str().to_string()) {
let value_str = v.to_str().unwrap_or("").to_string(); let value_str = v.to_str().unwrap_or("").to_string();
req.headers_mut().insert(key_str, value_str); req.headers_mut().insert(key_str, value_str);
}
} }
Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req) Result::<_, aws_smithy_types::error::operation::BuildError>::Ok(req)
}) })