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.

Sign up for GitHub

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

Jump to bottom

Run container as non-root user #991

Merged
merged 2 commits into from
Dec 13, 2023
Merged

Run container as non-root user #991

merged 2 commits into from
Dec 13, 2023

Conversation

hectorgomezv
Copy link
Member

Changes:

  • Create, assign permissions, and run the application Docker container using a non-root user.

@hectorgomezv hectorgomezv self-assigned this Dec 13, 2023
@hectorgomezv hectorgomezv requested a review from a team as a code owner December 13, 2023 09:13
@coveralls
Copy link

coveralls commented Dec 13, 2023 •
edited
Loading

Pull Request Test Coverage Report for Build 7194372882

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.772%
Totals Coverage Status
Change from base Build 7183460605: 0.0%
Covered Lines: 4823
Relevant Lines: 4834
💛 - Coveralls

@luarx luarx requested a review from Uxio0 December 13, 2023 09:43
@fmrsabino fmrsabino added the in review Someone is reviewing this Pull Request label Dec 13, 2023
fmrsabino
fmrsabino reviewed Dec 13, 2023
View reviewed changes
Dockerfile Outdated Show resolved Hide resolved
@hectorgomezv @fmrsabino
Update Dockerfile
c4c2eae 
Co-authored-by: Frederico Sabino <3332770+fmrsabino@users.noreply.github.com>
fmrsabino
fmrsabino approved these changes Dec 13, 2023
View reviewed changes
Copy link
Contributor

@fmrsabino fmrsabino left a comment

Choose a reason for hiding this comment

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

Awesome! 👏

Dockerfile Show resolved Hide resolved
@fmrsabino fmrsabino removed the in review Someone is reviewing this Pull Request label Dec 13, 2023
@hectorgomezv hectorgomezv merged commit 3af372a into main Dec 13, 2023
15 checks passed
@hectorgomezv hectorgomezv deleted the non-root-container branch December 13, 2023 11:44
@github-actions github-actions bot locked and limited conversation to collaborators Dec 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fmrsabino fmrsabino fmrsabino approved these changes

@Uxio0 Uxio0 Awaiting requested review from Uxio0

Assignees

@hectorgomezv hectorgomezv

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hectorgomezv @coveralls @fmrsabino