Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Add new Lambda config LAMBDA_LIMITS_CODE_SIZE_ZIPPED #933

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

joe4dev
Copy link
Member

@joe4dev joe4dev commented Nov 23, 2023

Follow up of localstack/localstack#9697

Motivation

As detected in snapshot updates and implemented in localstack/localstack#9697, AWS changed the behavior related to Lambda limits. Therefore, we should pro-actively document the option LAMBDA_LIMITS_CODE_SIZE_ZIPPED because everyone who used LAMBDA_LIMITS_CREATE_FUNCTION_REQUEST_SIZE, likey needs LAMBDA_LIMITS_CODE_SIZE_ZIPPED instead or in addition.

Further clarifications in the localstack PR, including this comment replying to @dfangl question localstack/localstack#9697 (comment)

My guess: The previous error message was too confusing. People uploaded large zips and got a message about request size:
Request must be smaller than 70167211 bytes for the CreateFunction operation

Now they check the zip (if relevant) before checking the request size limit leading to a clearer message:
Zipped size must be smaller than 52428800 bytes

Changes

  • Change the default of LAMBDA_LIMITS_CREATE_FUNCTION_REQUEST_SIZE following an AWS parity update
  • Add the configuration LAMBDA_LIMITS_CODE_SIZE_ZIPPED to the documentation because that it now checked first after a recent AWS update. Therefore, users who currently adjusted LAMBDA_LIMITS_CREATE_FUNCTION_REQUEST_SIZE, likely need LAMBDA_LIMITS_CODE_SIZE_ZIPPED as well

@joe4dev joe4dev self-assigned this Nov 23, 2023
Copy link

github-actions bot commented Nov 23, 2023

🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-933.surge.sh 🎊

Copy link
Member

@dfangl dfangl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for documenting!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants