Skip to content

PM-33527 - Don't leave orphaned event logs in storage#7783

Draft
prograhamming wants to merge 46 commits into
mainfrom
dirt/pm-33527/server-db-combined
Draft

PM-33527 - Don't leave orphaned event logs in storage#7783
prograhamming wants to merge 46 commits into
mainfrom
dirt/pm-33527/server-db-combined

Conversation

@prograhamming

@prograhamming prograhamming commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking
PM-33527

📔 Objective
In order to stay within GDPR compliance when a organization wants to be deleted we need to remove the event logs from Azure Table Storage. This PR is the database changes needed in order to delete the event logs using jobs and long running processes.

prograhamming and others added 30 commits April 21, 2026 08:09
…:bitwarden/server into dirt/pm-33527/remove-orphaned-event-logs
…warden/server into dirt/pm-33527/db-orphaned-event-logs
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 69.30233% with 66 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.56%. Comparing base (4abea38) to head (46bed85).

Files with missing lines Patch % Lines
.../Dirt/Repositories/TableStorage/EventRepository.cs 0.00% 45 Missing ⚠️
...ucture.Dapper/Dirt/Repositories/EventRepository.cs 0.00% 8 Missing ⚠️
...tityFramework/Dirt/Repositories/EventRepository.cs 0.00% 7 Missing ⚠️
src/Admin/Jobs/CleanUpOrganizationEventsJob.cs 92.72% 1 Missing and 3 partials ⚠️
src/Admin/Jobs/JobsHostedService.cs 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7783      +/-   ##
==========================================
+ Coverage   61.15%   65.56%   +4.40%     
==========================================
  Files        2175     2180       +5     
  Lines       96792    97005     +213     
  Branches     8730     8743      +13     
==========================================
+ Hits        59196    63597    +4401     
+ Misses      35486    31204    -4282     
- Partials     2110     2204      +94     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.


namespace Bit.Infrastructure.EntityFramework.Dirt.Models;

public class OrganizationDeleteTask : Core.Dirt.Entities.OrganizationDeleteTask
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Reliability Rating on New Code (required ≥ D)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant