[CAI-758] docker as user#2009
Merged
marcobottaro merged 7 commits intoCAI-629-refactor-chatbotfrom Feb 13, 2026
Merged
Conversation
🦋 Changeset detectedLatest commit: d866526 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the chatbot-related Dockerfiles and local compose configs to run containers as a non-root user to reduce security risk.
Changes:
- Add non-root user setup (
appuser/ UID 1000) and switchUSERin multiple Dockerfiles - Update docker-compose AWS credentials mount path from
/root/.awsto/home/appuser/.aws - Add a changeset bumping patched packages
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/chatbot/docker/sqs_to_lambda.Dockerfile | Create/switch to non-root user for SQS listener container |
| apps/chatbot/docker/compose.yaml | Mount AWS credentials under non-root user path |
| apps/chatbot/docker/compose.test.yaml | Mount AWS credentials under non-root user path in test compose |
| apps/chatbot/docker/app.test.Dockerfile | Create/switch to non-root user for test image |
| apps/chatbot/docker/app.local.Dockerfile | Create/switch to non-root user for local dev image |
| apps/chatbot/docker/app.Dockerfile | Switch Lambda image runtime to UID 1000 |
| apps/chatbot-monitor/docker/lambda_monitor.test.Dockerfile | Switch Lambda test image runtime to UID 1000 |
| apps/chatbot-monitor/docker/lambda_monitor.Dockerfile | Switch Lambda image runtime to UID 1000 |
| apps/chatbot-index/docker/lambda_refresh_index.test.Dockerfile | Switch Lambda test image runtime to UID 1000 |
| apps/chatbot-index/docker/lambda_refresh_index.Dockerfile | Switch Lambda image runtime to UID 1000 |
| apps/chatbot-index/docker/create_index.Dockerfile | Define task root + create/switch to non-root user |
| apps/chatbot-evaluate/docker/lambda.Dockerfile | Switch Lambda image runtime to UID 1000 |
| .changeset/solid-areas-marry.md | Patch releases for affected packages |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mdciri
approved these changes
Feb 10, 2026
Contributor
Jira Pull Request LinkThis Pull Request refers to the following Jira issue CAI-758 |
marcobottaro
approved these changes
Feb 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
List of Changes
Run dockerfile as normal user
Motivation and Context
Improve security, no need to be root.
How Has This Been Tested?
locally. Production environment Dockerfiles will be tested after deploy.
Screenshots (if appropriate):
Types of changes
Checklist: