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

Added new hook events sent by Gitlab starting in 9.5.4 #641

Closed
wants to merge 1 commit into from
Closed

Added new hook events sent by Gitlab starting in 9.5.4 #641

wants to merge 1 commit into from

Conversation

jaraujoduarte
Copy link

Since the 9.5.4 upgrade of Gitlab, when testing the hooks from the "integrations" section, the "X-Gitlab-Event" with the event description is sent in the form "xxx Event" (i.e. "Push Event") instead of the regular "xxx Hook", showing a 404 error on the Gitlab side when the hook is actually working as Gitlab does send "xxx Hook" kind of headers when the actual event is triggered.

Although it looks like a bug/inconsistency from Gitlab, this change extends the list of supported headers to overcome this inconsistency.

@DieBauer
Copy link

Duplicate of #609?

@omehegan
Copy link
Member

Bug was fixed on GitLab side, see #608.

@omehegan omehegan closed this Nov 13, 2017
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.

3 participants