moodle-an-hochschulen / moodle-local_session_keepalive Goto Github PK

Currently, the plugin keeps a Moodle session alive forever. It would be worthwhile to add support for a final end, for example after 7 days, so that the browser does not ping the server forever.

In Moodle 3.8 / MDL-34498, Moodle Core changed the Ajax Endpoint for keeping sessions alive, which this plugin used up to then, to a real webservice (see moodle/moodle@4e1ee0b).

To keep things working, the removed script is shipped within this plugin since then (see af992c1).

However, it would be good to re-build the mechanism to keep the session alive to use real Moodle webservices as well

Sorry for posting my question here. I am really interested whether your pluging works correctly with moodle 3.1?
Petr.

Same problem as in moodle-an-hochschulen/moodle-local_profiletheme#6

It would be a wonderful addition to add capabilities or exclude roles from keepalive sessions. For example, it may be good for an admin to keep a session a live, but for students I think it would be better to limit it to certain time.

If this plugin would be modified to include this, it would be wonderful.

Thanks,

https://github.com/moodleuulm/moodle-local_session_keepalive/blob/4b5404bf8c460df5e416556913eb919d4b74639f/amd/src/keepalive.js#L36

Just a reminder to replace it with another call.

1. On https://github.com/moodleuulm/moodle-local_session_keepalive#1-general-functionality, a note should be added that the keepalive interval has to be smaller (but not much smaller) than the Moodle session lifetime.
2. Optionally, it would be nice to have a check and warning message on settings.php if the proportion of both values does not make sense.

The H5P Editor is loaded in an iFrame which means that the plugin does not catch changes in H5P editor forms. This should be improved, if possible.

https://github.com/moodleuulm/moodle-local_session_keepalive/blob/master/amd/src/keepalive.js

Has the same problem as reported in the other plugin moodle-an-hochschulen/moodle-local_maintenance_livecheck#1

We need to set the minimal interval to session timeout with 1, not 10

P.S. PHP 7.1 in production for 1 week work well - thanks for plugin