Kdesktop 774 implementation of set rights method.

.github/workflows/windows.yml

@@ -40,6 +40,6 @@ jobs:
 
       - name: Run common tests
         run: ./build-Release/install/bin/kDrive_test_common.exe
-
+          
       - name: Clean-up generated code
         run : powershell ./infomaniak-build-tools/windows/build-drive.ps1 -clean all

src/libcommonserver/io/iohelper.cpp

@@ -507,4 +507,36 @@ bool IoHelper::createSymlink(const SyncPath &targetPath, const SyncPath &path, I
 
     return ioError == IoErrorSuccess;
 }
+
+#ifndef _WIN32
+//See iohelper_win.cpp for the Windows implementation
+bool IoHelper::setRights(const SyncPath &path, bool read, bool write, bool exec, IoError &ioError) noexcept {
+    return _setRightsStd(path, read, write, exec, ioError);
+}
+#endif
+
+bool IoHelper::_setRightsStd(const SyncPath &path, bool read, bool write, bool exec, IoError &ioError) noexcept {
+    ioError = IoErrorSuccess;
+    std::filesystem::perms perms = std::filesystem::perms::none;
+    if (read) {
+        perms |= std::filesystem::perms::owner_read;
+    }
+    if (write) {
+        perms |= std::filesystem::perms::owner_write;
+    }
+    if (exec) {
+        perms |= std::filesystem::perms::owner_exec;
+    }
+
+    std::error_code ec;
+    std::filesystem::permissions(path, perms, ec);
+    if (ec) {
+        ioError = posixError2ioError(ec.value());
+        return _isExpectedError(ioError);
+    }
+
+    return true;
+}
+
+
 }  // namespace KDC

src/libcommonserver/io/iohelper.h

@@ -20,6 +20,9 @@
 
 #include "libcommon/utility/types.h"
 #include "libcommonserver/log/log.h"
+#ifdef _WIN32
+#include <AccCtrl.h>
+#endif
 
 namespace KDC {
 
@@ -32,9 +35,9 @@ struct IoHelper {
         inline static void setLogger(log4cplus::Logger logger) { _logger = logger; }
 
 #ifdef _WIN32
-        static int _getRightsMethod;
+        static int _getAndSetRightsMethod;
 #endif
-
+        
         static IoError stdError2ioError(int error) noexcept;
         static IoError stdError2ioError(const std::error_code &ec) noexcept;
         static IoError posixError2ioError(int error) noexcept;
@@ -255,9 +258,28 @@ struct IoHelper {
          */
         static bool checkIfFileIsDehydrated(const SyncPath &path, bool &isDehydrated, IoError &ioError) noexcept;
 
-        // TODO: docstring and unit tests
+        //! Get the rights of the item indicated by `path`.
+        /*!
+         \param path is the file system path of the item.
+         \param read is a boolean indicating whether the item is readable.
+         \param write is a boolean indicating whether the item is writable.
+         \param exec is a boolean indicating whether the item is executable.
+         \param exists is a boolean indicating whether the item exists.
+         \return true if no unexpected error occurred, false otherwise.
+         */
         static bool getRights(const SyncPath &path, bool &read, bool &write, bool &exec, bool &exists) noexcept;
 
+        //! Set the rights of the item indicated by `path`.
+        /*!
+         \param path is the file system path of the item.
+         \param read is a boolean indicating whether the item should be readable.
+         \param write is a boolean indicating whether the item should be writable.
+         \param exec is a boolean indicating whether the item should be executable.
+         \param ioError holds the error returned when an underlying OS API call fails.
+         \return true if no unexpected error occurred, false otherwise.
+        */
+        static bool setRights(const SyncPath &path, bool read, bool write, bool exec, IoError &ioError) noexcept;
+
     protected:
         // These functions default to the std::filesystem functions.
         // They can be modified in tests.
@@ -283,6 +305,8 @@ struct IoHelper {
 #endif
         static bool _setTargetType(ItemType &itemType) noexcept;
         static bool _checkIfIsHiddenFile(const SyncPath &path, bool &isHidden, IoError &ioError) noexcept;
+
+        static bool _setRightsStd(const SyncPath &path, bool read, bool write, bool exec, IoError &ioError) noexcept;
 };
 
 }  // namespace KDC

src/libcommonserver/io/iohelper_linux.cpp

@@ -141,5 +141,4 @@ bool IoHelper::getRights(const SyncPath &path, bool &read, bool &write, bool &ex
     return true;
 }
 
-
 }  // namespace KDC

src/libcommonserver/io/iohelper_mac.cpp

@@ -222,15 +222,14 @@ bool IoHelper::getRights(const SyncPath &path, bool &read, bool &write, bool &ex
     ItemType itemType;
     const bool success = getItemType(path, itemType);
     if (!success) {
-        LOGW_WARN(logger(),
-                  L"Failed to check if the item is a symlink: " << Utility::formatIoError(path, itemType.ioError).c_str());
+        LOGW_WARN(logger(), L"Failed to get item type: " << Utility::formatIoError(path, itemType.ioError).c_str());
         return false;
     }
     exists = itemType.ioError != IoErrorNoSuchFileOrDirectory;
     if (!exists) {
         return true;
     }
-    const bool isSymlink = itemType.linkType = LinkTypeSymlink;
+    const bool isSymlink = itemType.linkType == LinkTypeSymlink;
 
     std::error_code ec;
     std::filesystem::perms perms =
@@ -242,7 +241,7 @@ bool IoHelper::getRights(const SyncPath &path, bool &read, bool &write, bool &ex
             return true;
         }
 
-        LOGW_WARN(logger(), L"Failed to get permissions - " << Utility::formatStdError(path, ec).c_str());
+        LOGW_WARN(logger(), L"Failed to get permissions: " << Utility::formatStdError(path, ec).c_str());
         return false;
     }