wuvt / trackman

WUVT's track logging web-app
GNU Affero General Public License v3.0
4 stars 3 forks source link

Webhook support? #67

Open mutantmonkey opened 5 years ago

mutantmonkey commented 5 years ago

Currently, metadater and radiotextual listen to the Trackman SSE endpoint and perform actions as necessary. It might be useful to add webhook support directly to Trackman to avoid the need to listen to the SSE endpoint. If doing so, the webhooks will need to run asynchronously to avoid delaying track logging.

It's not really clear to me if there's any benefit to adding this functionality directly to Trackman however, since these SSE listener applications can easily make requests to a webhook, but since we're already using Celery, we can easily run tasks asynchronously without any additional changes.

mutantmonkey commented 5 years ago

As of March 2019, we are no longer using Celery. Webhook support would probably be best implemented as a separate app that can be run alongside Trackman in the style of metadater.