-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"bad deno executable" after pre-handler error #113
Comments
Are you running the executable locally? Generally that won't work, AWS Lambda runs Amazon Linux 1 which requires a compatible binary (that isn't compatible with many linux distros). You can test using docker-lambda. Or am I misunderstanding the issue? How are you getting this? |
Those logs came from cloudwatch, using the 1.4.3 deno-lambda-layer zip from the releases page. |
I think I understand the issue. It seems like the layer must be used as a layer, rather than as a function (I didn't think this was a requirement but perhaps it is now?). I am going to update the bootstrap to error properly:
Will be fixed in #115 (see diff from bootstrap) |
So, it's actually that Fixed in al2 PR: 789e625 |
Thank you very much for deno-lambda, it's really great! I just wanted to add some details of things I found this morning as I was digging into a rough edge I bumped into. I was getting this error
For other API calls that were designed to be faster (<30sec), everything is working fine. Here's one scenario example of Cloudwatch logs with more context;
|
Sorry I should've cherry-picked the fix for 1.6.0, that error message is a red herring! Hopefully we can get al2 in before 1.6.1, otherwise ensure the bootstrap fix make their way in to 1.6.1 anyhow. |
The boostrap
investigate
function seems to handle pre-handler error's badly. I had uncaught exception during setup, and this showed up in the logs afterwards.After fixing the error the lambda ran fine, so I don't think the deno executable is actually bad.
The text was updated successfully, but these errors were encountered: