Skip to content

Commit 4f85436

Browse files
fix: Format code and ignore rustls-pemfile unmaintained advisory
- Apply cargo fmt to src/api.rs (vision endpoint formatting) - Ignore RUSTSEC-2025-0134 (rustls-pemfile unmaintained) - Transitive dependency from reqwest 0.11.x - Waiting for ecosystem migration to reqwest 0.12
1 parent a2e5ec8 commit 4f85436

2 files changed

Lines changed: 10 additions & 17 deletions

File tree

deny.toml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,10 +68,9 @@ feature-depth = 1
6868
# A list of advisory IDs to ignore. Note that ignored advisories will still
6969
# output a note when they are encountered.
7070
ignore = [
71-
#"RUSTSEC-0000-0000",
72-
#{ id = "RUSTSEC-0000-0000", reason = "you can specify a reason the advisory is ignored" },
73-
#"a-crate-that-is-yanked@0.1.1", # you can also ignore yanked crate versions if you wish
74-
#{ crate = "a-crate-that-is-yanked@0.1.1", reason = "you can specify why you are ignoring the yanked crate" },
71+
# rustls-pemfile unmaintained - transitive dependency through reqwest 0.11.x
72+
# We'll update when reqwest 0.12.x is stable and all our dependencies support it
73+
{ id = "RUSTSEC-2025-0134", reason = "Transitive dep from reqwest 0.11.x - waiting for ecosystem to migrate to reqwest 0.12" },
7574
]
7675
# If this is true, then cargo deny will use the git executable to fetch advisory database.
7776
# If this is false, then it uses a built-in git library.

src/api.rs

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1194,14 +1194,7 @@ pub async fn vision(
11941194
axum::http::StatusCode::INTERNAL_SERVER_ERROR
11951195
}
11961196

1197-
match crate::vision::process_vision_request(
1198-
req,
1199-
&model_name,
1200-
license_manager,
1201-
&state,
1202-
)
1203-
.await
1204-
{
1197+
match crate::vision::process_vision_request(req, &model_name, license_manager, &state).await {
12051198
Ok(response) => Json(response).into_response(),
12061199
Err(e) => {
12071200
// Check if it's a license error
@@ -1220,11 +1213,12 @@ pub async fn vision(
12201213
tracing::error!(status = %status, "Vision processing error: {}", full_message);
12211214
// Expose client error messages (4xx) to help users fix their requests.
12221215
// Hide server error details (5xx) unless running in dev mode.
1223-
let message = if status.is_client_error() || std::env::var("SHIMMY_VISION_DEV_MODE").is_ok() {
1224-
full_message
1225-
} else {
1226-
"Vision processing error".to_string()
1227-
};
1216+
let message =
1217+
if status.is_client_error() || std::env::var("SHIMMY_VISION_DEV_MODE").is_ok() {
1218+
full_message
1219+
} else {
1220+
"Vision processing error".to_string()
1221+
};
12281222

12291223
(
12301224
status,

0 commit comments

Comments
 (0)