From c56b49bd7d0cfc1de332b9301cfd2eec13d8b63d Mon Sep 17 00:00:00 2001
From: Tobias Bieniek <tobias@bieniek.cloud>
Date: Fri, 13 Dec 2024 16:43:55 +0100
Subject: [PATCH] router: Remove obsolete `DefaultBodyLimit` override (#10197)

Now that we are not using the `BytesRequest` in the publish endpoint, this limit isn't actually used anymore. The `StreamReader` does not care about the limit and the JSON metadata blob and the tarball have their dedicated limits already.
---
 src/router.rs | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/router.rs b/src/router.rs
index 572f617f27..1da715e710 100644
--- a/src/router.rs
+++ b/src/router.rs
@@ -1,4 +1,3 @@
-use axum::extract::DefaultBodyLimit;
 use axum::response::IntoResponse;
 use axum::routing::{delete, get, post, put};
 use axum::{Json, Router};
@@ -12,8 +11,6 @@ use crate::openapi::BaseOpenApi;
 use crate::util::errors::not_found;
 use crate::Env;
 
-const MAX_PUBLISH_CONTENT_LENGTH: usize = 128 * 1024 * 1024; // 128 MB
-
 pub fn build_axum_router(state: AppState) -> Router<()> {
     let (router, openapi) = BaseOpenApi::router()
         .routes(routes!(
@@ -26,9 +23,7 @@ pub fn build_axum_router(state: AppState) -> Router<()> {
         // Routes used by `cargo`
         .route(
             "/api/v1/crates/new",
-            put(krate::publish::publish)
-                .layer(DefaultBodyLimit::max(MAX_PUBLISH_CONTENT_LENGTH))
-                .get(krate::metadata::show_new),
+            put(krate::publish::publish).get(krate::metadata::show_new),
         )
         .route(
             "/api/v1/crates/:crate_id/owners",