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

Support for semicolon and new --insecure option #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

couloum
Copy link

@couloum couloum commented Dec 11, 2018

Hi,

I suggest a small patch for 1 bug fix and 1 new feature.

BUG FIX

The bug is described here: #13

I juste changed payload to use application/json content-type instead of
application/x-www-form-urlencoded. With this method, I observed that
caracters like semicolon are working.
This is one of the 2 accepted format from what is described in Mattermost documentation: https://docs.mattermost.com/developer/webhooks-incoming.html

New feature

I added a new --insecure option to bypass SSL cert check. I had this issue where we used a self-signed certificate to test mattermost. This could be useful for other people.

`application/x-www-form-urlencoded`. With this method, I observed that
caracters like semicolon are working.
Added a new `--insecure` option to bypass SSL cert check.
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.

1 participant