File fix-salt.utils.stringutils.to_str-calls-to-make... of Package salt

Overview Repositories Revisions Requests Users Attributes Meta

File fix-salt.utils.stringutils.to_str-calls-to-make-it-w.patch of Package salt (Revision 4cb1cc3ff955fa6c8b2d98d3913d834f)

Currently displaying revision 4cb1cc3ff955fa6c8b2d98d3913d834f , Show latest

From f348f291093ea3f7c841b03a975ae81b40963842 Mon Sep 17 00:00:00 2001
From: Victor Zhestkov <vzhestkov@suse.com>
Date: Wed, 25 Nov 2020 15:09:41 +0300
Subject: [PATCH] Fix salt.utils.stringutils.to_str calls to make it
 working with numeric uid/gid

---
 salt/modules/file.py | 16 ++++++++++------
 salt/states/file.py  | 11 +++++++++--
 2 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/salt/modules/file.py b/salt/modules/file.py
index d475e3c2e3..d3de4da467 100644
--- a/salt/modules/file.py
+++ b/salt/modules/file.py
@@ -5036,6 +5036,12 @@ def check_perms(
     is_dir = os.path.isdir(name)
     is_link = os.path.islink(name)
 
+    def __safe_to_str(s):
+        try:
+            return salt.utils.stringutils.to_str(s)
+        except:
+            return salt.utils.stringutils.to_str(str(s))
+
     # user/group changes if needed, then check if it worked
     if user:
         if isinstance(user, int):
@@ -5045,7 +5051,7 @@ def check_perms(
             and user_to_uid(user) != user_to_uid(perms["luser"])
         ) or (
             not salt.utils.platform.is_windows()
-            and salt.utils.stringutils.to_str(user) != perms["luser"]
+            and __safe_to_str(user) != perms["luser"]
         ):
             perms["cuser"] = user
 
@@ -5057,7 +5063,7 @@ def check_perms(
             and group_to_gid(group) != group_to_gid(perms["lgroup"])
         ) or (
             not salt.utils.platform.is_windows()
-            and salt.utils.stringutils.to_str(group) != perms["lgroup"]
+            and __safe_to_str(group) != perms["lgroup"]
         ):
             perms["cgroup"] = group
 
@@ -5089,8 +5095,7 @@ def check_perms(
             and user != ""
         ) or (
             not salt.utils.platform.is_windows()
-            and salt.utils.stringutils.to_str(user)
-            != get_user(name, follow_symlinks=follow_symlinks)
+            and __safe_to_str(user) != get_user(name, follow_symlinks=follow_symlinks)
             and user != ""
         ):
             if __opts__["test"] is True:
@@ -5111,8 +5116,7 @@ def check_perms(
             and group != ""
         ) or (
             not salt.utils.platform.is_windows()
-            and salt.utils.stringutils.to_str(group)
-            != get_group(name, follow_symlinks=follow_symlinks)
+            and __safe_to_str(group) != get_group(name, follow_symlinks=follow_symlinks)
             and group != ""
         ):
             if __opts__["test"] is True:
diff --git a/salt/states/file.py b/salt/states/file.py
index 50ceef1158..1083bb46d6 100644
--- a/salt/states/file.py
+++ b/salt/states/file.py
@@ -863,15 +863,22 @@ def _check_dir_meta(name, user, group, mode, follow_symlinks=False):
     if not stats:
         changes["directory"] = "new"
         return changes
+
+    def __safe_to_str(s):
+        try:
+            return salt.utils.stringutils.to_str(s)
+        except:
+            return salt.utils.stringutils.to_str(str(s))
+
     if (
         user is not None
-        and salt.utils.stringutils.to_str(user) != stats["user"]
+        and __safe_to_str(user) != stats["user"]
         and user != stats.get("uid")
     ):
         changes["user"] = user
     if (
         group is not None
-        and salt.utils.stringutils.to_str(group) != stats["group"]
+        and __safe_to_str(group) != stats["group"]
         and group != stats.get("gid")
     ):
         changes["group"] = group
-- 
2.37.3

Places

File fix-salt.utils.stringutils.to_str-calls-to-make-it-w.patch of Package salt (Revision 4cb1cc3ff955fa6c8b2d98d3913d834f)

Places