phpFreeChat has been getting a lot of heavy use on my site recently, and that turned up a problem that I didn't see before - in a part of the code I hadn't worked on. This statement in phpfrechat_room():

if (is_numeric($user->uid)) {
  $_SESSION['drupal_user'] = $user;
}

must be commented out or removed! It's unnecessary, and adds a huge amount of data to $_SESSION - enough to bring down a shared server. (It was my Web host who very snarkily brought this to my attention.)

I have to go out for a while now. When I get back, I'll try to upload a new version of the phpFreeChat module that incorporates this fix and a previous fix I made buit haven't uploaded yet. It's Christmas eve, though, so I might not be able to do it today.

Comments

permutations’s picture

Version: 5.x-1.x-dev » 5.x-1.3
Status: Active » Fixed

Fixed in Version 1.3.

Status: Fixed » Closed (fixed)

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