Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: Slack Notifications #354

Open
vdice opened this issue Apr 10, 2017 · 2 comments
Open

Refactor: Slack Notifications #354

vdice opened this issue Apr 10, 2017 · 2 comments

Comments

@vdice
Copy link
Member

vdice commented Apr 10, 2017

We've currently swapped out all previous invocations of the Slack plugin with use of our own utility (located in this repo) after bumping into multiple issues with the former. One drawback w/ the utility (essentially a bash script using Slack's Incoming Webhooks API) and its use is it leads to verbose/ugly DSL.

Let's revisit and either get back to using the Slack Plugin (as it stands, even w/ latest 2.2 version, we still see errors thrown like this one) or bundle up said utility for use in a less verbose fashion.

@vdice
Copy link
Member Author

vdice commented Apr 10, 2017

I believe the latest version of the Slack Plugin now allows use of a Credentials ID secret for the slack token (whereas it didn't before, which was the first big motivator for using our own utility). It would be great to get back to use of the plugin as long as the error in the description is no longer hit.

@Cryptophobia
Copy link

This issue was moved to teamhephy/jenkins-jobs#3

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

No branches or pull requests

2 participants