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

Do not log as error a webhook with an invalid branch name #4779

Merged
merged 1 commit into from Oct 23, 2018

Conversation

@humitos
Copy link
Member

@humitos humitos commented Oct 18, 2018

Although this endpoint is deprecated, we still receive request on it and many of them are invalid and do not contains the branch name or similar. This is logging a lot under Sentry and kill our quota.

@humitos humitos requested a review from Oct 18, 2018
stsewd
stsewd approved these changes Oct 22, 2018
@humitos humitos merged commit cef5bc4 into master Oct 23, 2018
1 check passed
@humitos humitos deleted the humitos/webhook/no-log branch Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants