Skip to content
Snippets Groups Projects

Audit events for package deletion

Merged Moaz Khalifa requested to merge 329588-audit-events-for-package-deletion into master
6 files
+ 56
32
Compare changes
  • Side-by-side
  • Inline
Files
6
@@ -32,12 +32,10 @@ def execute(batch_size: BATCH_SIZE)
no_access = false
min_batch_size = [batch_size, BATCH_SIZE].min
package_ids = []
@all_loaded_packages = [] # used in sending audit events in EE
packages.each_batch(of: min_batch_size) do |batched_packages|
loaded_packages = batched_packages.including_project_route.to_a
package_ids = loaded_packages.map(&:id)
@all_loaded_packages << loaded_packages
break no_access = true unless can_destroy_packages?(loaded_packages)
@@ -47,6 +45,7 @@ def execute(batch_size: BATCH_SIZE)
sync_maven_metadata(loaded_packages)
sync_npm_metadata(loaded_packages)
mark_package_files_for_destruction(loaded_packages)
send_audit_events(loaded_packages)
end
return UNAUTHORIZED_RESPONSE if no_access
@@ -59,7 +58,7 @@ def execute(batch_size: BATCH_SIZE)
private
attr_reader :packages, :current_user, :all_loaded_packages
attr_reader :packages, :current_user
def mark_package_files_for_destruction(packages)
::Packages::MarkPackageFilesForDestructionWorker.bulk_perform_async_with_contexts(
@@ -93,6 +92,9 @@ def can_destroy_packages?(packages)
end
end
# used in EE
def send_audit_events(_); end
def track_exception(error, package_ids)
Gitlab::ErrorTracking.track_exception(error, package_ids: package_ids)
end
Loading