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

Revert "allow rsyslog to be none"

common/lib/logsettings.py

@@ -84,6 +84,12 @@ 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',
@@ -108,15 +114,6 @@ 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)