Skip to content

Commit 75aa71c

Browse files
committed
only get and use GH App token if needed for sending orignating PR feedback
1 parent 9e8c6fa commit 75aa71c

File tree

2 files changed

+27
-5
lines changed

2 files changed

+27
-5
lines changed

.github/workflows/test-all-custom-flags.yml

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,17 @@ jobs:
2828
contents: read
2929
packages: read
3030
pull-requests: write
31+
outputs:
32+
send_pr_feedback: false
3133
steps:
34+
- name: Send PR feedback check
35+
id: set-send-pr-feedback
36+
run: |
37+
echo "send_pr_feedback=github.event.client_payload.origin_issue && vars.ENABLE_PR_FEEDBACK == 'true'" >> "$GITHUB_OUTPUT"
38+
3239
- name: Log in to Azure
3340
uses: bitwarden/gh-actions/azure-login@main
41+
if: steps.vars.outputs.send_pr_feedback == 'true'
3442
with:
3543
subscription_id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
3644
tenant_id: ${{ secrets.AZURE_TENANT_ID }}
@@ -39,16 +47,19 @@ jobs:
3947
- name: Get Azure Key Vault secrets
4048
id: get-kv-secrets
4149
uses: bitwarden/gh-actions/get-keyvault-secrets@main
50+
if: steps.vars.outputs.send_pr_feedback == 'true'
4251
with:
4352
keyvault: gh-org-bitwarden
4453
secrets: "BW-GHAPP-ID,BW-GHAPP-KEY"
4554

4655
- name: Log out from Azure
4756
uses: bitwarden/gh-actions/azure-logout@main
57+
if: steps.vars.outputs.send_pr_feedback == 'true'
4858

4959
- name: Generate GH App token
50-
uses: actions/create-github-app-token@30bf6253fa41bdc8d1501d202ad15287582246b4 # v2.0.3
5160
id: app-token
61+
uses: actions/create-github-app-token@30bf6253fa41bdc8d1501d202ad15287582246b4 # v2.0.3
62+
if: steps.vars.outputs.send_pr_feedback == 'true'
5263
with:
5364
app-id: ${{ steps.get-kv-secrets.outputs.BW-GHAPP-ID }}
5465
private-key: ${{ steps.get-kv-secrets.outputs.BW-GHAPP-KEY }}
@@ -180,7 +191,7 @@ jobs:
180191
# typical config case covered for success feedback
181192
- name: Communicate BIT failure on originating issue
182193
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
183-
if: failure() && github.event.client_payload.origin_issue && vars.ENABLE_PR_FEEDBACK == 'true'
194+
if: failure() && steps.vars.outputs.send_pr_feedback == 'true'
184195
with:
185196
github-token: ${{ steps.app-token.outputs.token }}
186197
script: |

.github/workflows/test-all.yml

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,17 @@ jobs:
2727
contents: read
2828
packages: read
2929
pull-requests: write
30+
outputs:
31+
send_pr_feedback: false
3032
steps:
33+
- name: Send PR feedback check
34+
id: set-send-pr-feedback
35+
run: |
36+
echo "send_pr_feedback=github.event.client_payload.origin_issue && vars.ENABLE_PR_FEEDBACK == 'true'" >> "$GITHUB_OUTPUT"
37+
3138
- name: Log in to Azure
3239
uses: bitwarden/gh-actions/azure-login@main
40+
if: steps.vars.outputs.send_pr_feedback == 'true'
3341
with:
3442
subscription_id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
3543
tenant_id: ${{ secrets.AZURE_TENANT_ID }}
@@ -38,16 +46,19 @@ jobs:
3846
- name: Get Azure Key Vault secrets
3947
id: get-kv-secrets
4048
uses: bitwarden/gh-actions/get-keyvault-secrets@main
49+
if: steps.vars.outputs.send_pr_feedback == 'true'
4150
with:
4251
keyvault: gh-org-bitwarden
4352
secrets: "BW-GHAPP-ID,BW-GHAPP-KEY"
4453

4554
- name: Log out from Azure
4655
uses: bitwarden/gh-actions/azure-logout@main
56+
if: steps.vars.outputs.send_pr_feedback == 'true'
4757

4858
- name: Generate GH App token
49-
uses: actions/create-github-app-token@30bf6253fa41bdc8d1501d202ad15287582246b4 # v2.0.3
5059
id: app-token
60+
uses: actions/create-github-app-token@30bf6253fa41bdc8d1501d202ad15287582246b4 # v2.0.3
61+
if: steps.vars.outputs.send_pr_feedback == 'true'
5162
with:
5263
app-id: ${{ steps.get-kv-secrets.outputs.BW-GHAPP-ID }}
5364
private-key: ${{ steps.get-kv-secrets.outputs.BW-GHAPP-KEY }}
@@ -180,7 +191,7 @@ jobs:
180191
181192
- name: Communicate BIT failure on originating issue
182193
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
183-
if: failure() && github.event.client_payload.origin_issue && vars.ENABLE_PR_FEEDBACK == 'true'
194+
if: failure() && steps.vars.outputs.send_pr_feedback == 'true'
184195
with:
185196
github-token: ${{ steps.app-token.outputs.token }}
186197
script: |
@@ -206,7 +217,7 @@ jobs:
206217
207218
- name: Communicate BIT success on originating issue
208219
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
209-
if: success() && github.event.client_payload.origin_issue && vars.ENABLE_PR_FEEDBACK == 'true'
220+
if: success() && steps.vars.outputs.send_pr_feedback == 'true'
210221
with:
211222
github-token: ${{ steps.app-token.outputs.token }}
212223
script: |

0 commit comments

Comments
 (0)