Merge pull request #4652 from edx/jarv/only-add-handler-if-rsyslog

allow rsyslog to be none

common/lib/logsettings.py

@@ -84,12 +84,6 @@ def get_logger_config(log_dir,
                 'filters': ['require_debug_false'],
                 'class': 'django.utils.log.AdminEmailHandler'
             },
-            'syslogger-remote': {
-                'level': 'INFO',
-                'class': 'logging.handlers.SysLogHandler',
-                'address': syslog_addr,
-                'formatter': 'syslog_format',
-            },
             'newrelic': {
                 'level': 'ERROR',
                 'class': 'lms.lib.newrelic_logging.NewRelicHandler',
@@ -114,6 +108,15 @@ def get_logger_config(log_dir,
             },
         }
     }
+    if syslog_addr:
+        logger_config['handlers'].update({
+            'syslogger-remote': {
+                'level': 'INFO',
+                'class': 'logging.handlers.SysLogHandler',
+                'address': syslog_addr,
+                'formatter': 'syslog_format',
+            },
+        })
 
     if dev_env:
         tracking_file_loc = os.path.join(log_dir, tracking_filename)