bitwarden / passwordless-server

Bitwarden Passwordless.dev infrastructure/backend (API, database, Docker, etc).
https://bitwarden.com/
Other
86 stars 28 forks source link

PAS-552 | Restore BackgroundService for Bitwarden's internal reporting #665

Closed jonashendrickx closed 3 months ago

jonashendrickx commented 3 months ago

Ticket

Description

Restore UserCountUpdaterBackgroundService for use in internal reporting at Bitwarden which happens bi-weekly. This service was initially dropped due to performance reasons.

Shape

Screenshots

Checklist

I did the following to ensure that my changes were tested thoroughly:

I did the following to ensure that my changes do not introduce security vulnerabilities:

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 34.83%. Comparing base (7f72d2f) to head (3f4cec4).

Files Patch % Lines
...roundServices/UserCountUpdaterBackgroundService.cs 0.00% 13 Missing :warning:
src/AdminConsole/Billing/BillingBootstrap.cs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #665 +/- ## ========================================== - Coverage 34.84% 34.83% -0.02% ========================================== Files 565 566 +1 Lines 30901 30915 +14 Branches 922 922 ========================================== Hits 10768 10768 - Misses 19990 20004 +14 Partials 143 143 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.