Cosmetics: drop debug spam

Author: robertwoj-microsoft

src/common/commonutils/FileUtils.c

@@ -1023,8 +1023,6 @@ int ReplaceMarkedLinesInFile(const char* fileName, const char* marker, const cha
                                 status = (0 == errno) ? EPERM : errno;
                                 OsConfigLogInfo(log, "ReplaceMarkedLinesInFile: cannot write to temporary file '%s' (%d)", tempFileName, status);
                             }
-                        } else {
-                            OsConfigLogInfo(log, "Skipping line: %s", line);
                         }
 
                         memset(line, 0, lineMax + 1);
@@ -1076,7 +1074,6 @@ int ReplaceMarkedLinesInFile(const char* fileName, const char* marker, const cha
 
     if (0 == status)
     {
-        OsConfigLogInfo(log, "ReplaceMarkedLinesInFile: replacing '%s' with '%s' in '%s'", marker, newline, fileName);
         if (preserveAccess)
         {
             if (0 != (status = RenameFileWithOwnerAndAccess(tempFileName, fileName, log)))

src/common/commonutils/UserUtils.c

@@ -121,7 +121,6 @@ static int CopyUserEntry(SimplifiedUser* destination, struct passwd* source, OsC
 
     if (0 != status)
     {
-        OsConfigLogError(log, "CopyUserEntry: failed to copy user entry for '%s' (uid: %u, gid: %u)", source->pw_name, source->pw_uid, source->pw_gid);
         ResetUserEntry(destination);
     }
 
@@ -359,7 +358,6 @@ int EnumerateUsers(SimplifiedUser** userList, unsigned int* size, char** reason,
 
     if (0 != (*size = GetNumberOfLinesInFile(passwdFile)))
     {
-        OsConfigLogInfo(log, "EnumerateUsers: %u lines in '%s'", *size, passwdFile);
         listSize = (*size) * sizeof(SimplifiedUser);
         if (NULL != (*userList = malloc(listSize)))
         {
@@ -369,7 +367,6 @@ int EnumerateUsers(SimplifiedUser** userList, unsigned int* size, char** reason,
 
             while ((NULL != (userEntry = getpwent())) && (i < *size))
             {
-                OsConfigLogInfo(log, "EnumerateUsers: user %u: '%s' (%u, %u) in '%s'", i, userEntry->pw_name, userEntry->pw_uid, userEntry->pw_gid, passwdFile);
                 if (0 != (status = CopyUserEntry(&((*userList)[i]), userEntry, log)))
                 {
                     OsConfigLogError(log, "EnumerateUsers: failed making copy of user entry (%d)", status);
@@ -836,8 +833,6 @@ int RemoveUser(SimplifiedUser* user, bool removeHome, OsConfigLogHandle log)
         return EPERM;
     }
 
-    OsConfigLogError(log, "RemoveUser: attempting to delete user '%s' (%u, %u) with home directory '%s'",
-        user->username, user->userId, user->groupId, user->home);
     if (NULL != (command = FormatAllocateString(commandTemplate, removeHome ? "-f -r" : "-f", user->username)))
     {
         if (0 == (status = ExecuteCommand(NULL, command, false, false, 0, 0, NULL, NULL, log)))

src/modules/compliance/src/lib/ComplianceInterface.cpp

@@ -142,7 +142,6 @@ int ComplianceMmiSet(MMI_HANDLE clientSession, const char* componentName, const
 
     try
     {
-        OsConfigLogInfo(engine.log(), "MmiSet(%p, %s, %s, %.*s, %d)", clientSession, componentName, objectName, payloadSizeBytes, payload, payloadSizeBytes);
         std::string payloadStr(payload, payloadSizeBytes);
         auto object = parseJSON(payloadStr.c_str());
         if (NULL == object || (JSONString != json_value_get_type(object.get()) && JSONObject != json_value_get_type(object.get())))

src/modules/compliance/src/lib/Evaluator.cpp

@@ -45,8 +45,6 @@ Result<Status> Evaluator::ExecuteRemediation()
         return result.error();
     }
 
-    OsConfigLogInfo(mLog, "Remediation result: %s", result.value() == Status::Compliant ? "Compliant" : "Non-compliant");
-    OsConfigLogInfo(mLog, "Remediation log: %s", mLogstream.str().c_str());
     return result;
 }