[PATCH] ApacheMonitor.c

[David Shane Holden <dp...@yahoo.com>]

This patch fixes AM where it throws an error window when trying to control
the service for lack of query status access.  

I don't want Mladen to feel like I'm stepping on his toes, but I really 
don't like
using the generic access unless there's a good reason to.  So I changed 
all the
service calls to use only the access rights that are needed.  Which IMO 
is the
more logical approach, but I might be wrong.

Shane

RE: [PATCH] ApacheMonitor.c

[Mladen Turk <mt...@mappingsoft.com>]

> -----Original Message-----
> From: David Shane Holden [mailto:dpejesh@yahoo.com] 
> Sent: Friday, July 12, 2002 8:11 PM
> To: dev@httpd.apache.org
> Subject: [PATCH] ApacheMonitor.c
> 
> 
> This patch fixes AM where it throws an error window when 
> trying to control the service for lack of query status access.  
> 
> I don't want Mladen to feel like I'm stepping on his toes, 
Don't worry abot that :-)

> but I really 
> don't like
> using the generic access unless there's a good reason to.  So 
> I changed 
> all the
> service calls to use only the access rights that are needed.  
> Which IMO 
> is the
> more logical approach, but I might be wrong.

I don't see why is that such a problem. I think that if you fail to see
the service but still cannot start it, then what is the purpose of
displaying the AM for the particular user?. Think it's a admin tool, not
a Guest one, and why that user will have to know if some service is
running on that box or not?

MT.