Use IterateBufferSize whilst querying repositories during adoption ch…

…eck (go-gitea#19140) (go-gitea#19160) Backport go-gitea#19140 The adoption page checks directories to see if they are repositories by querying the db on a per user basis. This can lead to problems if a user has a large number of repositories or putative repositories. This PR changes the buffering to check the db in IterataeBufferSize batches instead. Fix go-gitea#19137 Signed-off-by: Andrew Thornton <[email protected]>
zeripath · Mar 22, 2022 · 23b8214 · 23b8214
1 parent 08feb6b
commit 23b8214
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/services/repository/adopt.go b/services/repository/adopt.go
@@ -335,6 +335,13 @@ func ListUnadoptedRepositories(query string, opts *db.ListOptions) ([]string, in
 		}
 
 		repoNamesToCheck = append(repoNamesToCheck, name)
+		if len(repoNamesToCheck) > setting.Database.IterateBufferSize {
+			if err = checkUnadoptedRepositories(userName, repoNamesToCheck, unadopted); err != nil {
+				return err
+			}
+			repoNamesToCheck = repoNamesToCheck[:0]
+
+		}
 		return filepath.SkipDir
 	}); err != nil {
 		return nil, 0, err