axum-core/macros: Avoid double body_text() calls in rejection macros (

#3110)
tokio-rs · Dec 27, 2024 · 7c934f2 · 7c934f2
1 parent 32a948e
commit 7c934f2
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/axum-core/src/macros.rs b/axum-core/src/macros.rs
@@ -49,12 +49,14 @@ macro_rules! __define_rejection {
 
         impl $crate::response::IntoResponse for $name {
             fn into_response(self) -> $crate::response::Response {
+                let status = self.status();
+
                 $crate::__log_rejection!(
                     rejection_type = $name,
                     body_text = $body,
-                    status = http::StatusCode::$status,
+                    status = status,
                 );
-                (self.status(), $body).into_response()
+                (status, $body).into_response()
             }
         }
 
@@ -106,12 +108,15 @@ macro_rules! __define_rejection {
 
         impl $crate::response::IntoResponse for $name {
             fn into_response(self) -> $crate::response::Response {
+                let status = self.status();
+                let body_text = self.body_text();
+
                 $crate::__log_rejection!(
                     rejection_type = $name,
-                    body_text = self.body_text(),
-                    status = http::StatusCode::$status,
+                    body_text = body_text,
+                    status = status,
                 );
-                (self.status(), self.body_text()).into_response()
+                (status, body_text).into_response()
             }
         }