mattermost-github-integration

mattermost-github-integration copied to clipboard

The initial ping from GHE will fail with organizational hooks

1

The cause is rather simply that [server.py:41](https://github.com/softdevteam/mattermost-github-integration/blob/master/mattermostgithub/server.py#L41) tries to access a `repository` entry. The fix can be equally simple. ```diff diff --git a/mattermostgithub/server.py b/mattermostgithub/server.py index ad26bec..9daee73 100644 --- a/mattermostgithub/server.py +++...

jcaesar

`AssertionError: View function mapping is overwriting an existing endpoint function: root`

2

``` Traceback (most recent call last): File "mattermostgithub/server.py", line 17, in @app.route(config.SERVER['hook'] or "/", methods=['POST']) File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1080, in decorator self.add_url_rule(rule, endpoint, f, **options) File "/usr/lib/python2.7/dist-packages/flask/app.py", line 64, in...

bwesterb

always get bad request response before any code in def root() runs

3

I tried adding a print as the first line of def root(): so im running from docker Ive connected into the running docker container if I run wget --post-data="some=data" localhost:5000...

christopherreay

Feature request: ensure repo name is always in notification

1

Sometimes when a comment is left against an individual commit (maybe without a PR being raised?), the repo that commit came from is not shown in the notification, e.g.: ```...

snim2

Sometimes MM notifications are slow to appear because GH has DNS problems, or other status issues. It would be nice to check [GH status](https://status.github.com/api) periodically and notify users when GH...

snim2