Kaynağa Gözat

Return nil instead of result on sync error

Andrea Fazzi 5 yıl önce
ebeveyn
işleme
eae949196c
1 değiştirilmiş dosya ile 1 ekleme ve 3 silme
  1. 1 3
      cron/sync/sync.go

+ 1 - 3
cron/sync/sync.go

@@ -204,7 +204,7 @@ func (syncJob *SyncJob) SyncUsers(ldapClient *karmen_ldap.Client, karmenClient *
 
 			err = ldapClient.AddUser(user)
 			if err != nil {
-				return result, err
+				return nil, err
 			}
 
 			if user.GetRegenerate() {
@@ -425,14 +425,12 @@ func (syncJob *SyncJob) Run() {
 	}
 	log.Println("Sync administratives...")
 	if result, err := syncJob.SyncUsers(ldapClient, karmenClient, entries, users); err != nil {
-		log.Println(result)
 		panic(err)
 	} else {
 		log.Println(result)
 	}
 
 	if result, err := syncJob.SyncGroup(ldapClient, users, syncJob.conf.Sync.AdministrativesGroup); err != nil {
-		log.Println(result)
 		panic(err)
 	} else {
 		log.Println(result)