Overriding session timeout

twlichty · March 23, 2020, 5:14pm

I’m trying to override the session timeout for students. It’s currently set to 4 weeks. I noticed that the session expiry is hard coded in the _handle_successful_authentication_and_login function:

github.com

edx/edx-platform/blob/master/openedx/core/djangoapps/user_authn/views/login.py#L247


    """
    Handles clearing the failed login counter, login tracking, and setting session timeout.
    """
    if LoginFailures.is_feature_enabled():
        LoginFailures.clear_lockout_counter(user)


    _track_user_login(user, request)


    try:
        django_login(request, user)
        request.session.set_expiry(604800 * 4)
        log.debug("Setting user session expiry to 4 weeks")
    except Exception as exc:
        AUDIT_LOG.critical("Login failed - Could not create session. Is memcached running?")
        log.critical("Login failed - Could not create session. Is memcached running?")
        log.exception(exc)
        raise




def _track_user_login(user, request):
    """

Is there a reason for it to be hard coded? Is there an easy way of overriding it?

Thanks.

guruprasad · April 20, 2020, 12:10pm

@twlichty, you are welcome to create an upstream PR against the edx-platform repository that makes this configurable via /edx/etc/lms.yml (which has replaced the /edx/app/edxapp/lms.env.json after the Ironwood release) through a Django settings variable.

The most recent update which set this hard-coded value was this PR. But I cannot find any reason as to why this was not made configurable.

Topic		Replies	Views
LMS failure: 504 Timeout Site Operations Help	3	1067	November 5, 2020
Getting rate-limited request error on devstack LMS Development devstack , maple	5	1062	May 17, 2022
Codejail timeout advanced setting? Development	2	500	October 18, 2022
How to implement redis instead of memcached Site Operators how-to	5	1538	February 26, 2020
'long' object has no attribute 'is_anonymous' Site Operations Help	3	857	November 20, 2020

Overriding session timeout

Related Topics