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

control-service: remove executables from secure job builder #1202

Merged
merged 4 commits into from
Oct 3, 2022

Conversation

mivanov1988
Copy link
Contributor

Why: In order to reduce the attack surface, we need to remove all unnecessary system executables after the job building.

What: The change aims to remove all left and unnecessary executables after job building - chown, uname and pip.

Testing done: built image locally.

Signed-off-by: Miroslav Ivanov miroslavi@vmware.com

Copy link
Contributor

@gageorgiev gageorgiev left a comment

Choose a reason for hiding this comment

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

What about chmod? Also, fix the name of your PR.

@mivanov1988 mivanov1988 changed the title Person/miroslavi/secure job builder control-service: secure job builder cleanup Oct 3, 2022
@mivanov1988 mivanov1988 changed the title control-service: secure job builder cleanup control-service: remove executables from secure job builder Oct 3, 2022
@mivanov1988
Copy link
Contributor Author

mivanov1988 commented Oct 3, 2022

What about chmod? Also, fix the name of your PR.

The base image contains only chown, uname, sh, and python.

@mivanov1988 mivanov1988 merged commit a6c4cfb into main Oct 3, 2022
@mivanov1988 mivanov1988 deleted the person/miroslavi/secure-job-builder branch October 3, 2022 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants