Skip to content

Fix containers running as root in Dockerfiles#1048

Merged
StuartFerguson merged 2 commits intomasterfrom
copilot/codacy-issue-25bbb5b61fbc369b427e4148a8211785-fix
Mar 9, 2026
Merged

Fix containers running as root in Dockerfiles#1048
StuartFerguson merged 2 commits intomasterfrom
copilot/codacy-issue-25bbb5b61fbc369b427e4148a8211785-fix

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 9, 2026

Containers were running as root due to missing USER instructions — a high-severity security issue flagged by Codacy (missing-user-entrypoint).

Changes

  • SecurityService/Dockerfile — adds USER app before ENTRYPOINT
  • SecurityServiceTestUI/Dockerfile — same fix applied proactively
COPY --from=publish /app .
USER app
ENTRYPOINT ["dotnet", "SecurityService.dll"]

app is the non-root user built into Microsoft's .NET base images (≥ .NET 8).

Original prompt

This section details on the original issue you should resolve

<issue_title>[Codacy] High issue(s) in SecurityService/Dockerfile</issue_title>
<issue_description>Codacy detected 1 occurrence(s) of rule Semgrep_dockerfile.security.missing-user-entrypoint.missing-user-entrypoint:

  • High at SecurityService/Dockerfile:36 → By not specifying a USER, a program in the container may run as 'root'. This is a security hazard.

See full details in Codacy Report

Unique ID: codacy-issue-25bbb5b61fbc369b427e4148a8211785</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing user in Dockerfile for security compliance Fix containers running as root in Dockerfiles Mar 9, 2026
@StuartFerguson StuartFerguson marked this pull request as ready for review March 9, 2026 10:40
@StuartFerguson StuartFerguson merged commit 76d1501 into master Mar 9, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Codacy] High issue(s) in SecurityService/Dockerfile

2 participants