Fix Slack Notifications (#216)

MatKuhr · web-flow · commit 5f4d5f853036 · 2024-12-03T10:30:13.000Z
diff --git a/.github/workflows/continuous-integration.yaml b/.github/workflows/continuous-integration.yaml
@@ -74,10 +74,9 @@ jobs:
         if: ${{ github.ref == 'refs/heads/main' && failure() }}
         uses: slackapi/slack-github-action@v2.0.0
         with:
+          webhook: ${{ secrets.SLACK_WEBHOOK }}
+          webhook-type: incoming-webhook
           payload: |
             {
               "text": "🚨 Main Build Failed! 😬 Please inspect & fix by clicking <https://github.com/SAP/ai-sdk-java/actions/runs/${{ github.run_id }}|here>"
             }
-        env:
-          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}
-          SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK
diff --git a/.github/workflows/e2e-test.yaml b/.github/workflows/e2e-test.yaml
@@ -71,10 +71,10 @@ jobs:
         if: failure()
         uses: slackapi/slack-github-action@v2.0.0
         with:
+          webhook: ${{ secrets.SLACK_WEBHOOK }}
+          webhook-type: incoming-webhook
           payload: |
             {
               "text": "⚠️ End-to-end tests failed! 😬 Please inspect & fix by clicking <https://github.com/SAP/ai-sdk-java/actions/runs/${{ github.run_id }}|here>"
             }
-        env:
-          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}
-          SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK
+
diff --git a/.github/workflows/fosstars-report.yml b/.github/workflows/fosstars-report.yml
@@ -77,10 +77,9 @@ jobs:
         if: failure()
         uses: slackapi/slack-github-action@v2.0.0
         with:
+          webhook: ${{ secrets.SLACK_WEBHOOK }}
+          webhook-type: incoming-webhook
           payload: |
             {
               "text": "⚠️ OWASP Dependency check failed! 😬 Please inspect & fix by clicking <https://github.com/SAP/ai-sdk-java/actions/runs/${{ github.run_id }}|here>"
             }
-        env:
-          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}
-          SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK

Original file line number	Diff line number	Diff line change
`@@ -74,10 +74,9 @@ jobs:`
`74`	`74`	`if: ${{ github.ref == 'refs/heads/main' && failure() }}`
`75`	`75`	`uses: slackapi/[email protected]`
`76`	`76`	`with:`
	`77`	`+ webhook: ${{ secrets.SLACK_WEBHOOK }}`
	`78`	`+ webhook-type: incoming-webhook`
`77`	`79`	`payload: \|`
`78`	`80`	`{`
`79`	`81`	`"text": "🚨 Main Build Failed! 😬 Please inspect & fix by clicking <https://github.com/SAP/ai-sdk-java/actions/runs/${{ github.run_id }}\|here>"`
`80`	`82`	`}`
`81`		`- env:`
`82`		`- SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}`
`83`		`- SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK`
Original file line number	Diff line number	Diff line change
`@@ -71,10 +71,10 @@ jobs:`
`71`	`71`	`if: failure()`
`72`	`72`	`uses: slackapi/[email protected]`
`73`	`73`	`with:`
	`74`	`+ webhook: ${{ secrets.SLACK_WEBHOOK }}`
	`75`	`+ webhook-type: incoming-webhook`
`74`	`76`	`payload: \|`
`75`	`77`	`{`
`76`	`78`	`"text": "⚠️ End-to-end tests failed! 😬 Please inspect & fix by clicking <https://github.com/SAP/ai-sdk-java/actions/runs/${{ github.run_id }}\|here>"`
`77`	`79`	`}`
`78`		`- env:`
`79`		`- SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}`
`80`		`- SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK`
	`80`	`+`
Original file line number	Diff line number	Diff line change
`@@ -77,10 +77,9 @@ jobs:`
`77`	`77`	`if: failure()`
`78`	`78`	`uses: slackapi/[email protected]`
`79`	`79`	`with:`
	`80`	`+ webhook: ${{ secrets.SLACK_WEBHOOK }}`
	`81`	`+ webhook-type: incoming-webhook`
`80`	`82`	`payload: \|`
`81`	`83`	`{`
`82`	`84`	`"text": "⚠️ OWASP Dependency check failed! 😬 Please inspect & fix by clicking <https://github.com/SAP/ai-sdk-java/actions/runs/${{ github.run_id }}\|here>"`
`83`	`85`	`}`
`84`		`- env:`
`85`		`- SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}`
`86`		`- SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK`