Skip to content

Commit d17f970

Browse files
saibamoclaude
andcommitted
fix: install sofahutils in the image; drop CI token
The consolidation moved json_logger onto sofahutils, but the Dockerfile never installed it -> the image would fail at import. Add the public install. CI drops the token secret too. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent a607e74 commit d17f970

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,12 @@ jobs:
1313
with:
1414
python-version: "3.12"
1515
- name: Install dependencies
16-
env:
17-
SOFAH_GH_TOKEN: ${{ secrets.SOFAH_GH_TOKEN }}
1816
run: |
1917
python -m pip install --upgrade pip
2018
pip install pytest pytest-cov ruff bandit
2119
pip install flask pytz requests
22-
# sofahutils is a private dependency; CI installs it with an org/repo token secret.
23-
pip install "git+https://x-access-token:${SOFAH_GH_TOKEN}@github.com/sofahd/sofahutils.git"
20+
# sofahutils (public repo, no token needed)
21+
pip install "git+https://github.com/sofahd/sofahutils.git"
2422
- name: Lint (ruff, informational)
2523
run: ruff check .
2624
continue-on-error: true

Dockerfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@ RUN apk --no-cache -U add \
2222
waitress \
2323
requests \
2424
cryptography \
25-
pytz && \
25+
pytz \
26+
git+https://github.com/sofahd/sofahutils.git && \
2627
cd /home/api && \
2728
mkdir log_data && \
2829
chown api:api -R /home/api/*

0 commit comments

Comments
 (0)