Remove payload key

[cbartz]

Applicable spec: n/a

Overview

Remove accessing the payload key when parsing the webhook payload.

Rationale

The key is non-existent. The webhook payload has the form

{
  'action': 'queued',
  'workflow_job': {
    'id': 28406633456,
    'run_id': 10262527884,
    'workflow_name': 'CI',
    'head_branch': 'push-test',
    'run_url': 'https://api.github.com/repos/canonical/cbartz-runner-testing/actions/runs/10262527884',
    'labels': [
      'self-hosted',
      'large'
    ],
  ...
  },
  'repository': {
...
  },
  'organization': {
...
  },
  'enterprise': {
...
  },
  'sender': {
...
  }
}

see also https://pastebin.canonical.com/p/zYhYJ53Rnq/

Juju Events Changes

n/a

Module Changes

parse.py: Do not assume the existence of the payload key in the payload.

Library Changes

n/a

Checklist

• The charm style guide was applied
• The contributing guide was applied
• The changes are compliant with ISD054 - Managing Charm Complexity
• The documentation is generated using src-docs
• The documentation for charmhub is updated
• The PR is tagged with appropriate label (urgent, trivial, complex)

[github-actions]

Test coverage for 1908fc8

Name                           Stmts   Miss Branch BrPart  Cover   Missing
--------------------------------------------------------------------------
webhook_router/__init__.py         0      0      0      0   100%
webhook_router/app.py            103      0     32      0   100%
webhook_router/mq.py              22      0      7      0   100%
webhook_router/parse.py           47      0     14      0   100%
webhook_router/router.py          46      0     18      0   100%
webhook_router/validation.py       6      0      0      0   100%
--------------------------------------------------------------------------
TOTAL                            224      0     71      0   100%

Static code analysis report

Run started:2024-08-07 08:32:16.451846

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 1716
  Total lines skipped (#nosec): 7
  Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
  Total issues (by severity):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
  Total issues (by confidence):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
Files skipped (0):