Files
posthog/plugin-server/tests/worker
Harry Waye 8adac54130 chore(webhooks): remove abstractions from webhook consumer logic (#16418)
* chore(webhooks): remove abstractions from webhook consumer logic

Previously we were jumping through a few hoops to make webhook calls
e.g. still using the piscina abstraction, still using the runner code.
This commit removes those abstractions while still maintaining the
existing functionality wrt error handling and metrics gathering.

I'll leave further refactoring of the webhook consumer code to a
separate PR. For example, moving the statsd metrics to be based on
OpenMetrics instead. And further adding some tracing around key parts of
the webhook matching and firing logic.

* fix typing

* fix typing

* fix typing

* fix unit tests

* fix tests

* chore(plugin-server): simplify action manager deps

Previously we were passing in the kitchen sync, but the only
dependency is postgres. This should make it easier to e.g. refactor to
not need to load the kitchen sync on some deployments.

* chore(plugin-server): simplify hook commander deps

Previously we passed in DB which is a lot of stuff. Now we just pass in
the postgres pool.

* fix import
2023-07-10 11:02:04 +00:00
..