Skip to content
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

Bug fix in lambda-secrerlayer [CDS-1486] #154

Merged
merged 1 commit into from
Aug 25, 2024
Merged

Conversation

guyrenny
Copy link
Contributor

@guyrenny guyrenny commented Aug 25, 2024

Change layer code to use nodejs18 code instead of 16 when running in lambdas with nodejs20

Description

Fixes #

How Has This Been Tested?

Checklist:

  • I have updated the versions in the changed module in the template index.js and package.json files.
  • I have updated the relevant component changelog(s)
  • This change does not affect module (e.g. it's readme file change)

@guyrenny guyrenny requested a review from royfur August 25, 2024 07:53
@guyrenny guyrenny merged commit 57a2492 into master Aug 25, 2024
10 checks passed
ryantanjunming pushed a commit that referenced this pull request Oct 11, 2024
ryantanjunming added a commit that referenced this pull request Oct 11, 2024
…ning on lambdas with nodejs20 (#154)"

This reverts commit 4219206.
ryantanjunming added a commit that referenced this pull request Oct 11, 2024
* change layer code to use nodejs18 code instead of 16 when running on lambdas with nodejs20 (#154)

Signed-off-by: Ryan Tan <[email protected]>

* resource type filter & AP3 region

Signed-off-by: Ryan Tan <[email protected]>

* IAM conditioning

Signed-off-by: Ryan Tan <[email protected]>

* sam validate fix, changed key

Signed-off-by: Ryan Tan <[email protected]>

* fix invocation comment

* boolean correction

* set boolean as strings instead

* Revert "change layer code to use nodejs18 code instead of 16 when running on lambdas with nodejs20 (#154)"

This reverts commit 4219206.

---------

Signed-off-by: Ryan Tan <[email protected]>
Co-authored-by: guyrenny <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants