Skip to content

Conversation

@kilemensi
Copy link
Member

Description

This PR upgrades all apps to use build secrets when building Docker images. This should ensure all sensitive ENV vars are not included in the final built images.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Screenshots

N/A

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

@kilemensi kilemensi self-assigned this Sep 15, 2025
@kilemensi kilemensi added the bug Something isn't working label Sep 15, 2025
@github-project-automation github-project-automation bot moved this to 🚧 In Progress in COMMONS Sep 15, 2025
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

  + There are issues with passing secrets in reusable workflow
  + Need to find better way of passing secrets without making the
    reusable workflow aware of each and every secret name
@kilemensi kilemensi requested a review from a team September 15, 2025 13:36
@github-actions
Copy link
Contributor

Latest updated Preview URL

Name Review
codeforafrica-ui-pr-1274 Visit

Copy link
Contributor

@koechkevin koechkevin left a comment

Choose a reason for hiding this comment

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

Tested this with Codeforafrica and charteafrica

Copy link
Contributor

@thepsalmist thepsalmist left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@kilemensi kilemensi added this pull request to the merge queue Sep 16, 2025
Merged via the queue into main with commit b72e690 Sep 16, 2025
7 checks passed
@kilemensi kilemensi deleted the fix_docker_build_security branch September 16, 2025 10:33
@github-project-automation github-project-automation bot moved this from 🚧 In Progress to ✅ Done in COMMONS Sep 16, 2025
@kilemensi kilemensi mentioned this pull request Sep 16, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

6 participants