django-security icon indicating copy to clipboard operation
django-security copied to clipboard

SessionSecurityMiddleware Client Activity Keep-Alive

Open claytondaley opened this issue 9 years ago • 3 comments

I really like the all-in-one convenience of django-security, but the SessionSecurityMiddleware implementation lacks the client-side keep-alive available in django-session-security. The keep-alive is important to us because our product is used to guide a conversation with a customer so our users are often "active" on a page without server-side interaction.

Any interest adding a keep-alive feature to django-security? If so, what approach would you prefer? The licenses look compatible so it seems like any of the following would work:

  • Replace SessionSecurityMiddleware with the django-session-security implementation
  • Port the JS code to SessionSecurityMiddleware
  • Include both in django-security

claytondaley avatar Aug 17 '16 14:08 claytondaley

Interesting. We built our own little keep alive thing for our own application. I don't know if a better solution is to make django-session-security a dependency of django-security and just expose that. (Seems weird to just grab all of their code.)

funkaoshi avatar Aug 24 '16 18:08 funkaoshi

I ended up trading emails with James (author of DSS) to discuss a vulnerability. In that same private email chain, I mentioned this project and he said:

If someone contributed the client side script to django-security it would be great !

So the second option is already 100% approved. I'm a fan of a combined project so a single base of users/developers can ticket, fix, and release the combined codebase without needing commit access to both (thought I suspect James would facilitate that option as well).

claytondaley avatar Aug 24 '16 19:08 claytondaley

Ah cool. It's a bit hectic here, but i'll see if we can have someone take a look at this.

funkaoshi avatar Sep 15 '16 14:09 funkaoshi