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
feat: Allow legacy
option for lambdaHashingVersion
property
#10173
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10173 +/- ##
==========================================
- Coverage 85.37% 85.34% -0.03%
==========================================
Files 339 339
Lines 13819 13822 +3
==========================================
- Hits 11798 11797 -1
- Misses 2021 2025 +4
Continue to review full report at Codecov.
|
940137b
to
494d00c
Compare
@@ -147,7 +148,7 @@ class AwsCompileFunctions { | |||
|
|||
async addFileToHash(filePath, hash) { | |||
const lambdaHashingVersion = this.serverless.service.provider.lambdaHashingVersion; | |||
if (lambdaHashingVersion) { | |||
if (lambdaHashingVersion === '20201221') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Conceptually we should rather use >=
comparison (although it's highly unlikely we will introduce any other mode in a future)
@@ -62,7 +62,8 @@ class AwsCompileFunctions { | |||
'LAMBDA_HASHING_VERSION_V2', | |||
'Resolution of lambda version hashes was improved with better algorithm, ' + | |||
'which will be used in next major release.\n' + | |||
'Switch to it now by setting "provider.lambdaHashingVersion" to "20201221"' | |||
'Switch to it now by setting "provider.lambdaHashingVersion" to "20201221".\n' + | |||
'While it is highly encouraged to upgrade to new algorithm, you can still use the old approach by setting "provider.lambdaHashingVersion" to "legacy".' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
legacy
should be updated to 20200924
494d00c
to
2c74f72
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great 👍
Allow to use
legacy
option forlambdaHashingVersion
to keep using the old algorithmReported internally