From a21e8f9b13e1207618cbe04c888091e20a7fc681 Mon Sep 17 00:00:00 2001
From: Arya Khochare <91268931+Aryakoste@users.noreply.github.com>
Date: Mon, 4 Nov 2024 16:35:43 +0530
Subject: [PATCH] Fixed errcheck issues in server/channels/app/ldap.go (#29038)

---
 server/.golangci.yml        | 1 -
 server/channels/app/ldap.go | 4 +++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/server/.golangci.yml b/server/.golangci.yml
index 718e95c3b8..b346464b87 100644
--- a/server/.golangci.yml
+++ b/server/.golangci.yml
@@ -112,7 +112,6 @@ issues:
         channels/app/integration_action.go|\
         channels/app/integration_action_test.go|\
         channels/app/job_test.go|\
-        channels/app/ldap.go|\
         channels/app/login_test.go|\
         channels/app/migrations.go|\
         channels/app/notification_push.go|\
diff --git a/server/channels/app/ldap.go b/server/channels/app/ldap.go
index a3367c42c9..41557a133c 100644
--- a/server/channels/app/ldap.go
+++ b/server/channels/app/ldap.go
@@ -30,7 +30,9 @@ func (a *App) SyncLdap(c request.CTX, includeRemovedMembers bool) {
 				c.Logger().Error("Not executing ldap sync because ldap is not available")
 				return
 			}
-			ldapI.StartSynchronizeJob(c, false, includeRemovedMembers)
+			if _, appErr := ldapI.StartSynchronizeJob(c, false, includeRemovedMembers); appErr != nil {
+				c.Logger().Error("Failed to start LDAP sync job")
+			}
 		}
 	})
 }