Skip to content

Commit dad30ce

Browse files
authored
ci(release): switch token mint to client-id var (#721)
The `RELEASE_APP_ID` org secret was deleted in favor of a `RELEASE_APP_CLIENT_ID` org variable (#701). This swaps every workflow that mints a release App token to the new variable and updates the plan doc to match. Files touched: - `.github/workflows/release-rotation.yml` - `.github/workflows/release-permissions-check.yml` - `.github/workflows/ship-release.yml` - `.github/workflows/build-patch.yml` - `.github/workflows/build-release.yml` - `docs/release-automation-plan.md` `build-patch.yml` and `build-release.yml` had already moved to the `client-id:` input but were still reading the deleted `secrets.RELEASE_APP_ID` — this fixes that half-migrated state. Private-key references are unchanged.
1 parent 4d8f345 commit dad30ce

6 files changed

Lines changed: 12 additions & 10 deletions

File tree

.github/workflows/build-patch.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ jobs:
7373
id: app-token
7474
uses: actions/create-github-app-token@v3.1
7575
with:
76-
client-id: ${{ secrets.RELEASE_APP_ID }}
76+
client-id: ${{ vars.RELEASE_APP_CLIENT_ID }}
7777
private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }}
7878
owner: openemr
7979
repositories: openemr

.github/workflows/build-release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ jobs:
7373
id: app-token
7474
uses: actions/create-github-app-token@v3.1
7575
with:
76-
client-id: ${{ secrets.RELEASE_APP_ID }}
76+
client-id: ${{ vars.RELEASE_APP_CLIENT_ID }}
7777
private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }}
7878
owner: openemr
7979
repositories: openemr

.github/workflows/release-permissions-check.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
name: Release Permissions Check
22

33
# Manual probe of the release App's installed permissions on this repo.
4-
# Mints an App token from RELEASE_APP_ID + RELEASE_APP_PRIVATE_KEY and
4+
# Mints an App token from the RELEASE_APP_CLIENT_ID org variable +
5+
# RELEASE_APP_PRIVATE_KEY org secret and
56
# exercises only what tools/release/ rotation needs (per docs/release-automation-plan.md).
67
# Run after installing the App and after secrets rotations.
78

@@ -32,9 +33,9 @@ jobs:
3233

3334
- name: Mint release App token
3435
id: app-token
35-
uses: actions/create-github-app-token@v1
36+
uses: actions/create-github-app-token@v3.1
3637
with:
37-
app-id: ${{ secrets.RELEASE_APP_ID }}
38+
client-id: ${{ vars.RELEASE_APP_CLIENT_ID }}
3839
private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }}
3940

4041
- name: Probe — installation includes this repo

.github/workflows/release-rotation.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,9 @@ jobs:
4242
steps:
4343
- name: Mint release App token
4444
id: app-token
45-
uses: actions/create-github-app-token@v1
45+
uses: actions/create-github-app-token@v3.1
4646
with:
47-
app-id: ${{ secrets.RELEASE_APP_ID }}
47+
client-id: ${{ vars.RELEASE_APP_CLIENT_ID }}
4848
private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }}
4949

5050
- name: Checkout

.github/workflows/ship-release.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,9 @@ jobs:
4444
steps:
4545
- name: Mint release App token
4646
id: app-token
47-
uses: actions/create-github-app-token@v1
47+
uses: actions/create-github-app-token@v3.1
4848
with:
49-
app-id: ${{ secrets.RELEASE_APP_ID }}
49+
client-id: ${{ vars.RELEASE_APP_CLIENT_ID }}
5050
private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }}
5151
owner: openemr
5252
repositories: |

docs/release-automation-plan.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,8 @@ host most of the rewrite logic.
9393
## Permissions self-check
9494

9595
`.github/workflows/release-permissions-check.yml` (manual `workflow_dispatch`).
96-
Mints an App token from `RELEASE_APP_ID` + `RELEASE_APP_PRIVATE_KEY` and
96+
Mints an App token from the `RELEASE_APP_CLIENT_ID` org variable +
97+
`RELEASE_APP_PRIVATE_KEY` org secret and
9798
probes only what this repo's rotation workflow needs:
9899

99100
- `GET /installation/repositories` — confirm this repo is in the install list.

0 commit comments

Comments
 (0)