fix: borrow instead of as string (#674)

astuyve · web-flow · commit 0daf4e9fe475 · 2025-05-28T13:18:30.000-04:00
diff --git a/bottlecap/src/logs/lambda/processor.rs b/bottlecap/src/logs/lambda/processor.rs
@@ -164,7 +164,7 @@ impl LambdaProcessor {
                 if let Some(metrics) = metrics {
                     self.invocation_context.runtime_duration_ms = metrics.duration_ms;
                     if status == Status::Timeout {
-                        message.push_str(format!(" Task timed out after {:.2} seconds", metrics.duration_ms / 1000.0).as_str());
+                        message.push_str(&format!(" Task timed out after {:.2} seconds", metrics.duration_ms / 1000.0));
                     }
                 }
                 // Remove the `request_id` since no more orphan logs will be processed with this one

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ impl LambdaProcessor {`
`164`	`164`	`if let Some(metrics) = metrics {`
`165`	`165`	`self.invocation_context.runtime_duration_ms = metrics.duration_ms;`
`166`	`166`	`if status == Status::Timeout {`
`167`		`- message.push_str(format!(" Task timed out after {:.2} seconds", metrics.duration_ms / 1000.0).as_str());`
	`167`	`+ message.push_str(&format!(" Task timed out after {:.2} seconds", metrics.duration_ms / 1000.0));`
`168`	`168`	`}`
`169`	`169`	`}`
`170`	`170`	// Remove the `request_id` since no more orphan logs will be processed with this one