fensak-io / fensak

GitHub app for Fensak, a service that alows users to apply security best practices to GitOps.
https://fensak.io
Other
3 stars 0 forks source link

Filter push events on .fensak repo in web request #103

Closed yorinasub17 closed 11 months ago

yorinasub17 commented 11 months ago

Right now we are filtering the push event by repo in the asynchronous handler. This works, but is not great for the Deno KV quota since it requires a round trip with Queues.

Since all the info for filtering is available in the event data, we should do the filtering for push events only in the web request to avoid consuming kv credits. This is most important for orgs with many repos that may install the app on all repos in the org.

yorinasub17 commented 11 months ago

This is done in https://github.com/fensak-io/fensak/pull/104