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

skip json parsing if allowedUser or ignoredUser are empty #84

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

stefan-mees
Copy link

We are running github enterprise, therefor the github payload cannot be parsed cause the payload has an additional property enterprise. This pull requests introduces skipping of json parsing if no allowedUser and ignoredUser are empty

@stakater-user
Copy link
Contributor

@stefan-mees Image is available for testing. docker pull stakater/gitwebhookproxy:SNAPSHOT-PR-84-1

@rasheedamir
Copy link
Member

@stefan-mees plz update your branch

@stakater-user
Copy link
Contributor

@stefan-mees Image is available for testing. docker pull stakater/gitwebhookproxy:SNAPSHOT-PR-84-2

@stefan-mees
Copy link
Author

@rasheedamir

@stakater-user
Copy link
Contributor

@stefan-mees Yikes! You better fix it before anyone else finds out! Build 3 has Failed!

maxisam added a commit to maxisam/GitWebhookProxy that referenced this pull request Jan 5, 2023
maxisam added a commit to maxisam/GitWebhookProxy that referenced this pull request Jan 5, 2023
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.

None yet

3 participants