Added setting to follow symlinks

#48, #146

config.sample

@@ -8,6 +8,9 @@
 ; Optional, restrict scanner to these extensions. Default: none
 ;scanner_extensions = mp3 ogg
 
+; Should the scanner follow symbolic links? Default: no
+follow_symlinks = no
+
 [webapp]
 ; Optional cache directory. Default: /tmp/supysonic
 cache_dir = /var/supysonic/cache

docs/configuration.md

@@ -61,6 +61,10 @@ restricted to. Useful if you have multiple audio formats in your library but
 only want to serve some. If left empty, the scanner will try to read every file
 it finds.
 
+`follow_symlinks`: if set to `yes`, allows the scanner to follow symbolic links.
+Disabled by default, enable it only if you trust your file system as nothing is
+done to handle broken links or loops.
+
 ```ini
 [base]
 ; A database URI. See the 'schema' folder for schema creation scripts
@@ -71,6 +75,9 @@ database_uri = sqlite:////var/supysonic/supysonic.db
 
 ; Optional, restrict scanner to these extensions. Default: none
 scanner_extensions = mp3 ogg
+
+; Should the scanner follow symbolic links? Default: no
+follow_symlinks = no
 ```
 
 ## `[webapp]` section

supysonic/cli.py

@@ -251,6 +251,7 @@ def __folder_scan_foreground(self, folders, force):
         scanner = Scanner(
             force=force,
             extensions=extensions,
+            follow_symlinks=self.__config.BASE["follow_symlinks"],
             progress=TimedProgressDisplay(self.stdout),
             on_folder_start=self.__unwatch_folder,
             on_folder_end=self.__watch_folder,

supysonic/config.py

@@ -30,6 +30,7 @@ class DefaultConfig(object):
     BASE = {
         "database_uri": "sqlite:///" + os.path.join(tempdir, "supysonic.db"),
         "scanner_extensions": None,
+        "follow_symlinks": False,
     }
     WEBAPP = {
         "cache_dir": tempdir,

supysonic/daemon/server.py

@@ -82,6 +82,7 @@ def start_scan(self, folders=[], force=False):
         self.__scanner = Scanner(
             force=force,
             extensions=extensions,
+            follow_symlinks=self.__config.BASE["follow_symlinks"],
             on_folder_start=self.__unwatch,
             on_folder_end=self.__watch,
         )

supysonic/frontend/folder.py

@@ -7,17 +7,13 @@
 #
 # Distributed under terms of the GNU AGPLv3 license.
 
-import os.path
-import uuid
-
 from flask import current_app, flash, redirect, render_template, request, url_for
 from pony.orm import ObjectNotFound
 
 from ..daemon.client import DaemonClient
 from ..daemon.exceptions import DaemonUnavailableError
 from ..db import Folder
 from ..managers.folder import FolderManager
-from ..scanner import Scanner
 
 from . import admin_only, frontend
 

supysonic/scanner.py

@@ -59,6 +59,7 @@ def __init__(
         self,
         force=False,
         extensions=None,
+        follow_symlinks=False,
         progress=None,
         on_folder_start=None,
         on_folder_end=None,
@@ -71,6 +72,7 @@ def __init__(
 
         self.__force = force
         self.__extensions = extensions
+        self.__follow_symlinks = follow_symlinks
 
         self.__progress = progress
         self.__on_folder_start = on_folder_start
@@ -131,8 +133,7 @@ def __scan_folder(self, folder):
             for entry in scandir(path):
                 if entry.name.startswith("."):
                     continue
-                # TODO add config setting to allow following symlinks
-                if entry.is_symlink():
+                if entry.is_symlink() and not self.__follow_symlinks:
                     continue
                 elif entry.is_dir():
                     to_scan.append(entry.path)

tests/base/test_scanner.py

@@ -45,7 +45,7 @@ def __temporary_track_copy(self):
             yield tf
 
     def __scan(self, force=False):
-        self.scanner = Scanner(force)
+        self.scanner = Scanner(force=force)
         self.scanner.queue_folder("folder")
         self.scanner.run()