api-screenshot icon indicating copy to clipboard operation
api-screenshot copied to clipboard

fix: force nodejs 12

Open slorber opened this issue 3 years ago • 1 comments

I couldn't make this repo run by default on Netlify without downgrading Node.js versions.

14/16 didn't work, but 12 worked

I suggest specifying a version so that others trying to deploy it do not have to figure things out

Related:

  • https://stackoverflow.com/questions/66214552/tmp-chromium-error-while-loading-shared-libraries-libnss3-so-cannot-open-sha/67117619#67117619
  • https://answers.netlify.com/t/lambda-function-failing-in-netlify-build-20-3-1/48900
  • https://answers.netlify.com/t/chrome-aws-lambda-stopped-working-possibly-due-to-different-version-of-netlify-build/48847

slorber avatar Aug 26 '22 14:08 slorber

Deploy Preview for eleventy-api-screenshot ready!

Name Link
Latest commit
Latest deploy log https://app.netlify.com/sites/eleventy-api-screenshot/deploys/637ecb9089f6410244e8f08f
Deploy Preview https://deploy-preview-13--eleventy-api-screenshot.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Aug 26 '22 14:08 netlify[bot]

I think the intent here is good but unfortunately the code doesn’t quite work—I don’t believe that you can set environment variables for aws lambda/netlify functions via your netlify.toml

https://answers.netlify.com/t/netlify-functions-and-env-variables-from-netlify-toml/4404/28

These need to be set in Netlify UI instead—which unfortunately means will require an additional README step.

Thank you for the report!

zachleat avatar Nov 24 '22 01:11 zachleat