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

Add Hookdeck CLI #147

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

Add Hookdeck CLI #147

wants to merge 3 commits into from

Conversation

leggetter
Copy link

The Hookdeck platform is closed-source and has a free tier, but it is ultimately commercial.

The Hookdeck CLI works with the commercial offering but also allows anonymous connectivity without a Hookdeck account.

The Hookdeck platform is closed source, has a free tier, but is ultimately commercial.

The Hookdeck CLI works with the commercial offering, but also allows anonymous connectivity without a Hookdeck account.
@anderspitman
Copy link
Owner

Hey @leggetter, sorry for the delay. Does Hookdeck work with arbitrary HTTP traffic, or just webhooks, ie can I run a webserver behind it?

Don't worry about the conflicts, I'll fix those if we end up merging.

@leggetter
Copy link
Author

@anderspitman - yes, full pass-through support. Something I should flag - and happy to update the text description - is that because Hookdeck is purely asynchronous, the server response is not returned to the client that initiates the request. The initiating client will instead get a 200 OK for all valid requests, acknowledging that the request has been received.

@leggetter
Copy link
Author

@anderspitman - yes, full pass-through support. Something I should flag - and happy to update the text description - is that because Hookdeck is purely asynchronous, the server response is not returned to the client that initiates the request. The initiating client will instead get a 200 OK for all valid requests, acknowledging that the request has been received.

The text does say "asynchronous", but I want to make sure the above is clear.

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

2 participants