-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
Allow increase in timeout for webhooks #639
Comments
Hey @DavidJFowler You are indeed correct at it being fixed at 15 seconds
would you be interested in submitting a pull request for this feature req? There is already an opt disable_ssl_verification, which is taken from the pact configuration options, so that might give you a decent pattern to follow |
@YOU54F Thanks for the reply and the hint. I'd love to submit a PR, but I'm a stranger to Ruby and I just don't have the time. |
Good easy hacktober issue for someone who knows ruby. |
Pre issue-raising checklist
I have already (please mark the applicable with an
x
):Software versions
pact-broker docker version: pactfoundation/pact-broker:2.112.0-pactbroker2.107.1
Expected behaviour
It should be possible to specify the timeout for executing a webhook in order to give services time to warm up etc.
Actual behaviour
The timeout is fixed at 15 seconds. If the webhook takes longer than this time to return a value it can get triggered multiple times in error.
The text was updated successfully, but these errors were encountered: