Is there anything wrong with this patch? Thanks.

-------- Original Message --------
Subject: [PATCH scoreboard] adding port entry
Date: Fri, 15 Mar 2002 11:58:31 +0800 (SGT)
From: Stas Bekman <[EMAIL PROTECTED]>
Reply-To: [EMAIL PROTECTED]
To: [EMAIL PROTECTED]

When running port based vhosts we want to see the ports in the
mod_status's output (or other tools working with scoreboard). Otherwise
there is no added value in displaying vhosts entry at all.

The following patch adds the port entry to the worker_score struct. And
I've a patch for mod_status, but will wait first for Jeff to commit my
previous patch and the post it here.

Index: include/scoreboard.h
===================================================================
RCS file: /home/cvspublic/httpd-2.0/include/scoreboard.h,v
retrieving revision 1.44
diff -u -r1.44 scoreboard.h
--- include/scoreboard.h        13 Mar 2002 20:47:42 -0000      1.44
+++ include/scoreboard.h        15 Mar 2002 03:47:46 -0000
@@ -75,6 +75,7 @@
  #include "apr_thread_proc.h"
  #include "apr_portable.h"
  #include "apr_shm.h"
+#include "apr_network_io.h"

  /* Scoreboard file, if there is one */
  #ifndef DEFAULT_SCOREBOARD
@@ -156,6 +157,7 @@
      char client[32];          /* Keep 'em small... */
      char request[64];         /* We just want an idea... */
      char vhost[32];           /* What virtual host is being accessed? */
+    apr_port_t port;           /* On which port */
  };

  typedef struct {
Index: server/scoreboard.c
===================================================================
RCS file: /home/cvspublic/httpd-2.0/server/scoreboard.c,v
retrieving revision 1.62
diff -u -r1.62 scoreboard.c
--- server/scoreboard.c 13 Mar 2002 20:48:00 -0000      1.62
+++ server/scoreboard.c 15 Mar 2002 03:47:46 -0000
@@ -464,6 +464,7 @@
              }
              apr_cpystrn(ws->vhost, r->server->server_hostname,
                          sizeof(ws->vhost));
+            ws->port = r->server->port;
          }
      }




_____________________________________________________________________
Stas Bekman             JAm_pH      --   Just Another mod_perl Hacker
http://stason.org/      mod_perl Guide   http://perl.apache.org/guide
mailto:[EMAIL PROTECTED]  http://ticketmaster.com http://apacheweek.com
http://singlesheaven.com http://perl.apache.org http://perlmonth.com/

-- 


__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

Reply via email to