]> git.cworth.org Git - notmuch/commitdiff
test: fix passwd_sanitize()
authorFelipe Contreras <felipe.contreras@gmail.com>
Sat, 1 May 2021 11:54:10 +0000 (06:54 -0500)
committerDavid Bremner <david@tethera.net>
Sun, 2 May 2021 00:13:23 +0000 (21:13 -0300)
If any of the variables is empty the output is completely messed up,
because replace("", "FOO") puts "FOO" before every single character.

I don't have my full name configured, and this is what I get:

  USER_FULL_NAME=USER_FULL_NAME=USER_FULL_NAME USER_FULL_NAMEsUSER_FULL_NAMEtUSER_FULL_NAMEdUSER_FULL_NAMEoUSER_FULL_NAMEuUSER_FULL_NAMEtUSER_FULL_NAME USER_FULL_NAME=USER_FULL_NAME=USER_FULL_NAME

Let's check for empty strings before doing any replace.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
test/test-lib.sh

index 4c9f2a2103e32a18c2c14dc257669300ecbe4d6f..e13797a75a358b458c52c10332f672fcd6265337 100644 (file)
@@ -711,7 +711,12 @@ name = pw.pw_gecos.partition(",")[0]
 fqdn = socket.getfqdn()
 
 for l in sys.stdin:
-    l = l.replace(user, "USERNAME").replace(fqdn, "FQDN").replace(".(none)","").replace(name, "USER_FULL_NAME")
+    if user:
+        l = l.replace(user, "USERNAME")
+    if fqdn:
+        l = l.replace(fqdn, "FQDN").replace(".(none)","")
+    if name:
+        l = l.replace(name, "USER_FULL_NAME")
     sys.stdout.write(l)
 '
 }