From 4a52c1b1244d9b10334b6e67bdd68cda43586935 Mon Sep 17 00:00:00 2001
From: Ayman Bagabas <ayman.bagabas@gmail.com>
Date: Wed, 29 Mar 2023 17:36:27 -0400
Subject: [PATCH] fix(backend): cache repo rename

---
 server/backend/file/file.go | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/server/backend/file/file.go b/server/backend/file/file.go
index dfb805261..cf0b368fe 100644
--- a/server/backend/file/file.go
+++ b/server/backend/file/file.go
@@ -650,16 +650,29 @@ func (fb *FileBackend) DeleteRepository(repo string) error {
 //
 // It implements backend.Backend.
 func (fb *FileBackend) RenameRepository(oldName string, newName string) error {
-	oldName = filepath.Join(fb.reposPath(), utils.SanitizeRepo(oldName)+".git")
-	newName = filepath.Join(fb.reposPath(), utils.SanitizeRepo(newName)+".git")
-	if _, err := os.Stat(oldName); errors.Is(err, os.ErrNotExist) {
-		return fmt.Errorf("repository %q does not exist", strings.TrimSuffix(filepath.Base(oldName), ".git"))
+	oldName = utils.SanitizeRepo(oldName)
+	oldRepo := filepath.Join(fb.reposPath(), oldName+".git")
+	newName = utils.SanitizeRepo(newName)
+	newRepo := filepath.Join(fb.reposPath(), newName+".git")
+	if _, err := os.Stat(oldRepo); errors.Is(err, os.ErrNotExist) {
+		return fmt.Errorf("repository %q does not exist", strings.TrimSuffix(filepath.Base(oldRepo), ".git"))
 	}
-	if _, err := os.Stat(newName); err == nil {
-		return fmt.Errorf("repository %q already exists", strings.TrimSuffix(filepath.Base(newName), ".git"))
+	if _, err := os.Stat(newRepo); err == nil {
+		return fmt.Errorf("repository %q already exists", strings.TrimSuffix(filepath.Base(newRepo), ".git"))
 	}
 
-	return os.Rename(oldName, newName)
+	if err := os.Rename(oldRepo, newRepo); err != nil {
+		return err
+	}
+
+	// Update cache.
+	if r, ok := fb.repos[oldName]; ok {
+		r.path = newRepo
+		delete(fb.repos, oldName)
+		fb.repos[newName] = r
+	}
+
+	return nil
 }
 
 // Repository finds the given repository.