Skip to content

Commit d54058c

Browse files
fix: address CI lint and clarify worker-exit log
- update concurrent worker exit warning per review nit - drop redundant .cloned() in ApiGatewayV2 cookies mapping (clippy lint was pre-existing and unrelated to this PR)
1 parent f4b6e71 commit d54058c

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

lambda-http/src/response.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,6 @@ impl LambdaResponse {
9797
let cookies = headers
9898
.get_all(SET_COOKIE)
9999
.iter()
100-
.cloned()
101100
.map(|v| v.to_str().ok().unwrap_or_default().to_string())
102101
.collect();
103102
headers.remove(SET_COOKIE);

lambda-runtime/src/runtime.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -209,10 +209,13 @@ where
209209
Ok(Ok(())) => {
210210
// `concurrent_worker_loop` runs indefinitely, so an Ok return indicates
211211
// an unexpected worker exit; we still decrement because the task is gone.
212-
warn!(remaining_workers, "Concurrent worker exited unexpectedly without error");
212+
warn!(
213+
remaining_workers,
214+
"Concurrent worker exited cleanly (unexpected - loop should run forever)"
215+
);
213216
if first_error.is_none() {
214217
first_error = Some(Box::new(io::Error::other(
215-
"all concurrent workers exited unexpectedly without error",
218+
"all concurrent workers exited cleanly (unexpected - loop should run forever)",
216219
)));
217220
}
218221
}

0 commit comments

Comments
 (0)