l3montree-dev / l3montree-dev/devguard

Webhook handler returns 500 status code (Improve output for debugging)

Open
#1,159 3 comments 0 reactions 1 assignee View on GitHub

Nobody has claimed this yet.

component/devguard-api component/devguard-web
Dominant language
Go
Stars
161
Forks
43
Avg merge
1d 8h
Merged PRs (30d)
37

Description

Image

First of all: Everything works fine. Those issue events are pretty unrelated. This was devguard itself creating tickets and triggering the webhook. Just a 200 status code would be great!

  • Fix webhook handler to return 202 https status (Accepted) on webhooks which actually do something - return 200 on webhooks which we "dont care" about.
  • Add more debugging output to the webhook response. Currently it just says: "ok" - would be great to get some insights like: Did you update a dependency vuln?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.