Skip to content

feat(timeout): Added timeout parameter to set a webhook response wait timeout.#2

Open
drlove2002 wants to merge 12 commits intonavied:masterfrom
drlove2002:feat/timeout
Open

feat(timeout): Added timeout parameter to set a webhook response wait timeout.#2
drlove2002 wants to merge 12 commits intonavied:masterfrom
drlove2002:feat/timeout

Conversation

@drlove2002
Copy link

Features

  • Timeout to return a timeout error and stop the workflow if the webhook server doesn't respond (default is 0 means it'll wait forever)

Fix

  • X-Hub-Signature and X-Hub-Signature-236 digests now have prefixes, sha1= and sha256= respectively.

Docs

  • README now contains steps to test the close and collaborate on the project.

signature_sha1 with a sha1= prefix and signature_sha256 with a sha256= prefix
webhook sending fail for internal issues will also be displayed
Timeout will help to stop waiting for response and stop the workflow with a valid nice error message.
- Added how to contribute in this repository.
- Added how to test the code.
- Updated new code parameter (timeout)
- Updated Credit for navied/secure-webhook  :)
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.

1 participant