Merge pull request #24 from UnownHash/comstud/reload-acct-fix

Fix ReloadAccounts if accounts are removed
UnownHash · Oct 14, 2023 · f120afc · f120afc
2 parents ebc041a + 0c25760
commit f120afc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/accounts/accountManager.go b/accounts/accountManager.go
@@ -5,10 +5,11 @@ import (
 	"flygon/config"
 	"flygon/db"
 	"flygon/pogo"
-	log "github.com/sirupsen/logrus"
-	"gopkg.in/guregu/null.v4"
 	"sync"
 	"time"
+
+	log "github.com/sirupsen/logrus"
+	"gopkg.in/guregu/null.v4"
 )
 
 type AccountDetails struct {
@@ -86,6 +87,8 @@ func (a *AccountManager) ReloadAccounts() {
 			a.accounts = append(a.accounts[:x], a.accounts[x+1:]...)
 			a.inUse = append(a.inUse[:x], a.inUse[x+1:]...)
 			foundRecords = append(foundRecords[:x], foundRecords[x+1:]...)
+			// 'x' now points to next record, so we need to repeat
+			x--
 		}
 	}
 }

diff --git a/koji/main.go b/koji/main.go
@@ -92,10 +92,6 @@ func LoadKojiAreas(details *db.DbDetails) {
 		return
 	}
 
-	if err != nil {
-		log.Errorf("[KOJI]: %s", err)
-		return
-	}
 	var backgroundProcesses sync.WaitGroup
 
 	parallelSem := make(chan bool, 50)