Bug fix: manifests with extra spaces
authorTim Pierce <twp@unchi.org>
Fri, 12 Sep 2014 19:18:15 +0000 (15:18 -0400)
committerTim Pierce <twp@unchi.org>
Fri, 12 Sep 2014 19:20:20 +0000 (15:20 -0400)
Extend the regular expression to match manifest_text to permit more than
a single space in manifest entries (seen in e.g. 91534558193f42a2f7f8aca872e5a78d+15723)

No issue #

sdk/python/arvados/collection.py

index 686d694befec9d1aec84c43dcbe360b0a6008b09..f5c4066a4d7573cf35553df79adbc5ec35220469 100644 (file)
@@ -101,7 +101,7 @@ class CollectionReader(object):
         elif re.match(r'[a-z0-9]{5}-[a-z0-9]{5}-[a-z0-9]{15}$', manifest_locator_or_text):
             self._manifest_locator = manifest_locator_or_text
             self._manifest_text = None
-        elif re.match(r'((\S+)( [a-f0-9]{32}(\+\d+)(\+\S+)*)+( \d+:\d+:\S+)+$)+', manifest_locator_or_text, re.MULTILINE):
+        elif re.match(r'((\S+)( +[a-f0-9]{32}(\+\d+)(\+\S+)*)+( +\d+:\d+:\S+)+$)+', manifest_locator_or_text, re.MULTILINE):
             self._manifest_text = manifest_locator_or_text
             self._manifest_locator = None
         else: