Skip to content

Slack notification functionality on fail#175

Merged
jeremylong merged 2 commits intodependency-check:masterfrom
glennvdv:master
Mar 3, 2020
Merged

Slack notification functionality on fail#175
jeremylong merged 2 commits intodependency-check:masterfrom
glennvdv:master

Conversation

@glennvdv
Copy link
Copy Markdown

@glennvdv glennvdv commented Feb 7, 2020

This add's the ability to send a slack notification to a slack webhook, when the dependency analyze fails

@glennvdv glennvdv closed this Feb 10, 2020
@glennvdv glennvdv reopened this Feb 10, 2020
@jeremylong
Copy link
Copy Markdown
Contributor

Hey - thanks for the PR! I will be merging this before the next release. I just need to update the documentation in the primary ODC repo - I'll be leaving this open as a reminder until I get the documentation updated.

@jeremylong jeremylong added this to the 5.3.1 milestone Feb 10, 2020
@jeremylong
Copy link
Copy Markdown
Contributor

Just to make sure I am creating the correct documentation - for this to work people would have to create their own web hook for slack to receive and post these messages?

@glennvdv
Copy link
Copy Markdown
Author

glennvdv commented Mar 3, 2020

Yes, you have to create an incoming webhook and you have to specify that in the slack configuration block. In this example the slackweb hook url is a system propertie
dependencyCheck { format = 'ALL' failBuildOnCVSS = 7 suppressionFiles = ["$rootDir/gradle/owasp/exclude.xml"] outputDirectory="build/reports/owasp" slack { def slackWebhookUrl = project.gradle.startParameter.systemPropertiesArgs['slackWebhookUrl'] enabled = slackWebhookUrl?true:false webhookUrl = slackWebhookUrl } }

jeremylong added a commit to dependency-check/DependencyCheck that referenced this pull request Mar 3, 2020
@jeremylong jeremylong merged commit 61dbe75 into dependency-check:master Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants