Merge PR #788: Stay logged in to our instances [MNG-1938]

instance/models/mixins/openedx_config.py

@@ -34,6 +34,17 @@ def _get_configuration_variables(self):
         """
         Creates and returns a dictionary of ansible configuration variables for the instance
         """
+        # Vars that need to be set in both EDXAPP_LMS_ENV_EXTRA and EDXAPP_CMS_ENV_EXTRA:
+        extra_config = {
+            "BROKER_HEARTBEAT": 0,
+            "ADDL_INSTALLED_APPS": [],
+            # Stay logged in for one year, not two weeks
+            "SESSION_COOKIE_AGE": 31536000,
+            # Use database-backed sessions to stay logged in across appserver deploys.
+            # (We have EDXAPP_CLEARSESSIONS_CRON_ENABLED to clean up old data.)
+            "SESSION_ENGINE": "django.contrib.sessions.backends.cached_db",
+        }
+
         template = {
             # System
             "COMMON_HOSTNAME": self.server_hostname,
@@ -63,14 +74,10 @@ def _get_configuration_variables(self):
             # Note: use EDXAPP_CMS_ENV_EXTRA and/or EDXAPP_LMS_ENV_EXTRA; don't use
             # EDXAPP_ENV_EXTRA as the other two overwrite this.
             "EDXAPP_CMS_ENV_EXTRA": {
-                "BROKER_HEARTBEAT": 0,
-                "ADDL_INSTALLED_APPS": [
-                ],
+                **extra_config,
             },
             "EDXAPP_LMS_ENV_EXTRA": {
-                "BROKER_HEARTBEAT": 0,
-                "ADDL_INSTALLED_APPS": [
-                ],
+                **extra_config,
                 "HEARTBEAT_EXTENDED_CHECKS": [
                     # celery check is default, but we need to include it here
                     # because we also add other extended checks