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 job status update via webhook #203

Merged
merged 5 commits into from
Nov 8, 2021
Merged

Conversation

latenssi
Copy link
Member

@latenssi latenssi commented Nov 8, 2021

Allow users to set a URL which receives job status updates as a webhook. The endpoint should accept POST requests of type application/json and always respond with a status code of 200.

resolves #180

@latenssi latenssi force-pushed the latenssi/webhook-job-status branch from 14bbd88 to ed099fa Compare November 8, 2021 10:56
@latenssi latenssi force-pushed the latenssi/webhook-job-status branch from ed099fa to fef877b Compare November 8, 2021 10:57
@latenssi latenssi linked an issue Nov 8, 2021 that may be closed by this pull request
@latenssi latenssi requested a review from nanuuki November 8, 2021 11:07
Copy link
Contributor

@nanuuki nanuuki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the panic message, works like a charm 👍

jobs/options.go Outdated Show resolved Hide resolved
@latenssi latenssi merged commit 3fc4480 into main Nov 8, 2021
@latenssi latenssi deleted the latenssi/webhook-job-status branch November 8, 2021 12:57
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.

Webhook for Transactions (Jobs) Status
2 participants