kallithea Changeset - 742d1b8ca263

Changeset - 742d1b8ca263

Parent rev.

Child rev.

[Not reviewed]

beta

0 2 0

Marcin Kuzminski - 13 years ago 2012-10-26 00:26:20
marcin@python-works.com

use str() on os.walk passing unicode can lead to UnicodeDecode errors when iterating

2 files changed with 2 insertions and 2 deletions:

rhodecode/lib/cleanup.py

rhodecode/lib/hooks.py

0 comments (0 inline, 0 general)

rhodecode/lib/cleanup.py

➞

Show inline comments

@@ @@ -88,7 +88,7 @@ class CleanupCommand(BasePasterCommand): @@
         repos_location = RhodeCodeUi.get_repos_location()
         to_remove = []
-        for dn, dirs, f in os.walk(str(repos_location)):
+        for dn, dirs, f in os.walk(safe_str(repos_location)):
             for loc in dirs:
                 if REMOVED_REPO_PAT.match(loc):
                     to_remove.append([loc, self._extract_date(loc)])

rhodecode/lib/hooks.py

➞

Show inline comments

@@ @@ -46,7 +46,7 @@ def _get_scm_size(alias, root_path): @@
         alias += '.'
     size_scm, size_root = 0, 0
     for path, dirs, files in os.walk(root_path):
+    for path, dirs, files in os.walk(safe_str(root_path)):
         if path.find(alias) != -1:
             for f in files:
                 try:

0 comments (0 inline, 0 general)