sanity check: ensure a valid parameter is passed to add_warcs

webrecorder · Nov 13, 2018 · 9e9eb2e · 9e9eb2e
1 parent 6d9122e
commit 9e9eb2e
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pywb/manager/manager.py b/pywb/manager/manager.py
@@ -126,8 +126,10 @@ def add_warcs(self, warcs, method='copy'):
             elif method == 'symlink':
                 os.symlink(filename, os.path.join(self.archive_dir,
                                                   os.path.basename(filename)))
-            else:
+            elif method == 'copy':
                 shutil.copy2(filename, self.archive_dir)
+            else:
+                raise NotImplementedError('unknown method name: %s' % method)
             full_paths.append(os.path.join(self.archive_dir, filename))
 
         self._index_merge_warcs(full_paths, self.DEF_INDEX_FILE)