Avoid syncing extra details for forks

ecosyste-ms · Oct 20, 2023 · a720813 · a720813
1 parent 8f6d09b
commit a720813
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/app/models/host.rb b/app/models/host.rb
@@ -79,7 +79,7 @@ def sync_repository(full_name)
         end
         repo.save
         repo.ping_packages_async if repo_changed && repo.persisted?
-        repo.sync_extra_details_async if repo_changed && repo.persisted? && repo.files_changed?
+        repo.sync_extra_details_async if !repo.fork? && repo_changed && repo.persisted? && repo.files_changed?
         repo.sync_owner
         repo
       end

diff --git a/app/models/repository.rb b/app/models/repository.rb
@@ -211,14 +211,15 @@ def delete_old_manifests(new_manifests)
   end
 
   def self.sync_extra_details_async
-    Repository.where(files_changed: true).limit(600).order('pushed_at asc').select('id').each(&:sync_extra_details_async)
+    Repository.where(files_changed: true, fork: false).limit(600).order('pushed_at asc').select('id').each(&:sync_extra_details_async)
   end
 
   def sync_extra_details_async
     SyncExtraDetailsWorker.perform_async(self.id)
   end
 
   def sync_extra_details
+    return if fork?
     return unless files_changed?
     if pushed_at.present?
       parse_dependencies unless dependencies_parsed_at.present? && dependencies_parsed_at > pushed_at

diff --git a/app/sidekiq/ping_worker.rb b/app/sidekiq/ping_worker.rb
@@ -15,7 +15,7 @@ def perform(host_name, full_name)
       end
 
       repository.sync_async
-      repository.sync_extra_details_async if repository.files_changed?
+      repository.sync_extra_details_async if !repository.fork? && repository.files_changed?
     else
       host.sync_repository_async(full_name)
     end