I think on sites where nodejs isn't loaded everywhere, you can get into situations where the nodejs session key isn't always set, so when a user logs out, there's no reason (or even way) to log them out of the nodejs server. This is the error I've seen:

Notice: Undefined index: nodejs_config in nodejs_user_logout() (line 520 of /sites/all/modules/nodejs/nodejs.module).

Attached patch should fix this.

CommentFileSizeAuthor
nodejs-logout_authcheck.patch452 byteschellman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chellman created an issue. See original summary.

glekli’s picture

Status: Needs review » Fixed

Great catch. Thank you. This has been committed.

Status: Fixed » Closed (fixed)

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

joelstein’s picture

The same bug exists in Drupal 8. Here's the follow-up issue:

#2987687: PHP notice when logging out