Skip to content

Commit 85d1cb8

Browse files
authored
Log function error (#892)
Bring back this log that was lost between version 0.10 and 0.11.
1 parent d03bbc4 commit 85d1cb8

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

lambda-runtime/src/layers/panic.rs

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,9 @@ where
5858
let fut = AssertUnwindSafe(task).catch_unwind();
5959
CatchPanicFuture::Future(fut, PhantomData)
6060
}
61-
Err(err) => {
62-
error!(error = ?err, "user handler panicked");
63-
CatchPanicFuture::Error(err)
61+
Err(error) => {
62+
error!(?error, "user handler panicked");
63+
CatchPanicFuture::Error(error)
6464
}
6565
}
6666
}
@@ -85,15 +85,19 @@ where
8585
match self.project() {
8686
CatchPanicFutureProj::Future(fut, _) => match fut.poll(cx) {
8787
Poll::Ready(ready) => match ready {
88-
Ok(inner_result) => Poll::Ready(inner_result.map_err(|err| err.into())),
89-
Err(err) => {
90-
error!(error = ?err, "user handler panicked");
91-
Poll::Ready(Err(Self::build_panic_diagnostic(&err)))
88+
Ok(Ok(success)) => Poll::Ready(Ok(success)),
89+
Ok(Err(error)) => {
90+
error!("{error:?}");
91+
Poll::Ready(Err(error.into()))
92+
}
93+
Err(error) => {
94+
error!(?error, "user handler panicked");
95+
Poll::Ready(Err(Self::build_panic_diagnostic(&error)))
9296
}
9397
},
9498
Poll::Pending => Poll::Pending,
9599
},
96-
CatchPanicFutureProj::Error(err) => Poll::Ready(Err(Self::build_panic_diagnostic(err))),
100+
CatchPanicFutureProj::Error(error) => Poll::Ready(Err(Self::build_panic_diagnostic(error))),
97101
}
98102
}
99103
}

0 commit comments

Comments
 (0)