Skip to content

Commit f646e86

Browse files
committed
return job enqueueing errors to client
1 parent 98ce51f commit f646e86

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

src/controllers/version/docs.rs

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
use super::CrateVersionPath;
44
use crate::app::AppState;
55
use crate::auth::AuthCheck;
6-
use crate::util::errors::{AppResult, forbidden};
6+
use crate::util::errors::{AppResult, forbidden, server_error};
77
use crate::worker::jobs;
88
use axum::response::{IntoResponse as _, Response};
99
use crates_io_worker::BackgroundJob as _;
@@ -36,15 +36,16 @@ pub async fn rebuild_version_docs(
3636
// validate if version & crate exist
3737
path.load_version_and_crate(&mut conn).await?;
3838

39-
if let Err(error) = jobs::DocsRsQueueRebuild::new(path.name, path.version)
39+
jobs::DocsRsQueueRebuild::new(path.name, path.version)
4040
.enqueue(&mut conn)
4141
.await
42-
{
43-
error!(
44-
?error,
45-
"docs_rs_queue_rebuild: Failed to enqueue background job"
46-
);
47-
}
42+
.map_err(|error| {
43+
error!(
44+
?error,
45+
"docs_rs_queue_rebuild: Failed to enqueue background job"
46+
);
47+
server_error("failed to enqueue background job")
48+
})?;
4849

4950
Ok(StatusCode::CREATED.into_response())
5051
}

0 commit comments

Comments
 (0)