Hi there, I'm preparing a patch to get the number of connected/logged-in/registered users on a Drupal portal.

Comments

ywarnier’s picture

StatusFileSize
new2.5 KB

Here it is. Let me know if you accept it. It's quite handful to me and I intent to use it to write a small Munin plugin to track the number of users connected on my sites throughout the day. Will let you know when I get this done.

ywarnier’s picture

Status: Active » Needs review
moshe weitzman’s picture

Status: Needs review » Closed (won't fix)

Not appropriate for drush core. Maybe try the munin project.

msonnabaum’s picture

I don't know, it might be useful to have a more general drush stats or reporting command that would be consumed over snmp by cacti, munin, or a hosted monitoring app.

I'm not sold on this implementation, but I don't know that it can't be generalized for core.

ywarnier’s picture

I don't get why it wouldn't be considered useful. Isn't Drush about improving the management of Drupal from the command line? Isn't that considered useful on shared hosting servers to be able to know how many users are currently connected (given it generally has massive impact on your server load)? I mentioned Munin, but I'm actually using it live whenever there is load on my server to try and identify where the load might come from.

Can you recommend a place (inside Drupal) where this feature would be better accepted then?

Should I then start a new project just for this feature? Seems like a huge overhead to me.

I'm a bit confused as to why a feature that doesn't have negative impact would be rejected with such little explanation. I can take the responsibility to maintain the feature if this is the problem... In any case, an explanation would be appreciated.

Quoting the project homepage: "drush is a command line shell and scripting interface for Drupal, a veritable Swiss Army knife designed to make life easier for those of us who spend some of our working hours hacking away at the command prompt."
This feature saves me hours per month checking up on site statuses.

tunic’s picture

I guess moshe weitzman don't want this patch to be included in core because is not a core funcionality.

You can develop just a Drush extension with this functionality, a Drupal.org managed module that only includes a drush extension, not a real drush module (I think drush make was initially a pure drush extension module, for example).

Personally, I think it's a very good functionality, thanks for sharing.