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

Remove build golang lambda #222

Open
wchaws opened this issue Feb 24, 2023 · 0 comments · May be fixed by #535
Open

Remove build golang lambda #222

wchaws opened this issue Feb 24, 2023 · 0 comments · May be fixed by #535
Labels
enhancement New feature or request

Comments

@wchaws
Copy link
Contributor

wchaws commented Feb 24, 2023

Issue

Currently, this repository is experiencing frequent security alerts, particularly stemming from the Go(lang) codebase, which is proving to be quite disruptive.

Solutions thoughts

Instead of compiling Skopeo from scratch, consider downloading the Skopeo CLI tool from the official source during the execution of the Custom Resource Lambda. However, it's worth noting that the Skopeo official repository does not offer any static binaries, as outlined in this document.

Another approach could involve creating a separate repository dedicated to providing pre-built lambda function binaries, similar to cdklabs/awscdk-asset-kubectl. By doing so, the security issue would transit to the separate repository, although it would still require ongoing efforts to maintain.

Another approach is get rid of skopeo, use crane instead. Since crane has prebuilt static binary. But if we run crane in lambda it encounter Error: mkdir /home/sbx_user1051: read-only file system error. But we can solve this problem via set env DOCKER_CONFIG=/tmp/.docker in lambda env.

@wchaws wchaws pinned this issue Aug 30, 2023
@wchaws wchaws linked a pull request Feb 20, 2024 that will close this issue
6 tasks
@mrgrain mrgrain added the enhancement New feature or request label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants