Skip to content

Commit

Permalink
Add run message and report to webhook message
Browse files Browse the repository at this point in the history
This does increase the message size, though I imagine that's marginal,
and is backwards compatible in that old keys remain in place and
aren't modified. The goal is to make the actual GHC app more set and
forget, and we can handle the errors with a little more discretion on
the webhook end.
  • Loading branch information
Henry Walshaw committed Aug 29, 2024
1 parent 1392308 commit e883baf
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions GeoHealthCheck/notifications.py
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,8 @@ def do_webhook(config, resource, run, status_changed, result):
params['ghc.resource.title'] = resource.title
params['ghc.resource.type'] = resource.resource_type
params['ghc.resource.view'] = resource_view
params['ghc.run.message'] = run.message
params['ghc.run.report'] = run.report

try:
r = requests.post(url, params)
Expand Down

0 comments on commit e883baf

Please sign in to comment.