kopia lustrzana https://github.com/Aircoookie/WLED
Fix GitHub workflow secret access from forked PRs
Co-authored-by: netmindz <442066+netmindz@users.noreply.github.com>pull/4856/head
rodzic
624042d97e
commit
cd8ddb81e1
|
@ -1,12 +1,13 @@
|
||||||
name: Notify Discord on PR Merge
|
name: Notify Discord on PR Merge
|
||||||
on:
|
on:
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
pull_request:
|
pull_request_target:
|
||||||
types: [closed]
|
types: [closed]
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
notify:
|
notify:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
if: github.event.pull_request.merged == true
|
||||||
steps:
|
steps:
|
||||||
- name: Get User Permission
|
- name: Get User Permission
|
||||||
id: checkAccess
|
id: checkAccess
|
||||||
|
@ -23,11 +24,6 @@
|
||||||
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
|
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
|
||||||
echo "Job originally triggered by ${{ github.actor }}"
|
echo "Job originally triggered by ${{ github.actor }}"
|
||||||
exit 1
|
exit 1
|
||||||
- name: Checkout code
|
|
||||||
uses: actions/checkout@v3
|
|
||||||
with:
|
|
||||||
ref: ${{ github.event.pull_request.head.sha }} # This is dangerous without the first access check
|
|
||||||
- name: Send Discord notification
|
- name: Send Discord notification
|
||||||
# if: github.event.pull_request.merged == true
|
|
||||||
run: |
|
run: |
|
||||||
curl -H "Content-Type: application/json" -d '{"content": "Pull Request ${{ github.event.pull_request.number }} merged by ${{ github.actor }}"}' ${{ secrets.DISCORD_WEBHOOK_BETA_TESTERS }}
|
curl -H "Content-Type: application/json" -d '{"content": "Pull Request ${{ github.event.pull_request.number }} merged by ${{ github.actor }}"}' ${{ secrets.DISCORD_WEBHOOK_BETA_TESTERS }}
|
||||||
|
|
Ładowanie…
Reference in New Issue