-
Notifications
You must be signed in to change notification settings - Fork 886
feat: add iron bank Dockerfile & manifest #5934
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
Conversation
Co-authored-by: Dean Sheather <[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.
I don't want this to be a top-level folder. Let's put it inside of scripts
.
|
@deansheather I don't think there's a point in creating a new folder type for this because it's so bespoke. |
prettier is returning the following error:
@deansheather let me know the best path forward to resolve. |
@ericpaulsen I'd try checking out |
We should merge this even with all of the vulnerabilities reported by trivy, and focus on fixing them after merge. Some of them are vulnerabilities in the base image which we might not be able to resolve. |
the requested changes were made.
adds Iron Bank-compatible Dockerfile & hardening manifest. this will be the master copy that we will copy into the Iron Bank repo.