-
Notifications
You must be signed in to change notification settings - Fork 9
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
Use distroless/static image instead of distroless/base since glibc is not used #49
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Vilmos Martinek <[email protected]>
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.
ah nice @vilmosmartinek, this looks like a good improvement to make. Thank you for taking the initiative to do so 🙇♂️ - it's actually particularly on my mind as I just finished writing up the security self-assessment in crossplane/crossplane#6143 😉
I see that you've opened similar PRs across a few other function repos and also the template repo (crossplane/function-template-go#88), thank you for being thorough.
Let's see if we can get these all merged and new versions of the functions published.
Were you able to do some local testing already to build with this change and verify the function works well still? I don't think there's much in the way of e2e tests here that would test that automatically...
Thank you for taking the time to review my PRs! I appreciate your kind words. I also had a chance to check out the security self-assessment you wrote in crossplane/crossplane#6143, and I must say it is very well done. Great work! One note, you mention the Responsible Disclosures Process defined in the security
We have a couple of Crossplane Functions built in-house that are already using the I can make a similar "smoke test" here as well by rendering the example composition. |
As a smoke test I've rendered the example in this repository using freshly built images of both
@jbw976 Let me know if you can think of anything else I can / should test! |
Description of your changes
This pull request changes the base image of the function to
gcr.io/distroless/static-debian12:nonroot
.Fixes #48
I have:
- [ ] Added or updated unit tests for my change.