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

Run action under default github runner UID #43

Merged
merged 2 commits into from
Dec 8, 2023
Merged

Run action under default github runner UID #43

merged 2 commits into from
Dec 8, 2023

Conversation

ihulsbus
Copy link
Contributor

@ihulsbus ihulsbus commented Dec 5, 2023

This PR adds a USER directive to the dockerfile to set the runtime user to a non-root user and the same user as used by the github runners for actions runs. Closes #42

Copy link
Owner

@haya14busa haya14busa left a comment

Choose a reason for hiding this comment

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

LGTM

Dockerfile Show resolved Hide resolved
@haya14busa haya14busa merged commit 78ab5a1 into haya14busa:master Dec 8, 2023
5 checks passed
Copy link
Contributor

github-actions bot commented Dec 8, 2023

🚀 [bumpr] Bumped! New version:v1.10.0 Changes:v1.9.0...v1.10.0

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.

action runs under a different user
2 participants