Need a way to purge a user who logs out from the node.js server auth cache.

Comments

Anonymous’s picture

Anonymous (not verified) commented
Category: bug » task

yes! on the TODO.

we need an overall API here, so that one can view the innards of a node.js instance, and control it.

mikey_p’s picture

mikey_p commented

Should we make this a whole REST API sort of thing? would there ever be any benefit to pushing sessions/auth into node.js?

Anonymous’s picture

Anonymous (not verified) commented

i'm not convinced its necessary to push sessions into node.js. it *might* be something to investigate from a performance point of view later, but right now, its not necessary, because node.js code defers to drupal for all un-authed clients anyway.

the node.js code uses express.js (http://expressjs.com/index.html), so it should be pretty easy to wire all of this up.

Anonymous’s picture

Anonymous (not verified) commented
Version: » 7.x-1.0-alpha4

implemented this here:

http://drupalcode.org/project/nodejs.git/commit/111e196

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed
mikey_p’s picture

mikey_p commented
Status: Fixed » Needs work 
+ * Implements hook_user_logout().
+ */
+function hook_user_logout($account) {
+  nodejs_kick_user($account->uid);
+}
+
+/**

Should be nodejs_user_logout($account)

Anonymous’s picture

Anonymous (not verified) commented

Doh!

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Fixed

fixed for realz:

http://drupalcode.org/project/nodejs.git/commit/cebc5b9

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 111e196 on 6.x-1.x, 7.x-1.x, drupalconlondon, auth-refactor, 8.x-1.x, 8.x-1.x-head by beejeebus: 
    #1068904: kick a user off the node.js server when they log out of drupal...