aws-samples / aws-samples/devsecops-cicd-containers
Issue with
- Dominant language
- Python
- Stars
- 57
- Forks
- 50
- PR merge metrics
- No merged PRs in 30d
Description
As per the document -
[] New features and changes in AWS CLI version 2 - Binary parameters are passed as base64-encoded strings by default - https://docs.aws.amazon.com/cli/latest/userguide/cliv2-migration-changes.html#cliv2-migration-binaryparam
The below AWS CLI command will fail for customers using CodeBuild instances that uses AWS CLI version 2 -
https://github.com/aws-samples/devsecops-cicd-containers/blob/9da3f0ee8a37ff3481f9d3b1677decb126116e5b/buildspec-owasp-zap.yml#L37
Kindly keep an option to check for the cli versions and implement -
FOR CLI version 1.x
> aws lambda invoke --function-name ImpToSecurityHubEKS --payload file://payload.json owaspzap_scan_report.json && echo "LAMBDA_SUCCEDED" || echo "LAMBDA_FAILED";
FOR CLI version 2.x
> aws lambda invoke --function-name ImpToSecurityHubEKS --payload file://payload.json owaspzap_scan_report.json --cli-binary-format raw-in-base64-out && echo "LAMBDA_SUCCEDED" || echo "LAMBDA_FAILED";
Contributor guide
Assessment
This issue has not been assessed yet.