Skip to content

Conversation

@bilindhajer
Copy link
Contributor

@bilindhajer bilindhajer commented Jul 21, 2025

  • Removes:
    • aws-sdk as a runtime dependency.
    • All node support prior to version 20.
  • Adds:
    • @aws-sdk/client-cloudformation as a runtime dependency.
  • Updates:
    • bin/validate-template to use @aws-sdk/client-cloudformation.
    • Lambda shortcut lambda runtime default to use node22.x

@bilindhajer bilindhajer requested a review from a team as a code owner July 21, 2025 17:17
@bilindhajer bilindhajer marked this pull request as draft July 21, 2025 17:20
@bilindhajer bilindhajer marked this pull request as ready for review July 21, 2025 19:12
},
Description: 'my description',
Environment: { Variables: { A: 'a' } },
Environment: { Variables: { MyCoolEnv: 'a' } },
Copy link
Contributor Author

Choose a reason for hiding this comment

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

new cfn-lint is catching these as invalid environments thus we shouldnt test with these

Handler: 'index.something',
KmsKeyArn: 'arn:aws:kms:us-east-1:123456789012:key/fake',
Layers: ['arn:aws:fake:layer/abc'],
Layers: ['arn:aws:lambda:us-east-2:590474943231:layer:AWS-Parameters-and-Secrets-Lambda-Extension:4'],
Copy link
Contributor Author

@bilindhajer bilindhajer Jul 21, 2025

Choose a reason for hiding this comment

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

new cfn-lint is catching these as invalid layer arn thus we shouldnt test with these

TracingConfig: { Mode: 'Active' },
VpcConfig: {
SecurityGroupIds: ['sg-12345'],
SecurityGroupIds: ['sg-12345678'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

new cfn-lint is catching these as invalid security groups thus we shouldnt test with these

LogicalName: 'MyQueue',
VisibilityTimeout: 60,
maxReceiveCount: 100,
ContentBasedDeduplication: true,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

new cfn-lint is catching these as invalid thus we shouldnt test with these

aws-sam-cli==1.94.0
cfn-lint==0.78.1
aws-sam-cli==1.142.1
cfn-lint==1.36.1
Copy link
Contributor Author

Choose a reason for hiding this comment

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

latest dev dependencies used in tests

Copy link

Choose a reason for hiding this comment

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

nit: can we use the aws-sam-cli >= 1.142.1 to prevent this type of issue in the future?

},
"dependencies": {
"aws-sdk": "^2.1425.0",
"@aws-sdk/client-cloudformation": "^3.848.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removes aws-sdk and only use @aws-sdk/client-cloudformation

cmaster11
cmaster11 previously approved these changes Jul 22, 2025
@bilindhajer bilindhajer merged commit b7027fc into master Jul 22, 2025
6 checks passed
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.

3 participants