Skip to content

Internal: Refactor how webhooks work#159

Merged
turt2live merged 4 commits into
mainfrom
travis/webhooks
Jun 15, 2026
Merged

Internal: Refactor how webhooks work#159
turt2live merged 4 commits into
mainfrom
travis/webhooks

Conversation

@turt2live

@turt2live turt2live commented Jun 12, 2026

Copy link
Copy Markdown
Member

This PR is a step towards replacing the "audit context". The approximate plan is to move to a "record/publish" pattern instead of an audit pattern. Recorders can then remember some details about harms on events. Webhooks would be one of the "publishers", so not much effort is made to keep the audit code clean. Implementing recorders/publishers is for a future PR.

This PR adds an interface for notifiers with an assumption that eventually policyserv will be able to send events itself, which would be a new notifier type.

Pull Request Checklist

  • Pull request is based on the main branch.
  • Pull request title describes the changes in a way a user would understand. For example, "Fix keywords filter not applying" instead of "Read keywords slice in a loop".
  • Code style matches surrounding code.
  • Tests are added for new code (and existing code) if possible.
    • If not possible, please explain why in your PR description.
  • The CI checks pass.

@turt2live turt2live requested a review from a team as a code owner June 12, 2026 20:32
@turt2live turt2live merged commit 22b4010 into main Jun 15, 2026
11 of 12 checks passed
@turt2live turt2live deleted the travis/webhooks branch June 15, 2026 18:19
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.

2 participants