Skip to content

Commit 3589920

Browse files
authored
address clippy warnings for azure_messaging_servicebus (#1404)
1 parent 3da0693 commit 3589920

File tree

2 files changed

+9
-13
lines changed

2 files changed

+9
-13
lines changed

sdk/messaging_servicebus/src/service_bus/mod.rs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,11 @@ fn finalize_request(
4343
request.insert_header(headers::AUTHORIZATION, sas);
4444

4545
// get req body to return
46-
match body {
47-
Some(msg) => request.set_body(msg),
48-
None => {
49-
request.insert_header(headers::CONTENT_LENGTH, "0"); // added to avoid truncation errors
50-
request.set_body(azure_core::EMPTY_BODY);
51-
}
46+
if let Some(body) = body {
47+
request.set_body(body);
48+
} else {
49+
request.insert_header(headers::CONTENT_LENGTH, "0"); // added to avoid truncation errors
50+
request.set_body(azure_core::EMPTY_BODY);
5251
}
5352

5453
Ok(request)

sdk/messaging_servicebus/src/utils.rs

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,9 @@ pub fn body_bytes_to_utf8(bytes: &[u8]) -> Result<String, Error> {
3333
}
3434

3535
pub fn get_head_url(namespace: &str, queue_or_topic: &str, subscription: Option<&str>) -> String {
36-
match subscription {
37-
Some(sub) => format!(
38-
"https://{namespace}.servicebus.windows.net/{queue_or_topic}/subscriptions/{sub}/messages/head"
39-
),
40-
None => format!(
41-
"https://{namespace}.servicebus.windows.net/{queue_or_topic}/messages/head"
42-
),
36+
if let Some(sub) = subscription {
37+
format!("https://{namespace}.servicebus.windows.net/{queue_or_topic}/subscriptions/{sub}/messages/head")
38+
} else {
39+
format!("https://{namespace}.servicebus.windows.net/{queue_or_topic}/messages/head")
4340
}
4441
}

0 commit comments

Comments
 (0)