supabase / realtime / 43180742677

93%

Build Type

push

github

Committed by web-flow

Commit Message

feat(policy_eval): Include role in policy eval result metric (#2553)

At first I wanted to use a `log` to also carry the `tenant_id` to be
able to check the concrete roles and setup more easily if needbe.

However, figured that at a first step it'd be easier and cheaper to
include the role in the outcome so we can test the hypothesis that
it's a service role which we can easily fix to make decidable.

The metric is very bound - only naming known roles and putting the
others into an `other` bucket so this should be cheap enough from
my understanding.

If we see a lot of "other" we can still introduce logging.

Coverage Stats

6 of 6 new or added lines in 2 files covered. (100.0%)

1 existing line in 1 file now uncovered.

4808 of 5159 relevant lines covered (93.2%)

4643.6 hits per line