search

silvermine / serverless-plugin-cloudfront-lambda-edge

Adds Lambda@Edge support to Serverless
MIT License
296 stars 41 forks source link

Suggestion: Validate presence of `memorySize: 128` and `timeout: 1` #61

Closed skylarmb closed 3 years ago

skylarmb commented 3 years ago

It appears these must be set in order to use this plugin/lambda@edge in general otherwise you get an error on deploy (The function memory size is larger than the maximum allowed size...). It would be nice to catch this at the packaging stage with a simple validation

jthomerson commented 3 years ago

I'm not really in favor of this because we'd have to update these values any time AWS changes their limits, and then users have to upgrade to take advantage of it.