Skip to content

Commit 2df6e2b

Browse files
authored
Merge pull request #53 from warestack/fix/github-app-auth-env-vars
fix: GitHub app auth env vars
2 parents 267e19f + 5435c37 commit 2df6e2b

3 files changed

Lines changed: 11 additions & 11 deletions

File tree

.github/workflows/eks-deploy.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,10 @@ jobs:
3333
AWS_ACCESS_KEY_ID: "${{ secrets.AWS_ACCESS_KEY_ID }}"
3434
AWS_SECRET_ACCESS_KEY: "${{ secrets.AWS_SECRET_ACCESS_KEY }}"
3535

36-
# GitHub OAuth secrets
37-
APP_NAME_GITHUB: "${{ secrets.APP_NAME_GITHUB }}"
38-
CLIENT_ID_GITHUB: "${{ secrets.CLIENT_ID_GITHUB }}"
39-
APP_CLIENT_SECRET: "${{ secrets.APP_CLIENT_SECRET }}"
36+
# GitHub App Configuration
37+
APP_NAME_GITHUB: "watchflow"
38+
APP_CLIENT_ID_GITHUB: "${{ secrets.APP_CLIENT_ID_GITHUB }}"
39+
APP_CLIENT_SECRET_GITHUB: "${{ secrets.APP_CLIENT_SECRET_GITHUB }}"
4040
PRIVATE_KEY_BASE64_GITHUB: "${{ secrets.PRIVATE_KEY_BASE64_GITHUB }}"
4141
WEBHOOK_SECRET_GITHUB: "${{ secrets.WEBHOOK_SECRET_GITHUB }}"
4242

@@ -95,8 +95,8 @@ jobs:
9595
--set image.repository=${{ env.IMAGE_REPOSITORY }} \
9696
--set image.tag=${{ env.IMAGE_TAG }} \
9797
--set secrets.APP_NAME_GITHUB=${{ env.APP_NAME_GITHUB }} \
98-
--set secrets.CLIENT_ID_GITHUB=${{ env.CLIENT_ID_GITHUB }} \
99-
--set secrets.APP_CLIENT_SECRET=${{ env.APP_CLIENT_SECRET }} \
98+
--set secrets.APP_CLIENT_ID_GITHUB=${{ env.APP_CLIENT_ID_GITHUB }} \
99+
--set secrets.APP_CLIENT_SECRET_GITHUB=${{ env.APP_CLIENT_SECRET_GITHUB }} \
100100
--set secrets.PRIVATE_KEY_BASE64_GITHUB=${{ env.PRIVATE_KEY_BASE64_GITHUB }} \
101101
--set secrets.WEBHOOK_SECRET_GITHUB=${{ env.WEBHOOK_SECRET_GITHUB }} \
102102
--set secrets.OPENAI_API_KEY=${{ env.OPENAI_API_KEY }} \

helm-chart/values.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,9 +100,9 @@ affinity: {}
100100

101101
# Application secrets configuration - only the required ones (no defaults in config.py)
102102
secrets:
103-
APP_NAME_GITHUB: ""
104-
CLIENT_ID_GITHUB: ""
105-
APP_CLIENT_SECRET: ""
103+
APP_NAME_GITHUB: "watchflow"
104+
APP_CLIENT_ID_GITHUB: ""
105+
APP_CLIENT_SECRET_GITHUB: ""
106106
PRIVATE_KEY_BASE64_GITHUB: ""
107107
WEBHOOK_SECRET_GITHUB: ""
108108
OPENAI_API_KEY: ""

src/core/config/settings.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,10 +126,10 @@ def validate(self) -> bool:
126126
errors.append("APP_NAME_GITHUB is required")
127127

128128
if not self.github.app_id:
129-
errors.append("CLIENT_ID_GITHUB is required")
129+
errors.append("APP_CLIENT_ID_GITHUB is required")
130130

131131
if not self.github.app_client_secret:
132-
errors.append("APP_CLIENT_SECRET is required")
132+
errors.append("APP_CLIENT_SECRET_GITHUB is required")
133133

134134
if not self.github.private_key:
135135
errors.append("PRIVATE_KEY_BASE64_GITHUB is required")

0 commit comments

Comments
 (0)