Use /\s+/ to split source_directories to handle 1+ spaces.

This bug is can be quite annoying because when you accidentally used
something like:

```ini
[location]
source_directories: backup_one  backup_two
;                              A (Additional space here)
```

It would call Attic/Borg with ('backup_one', '', 'backup_two') which in
turn backups your whole $PWD.
This commit is contained in:
Robin Schneider 2016-01-31 11:42:07 +01:00
parent e1605ae6ab
commit 9f5c5c8e13
3 changed files with 22 additions and 2 deletions

4
NEWS
View file

@ -1,3 +1,7 @@
0.1.8-dev
* Fixed handling of repeated spaces in source_directories which resulted in backup up everything.
0.1.7 0.1.7
* #11: Fixed parsing of punctuation in configuration file. * #11: Fixed parsing of punctuation in configuration file.

View file

@ -1,5 +1,6 @@
from datetime import datetime from datetime import datetime
import os import os
import re
import platform import platform
import subprocess import subprocess
@ -63,7 +64,7 @@ def create_archive(
list of source directories, a local or remote repository path, and a command to run, create an list of source directories, a local or remote repository path, and a command to run, create an
attic archive. attic archive.
''' '''
sources = tuple(source_directories.split(' ')) sources = tuple(re.split('\s+', source_directories))
exclude_flags = ('--exclude-from', excludes_filename) if excludes_filename else () exclude_flags = ('--exclude-from', excludes_filename) if excludes_filename else ()
compression = storage_config.get('compression', None) compression = storage_config.get('compression', None)
compression_flags = ('--compression', compression) if compression else () compression_flags = ('--compression', compression) if compression else ()
@ -167,7 +168,7 @@ def _make_check_flags(checks, check_last=None):
('repository',) ('repository',)
This will be returned as: This will be returned as:
('--repository-only',) ('--repository-only',)
Additionally, if a check_last value is given, a "--last" flag will be added. Note that only Additionally, if a check_last value is given, a "--last" flag will be added. Note that only

View file

@ -71,6 +71,21 @@ def test_create_archive_should_call_attic_with_parameters():
) )
def test_create_archive_with_two_spaces_in_source_directories():
insert_subprocess_mock(CREATE_COMMAND)
insert_platform_mock()
insert_datetime_mock()
module.create_archive(
excludes_filename='excludes',
verbosity=None,
storage_config={},
source_directories='foo bar',
repository='repo',
command='attic',
)
def test_create_archive_with_none_excludes_filename_should_call_attic_without_excludes(): def test_create_archive_with_none_excludes_filename_should_call_attic_without_excludes():
insert_subprocess_mock(CREATE_COMMAND_WITHOUT_EXCLUDES) insert_subprocess_mock(CREATE_COMMAND_WITHOUT_EXCLUDES)
insert_platform_mock() insert_platform_mock()