Project

General

Profile

Actions
Copy link

Bug #2740

closed

mod_status with "?auto" modifier not showing keep-alive (k) status on Scoreboard

Added by idfnet over 8 years ago. Updated over 8 years ago.

Status:
Fixed
Priority:
Normal
Category:
mod_status
Target version:
1.4.41
ASK QUESTIONS IN Forums:

Description

The Scoreboard generated by mod_status for the URL "/server-status" differs from the one generated for the URL "/server-status?auto":
With the "?auto" modifier, connections in state keep-alive (k) are not listed.

Sample output without modifier:

krrkWkkkkCrCkrrCrkrkrCWrkkWrkkkkkrkkkkkkkrkCrrkkrC
rkCkrrrCkkrkkkrrCCkrkCrCrrkkrCCrkCkrrrCkrrkCkkrrrk
rkWrkkrCrrrrkCrrrkkkkrrrrkkrrrkrrkrWkrrrrCkkrCrrCk
kkkkrCkkkkrhrrCCkWWrkkWrrrWrrkWCCrCrCCrrrrrrkrkkrr
rkkkkkrCCrWCrkrkrCkCrkCrhrrrCrrkCrCrkkkkrCrrrCkCCC
rrkrrrkkWrkCWkkWWrrrkrkrkrkrkkkrrhrrrrkCkWkkrrkrrr
kCrkrkkrkrWrrkrkkkkrkrrrrrrCCrkkrrkkrrkkrrCkrrCrkk
WCkrkrrkrrkkkkrkkrWrCkCrrrrCCrkkrkCCCrrrkrkrrrCCrC
CkkkCCkWkrhhkrrCkkrkrkkCrrCCCCCCkkrrkrrrrrrkkkkkkk
krkkrkrkrkrhr

Sample output with modifier (linebreaks added):

rrrrWrrrrCrCrrrCrrrrrCWrrrWrrrrrrrrrrrrrrrrCrrrrrC
rrCrrrrCrrrrrrrrCCrrrCrCrrWrrCCrrCrrrrCrrrrCrrrrrr
rrWrrrrCrrrrrCrhrrrrrrrrrrrrrrrrrrrWrrrrrrrrrCrrCr
rrrrrCrrrrrWrrCCrWWrrrWrrrWrrrWCCrCrrCrrrrrrrrrrrr
rrrrrrrCCrWCrrrrrCrCrrCrrrrrCrrrCrCrrrrrrCrrrCrCCC
rrrrrrrrWrrCWrrWWrrrrrrrrrrrrrrrrhrrrrrCrWrrrrrrrr
rCrrrrrrrrWrrrrrrrrrrrrrrrrCCrrrrrrrrrrrrrCrrrCrrr
WCrrrrrrrrrrrrrrrrrrCrCrrrrCCrrrrrCCCrrrrrrrrrCCrC
CrrrCCrrrrrhrrrCrrrrrrrCrrCCCCCCrrrrrrrrrrrrrrrrrr
rrrrrrrrrWrrrr____________________________________
__________________________________________________
__________________________________________________
__________________________________________________
__________________________________________________
__________________________________________________
__________________

Actions
Copy link
 #1

Updated by gstrauss over 8 years ago

  • Status changed from New to Patch Pending
  • Target version changed from 1.4.x to 1.4.41

Apparently, this was only partially added in 2009 in https://redmine.lighttpd.net/issues/1202
Here's the change for text output (obtained with /server-status?auto)

diff --git a/src/mod_status.c b/src/mod_status.c
index a37b6cd..647f403 100644
--- a/src/mod_status.c
+++ b/src/mod_status.c
@@ -638,7 +638,10 @@ static handler_t mod_status_handle_server_status_text(server *srv, connection *c
        buffer_append_string_len(b, CONST_STR_LEN("Scoreboard: "));
        for (k = 0; k < srv->conns->used; k++) {
                connection *c = srv->conns->ptr[k];
-               const char *state = connection_get_short_state(c->state);
+               const char *state =
+                 (CON_STATE_READ == c->state && !buffer_string_is_empty(c->request.orig_uri))
+                   ? "k" 
+                   : connection_get_short_state(c->state);
                buffer_append_string_len(b, state, 1);
        }
        for (l = 0; l < srv->conns->size - srv->conns->used; l++) {

Actions
Copy link
 #2

Updated by gstrauss over 8 years ago

  • Status changed from Patch Pending to Fixed
  • % Done changed from 0 to 100
Actions
Copy link

Also available in: Atom