Re: Endless loop in split_on_bdry() of library/parser_multipart.c?

2006-06-01 Thread Bojan Smojver

Quoting Philip M. Gollucci [EMAIL PROTECTED]:


Seems to be Fedora Core X specific.


Happens on x84_64 as well and with 2.07. Rebuilding the package in  
Fedora Extras 5 now.


--
Bojan


debug apache

2006-06-01 Thread Alexander Lazic

Hi,

i have read http://httpd.apache.org/dev/debugging.html and asked me
follwing:

Is it possible to get a logentry from the hooks in apache to see the
flow of a request?

eg:

$REQUEST accepted on $FD
$REQUEST handled by $MOD_xxx
$REQUEST handled by $MOD_xxx
$REQUEST handled by $MOD_xxx
$REQUEST handled by $MOD_xxx
$REQUEST send to client

or

$REQUEST accepted on $FD
$REQUEST handled by $MOD_xxx
$REQUEST handled by $MOD_PROXY_xxx
$REQUEST send on $FD to $PROXY
$REQUEST handled by $MOD_xxx
$REQUEST handled by $MOD_xxx
$REQUEST send to client

I hope i have asked clean enough so that anybody can answer my question
;-)

regards

Alex


Re: [EMAIL PROTECTED] debug apache

2006-06-01 Thread Mathieu CARBONNEAUX




it's a very good idea to add this!From: Alexander Lazic [mailto:[EMAIL PROTECTED]To: dev@httpd.apache.orgSent: Thu, 01 Jun 2006 08:14:11 +0200Subject: [EMAIL PROTECTED] debug apacheHi,i have read http://httpd.apache.org/dev/debugging.html and asked mefollwing:Is it possible to get a logentry from the hooks in apache to see theflow of a request?eg:$REQUEST accepted on $FD$REQUEST handled by $MOD_xxx$REQUEST handled by $MOD_xxx$REQUEST handled by $MOD_xxx$REQUEST handled by $MOD_xxx$REQUEST send to clientor$REQUEST accepted on $FD$REQUEST handled by $MOD_xxx$REQUEST handled by $MOD_PROXY_xxx$REQUEST send on $FD to $PROXY$REQUEST handled by $MOD_xxx$REQUEST handled by $MOD_xxx$REQUEST send to clientI hope i have asked clean enough so that anybody can answer my question;-)regardsAlex


Re: svn commit: r395211 - /httpd/httpd/trunk/configure.in

2006-06-01 Thread Joe Orton
On Thu, May 25, 2006 at 09:01:44PM +0200, Ruediger Pluem wrote:
 On 04/19/2006 12:57 PM,
  Author: jorton
  Date: Wed Apr 19 03:57:20 2006
  New Revision: 395211
...
  +if test x$with_included_apr = xyes; then
  +   apr_found=reconfig
  +   apr_config=$srcdir/srclib/apr/apr-1-config
  +else 
 
 Is it really a good idea to hardcode apr-1-config or shouldn't it be 
 something like
 apr-?-config (Same question for apu-1-config below)?

The 1 requirement is hardcoded in the APR_FIND_APx calls anyway, so I 
don't think this is a big deal.  They could all be replaced with some 
variable I suppose.

  +if test x$with_included_apr = xyes; then
  +   apu_found=reconfig
  +   apu_config=${srcdir}/srclib/apr-util/apu-1-config
  +else 
  +   APR_FIND_APU($srcdir/srclib/apr-util, ./srclib/apr-util, 1, 1)
  +fi

joe


Re: httpd-win.conf broken on trunk

2006-06-01 Thread William A. Rowe, Jr.

Garrett Rooney wrote:

It looks like the trunk version of httpd has been busted on win32 ever
since the big authz refactoring in r368027.  I'd be happy to make the
changes to get it working again, if someone would be so kind as to
point me to some sort of documentation on how exactly one goes from
The old way to The new way...


Garrett please take a look at my commits a couple hrs ago, see if they
are in sync with the direction you want to go.  Need to look to backporting
as these changes are needed on 2.2 IIRC.

Bill


Re: Endless loop in split_on_bdry() of library/parser_multipart.c?

2006-06-01 Thread Philip M. Gollucci

Bojan Smojver wrote:

Quoting Joe Schaefer [EMAIL PROTECTED]:


At least now it's a bit clearer why the no-strict-aliasing
optimization is getting confused ;-)


Hey, speak for yourself ;-)


FreeBSD 6.1-RELEASE
gcc version 4.1.2 20060526 (prerelease)
gcc version 3.4.4 20050518

default maintainer mode CFLAGS work and I can also use combination of these:
-O3 -fno-strict-aliasing -fomit-frame-pointer

Seems to be Fedora Core X specific.

Should we figure out how to re-write the loop so it doesn't matter,
add a prereq check in build/version_check.pl, or just leave it ?



--

Philip M. Gollucci ([EMAIL PROTECTED]) 323.219.4708
Consultant / http://p6m7g8.net/Resume/resume.shtml
Senior Software Engineer - TicketMaster - http://ticketmaster.com
1024D/A79997FA F357 0FDD 2301 6296 690F  6A47 D55A 7172 A799 97F

It takes a minute to have a crush on someone, an hour to like someone,
and a day to love someone, but it takes a lifetime to forget someone...


Re: Endless loop in split_on_bdry() of library/parser_multipart.c?

2006-06-01 Thread Bojan Smojver
On Thu, 2006-06-01 at 02:34 -0700, Philip M. Gollucci wrote:

 Seems to be Fedora Core X specific.

Yeah, very possible.

 Should we figure out how to re-write the loop so it doesn't matter,
 add a prereq check in build/version_check.pl, or just leave it ?

I maintain Fedora Extras libapreq2 package and I already applied the
relevant flag. Let's see what compiler folks tell us and then we can
rewrite if needed.

-- 
Bojan



Re: svn commit: r395211 - /httpd/httpd/trunk/configure.in

2006-06-01 Thread Joe Orton
On Thu, Jun 01, 2006 at 12:27:33PM +0200, Plüm, Rüdiger, VF EITO wrote:
  The 1 requirement is hardcoded in the APR_FIND_APx calls 
  anyway, so I 
  don't think this is a big deal.  They could all be replaced with some 
  variable I suppose.
 
 Can you replace them with a variable, such that later on only this variable
 needs to be adjusted? Then I would be +1 for backport immediately.

Sure, done, r410828.

joe


Re: svn commit: r395211 - /httpd/httpd/trunk/configure.in

2006-06-01 Thread Plüm , Rüdiger , VF EITO


 -Ursprüngliche Nachricht-
 Von: Joe Orton 
  
  Can you replace them with a variable, such that later on 
 only this variable
  needs to be adjusted? Then I would be +1 for backport immediately.
 
 Sure, done, r410828.

Thanks. +1 on backport to 2.2.x

Regards

Rüdiger



Re: httpd-win.conf broken on trunk

2006-06-01 Thread Garrett Rooney

On 6/1/06, William A. Rowe, Jr. [EMAIL PROTECTED] wrote:

Garrett Rooney wrote:
 It looks like the trunk version of httpd has been busted on win32 ever
 since the big authz refactoring in r368027.  I'd be happy to make the
 changes to get it working again, if someone would be so kind as to
 point me to some sort of documentation on how exactly one goes from
 The old way to The new way...

Garrett please take a look at my commits a couple hrs ago, see if they
are in sync with the direction you want to go.  Need to look to backporting
as these changes are needed on 2.2 IIRC.


They seem reasonable to me, although I haven't tried them out.

One thing that seems odd, it looks like Makefile.win is still copying
docs/conf/httpd-win.conf to conf/httpd.conf.default, isn't the goal of
the previous changes to get a massaged version of httpd-std.conf.in
there?

-garrett


Re: svn commit: r410757 - /httpd/httpd/trunk/docs/conf/extra/httpd-info.conf.in

2006-06-01 Thread Joshua Slive

On 6/1/06, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:


  Add example/default allow-from for localhost, please???



 Location /server-status
 SetHandler server-status
 Require host .example.com
+Allow from 127
 /Location


I think you are looking for
Require ip 127
or something like that.  But I'm not sure why it is necessary.  We've
always just used .example.com here, haven't we?

Joshua.


Re: svn commit: r410758 - /httpd/httpd/trunk/docs/conf/extra/httpd-dav.conf.in

2006-06-01 Thread Joshua Slive

On 6/1/06, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:


  Ewww... can't we be consistant with our workfiles paths?




-DavLockDB @@ServerRoot@@/var/DavLock
+DavLockDB @exp_runtimedir@/DavLock


I believe the issue here is that exp_runtimedir is writable only by
root, while DavLockDB needs to be writable by child processes.  I
tried to introduce the var directory in the docs as an example of a
child-writable directory, and have tried to use it consistently as
such.  I consider the stuff in conf/extra to be essentially
documentation, not necessarily working configurations.

Joshua.


Re: svn commit: r410761 - /httpd/httpd/trunk/docs/conf/extra/httpd-mpm.conf.in

2006-06-01 Thread Joshua Slive

On 6/1/06, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:

Author: wrowe
Date: Wed May 31 22:42:13 2006
New Revision: 410761

URL: http://svn.apache.org/viewvc?rev=410761view=rev
Log:

  That's the point, isn't it?  All mpm's in one basket?


Sure, but windows has its own config file where we left the mpm_winnt
stuff in.  So putting it here is duplication.

Joshua.


Re: svn commit: r410757 - /httpd/httpd/trunk/docs/conf/extra/httpd-info.conf.in

2006-06-01 Thread Joe Orton
On Thu, Jun 01, 2006 at 09:17:27AM -0400, Joshua Slive wrote:
 On 6/1/06, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:
   Add example/default allow-from for localhost, please???
 
  Location /server-status
  SetHandler server-status
  Require host .example.com
 +Allow from 127
  /Location
 
 I think you are looking for
 Require ip 127
 or something like that.  But I'm not sure why it is necessary.  We've
 always just used .example.com here, haven't we?

I agree it would be better to omit it.  It's misleading to say that 
127.0.0.0/8 is sufficient to allow access from the local host anyway; on 
any IPv6-enabled host you need to allow access from ::1 too.

joe


Re: svn commit: r410761 - /httpd/httpd/trunk/docs/conf/extra/httpd-mpm.conf.in

2006-06-01 Thread Garrett Rooney

On 6/1/06, Joshua Slive [EMAIL PROTECTED] wrote:

On 6/1/06, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:
 Author: wrowe
 Date: Wed May 31 22:42:13 2006
 New Revision: 410761

 URL: http://svn.apache.org/viewvc?rev=410761view=rev
 Log:

   That's the point, isn't it?  All mpm's in one basket?

Sure, but windows has its own config file where we left the mpm_winnt
stuff in.  So putting it here is duplication.


Considering that the httpd-win.conf file is so out of date that it
doesn't even work, I think Bill's goal here is to make it use the same
configuration files as the rest of the platforms...

-garrett


Re: There should be a filter spec

2006-06-01 Thread Garrett Rooney

On 5/26/06, Joachim Zobel [EMAIL PROTECTED] wrote:

Hi.

I need a specification that tells me what a filter must/should and must
not/should not do. Is there something alike?

My actual problem is that I have a filter that if nothing was modified
drops all content, sets f-r-status to 304 and sends an eos bucket.
This works. If I pass the output through mod_transform the browser does
get a 200 without content instead.

So probably mod_transform is broken. But it is difficult to decide what
is broken without such a spec.


I've had similar issues lately, it's very unclear how a filter setting
f-r-status or f-r-status_line should act.  Depending on what
modules your working with, that may be enough to get an error out to
the browser, and it may not.  Some specific rules about that sort of
thing would be quite useful...

-garrett


Re: There should be a filter spec

2006-06-01 Thread Plüm , Rüdiger , VF EITO


 von Garrett Rooney
 Gesendet: Donnerstag, 1. Juni 2006 16:32
 An: dev@httpd.apache.org; [EMAIL PROTECTED]
 Betreff: Re: There should be a filter spec
 

 
 I've had similar issues lately, it's very unclear how a filter setting
 f-r-status or f-r-status_line should act.  Depending on what
 modules your working with, that may be enough to get an error out to
 the browser, and it may not.  Some specific rules about that sort of
 thing would be quite useful...

As far as I remember there had been also a discussion on who owns a brigade.
So who has to call / should not call apr_brigade_destroy / apr_brigade_cleanup
in the filter chain. I think rules for this would be also useful.

Regards

Rüdiger



Re: [PATCH] setenvif filter

2006-06-01 Thread Brian Akins

Francois Pesce wrote:

These patches may fix the r-content_type behaviour. Are you OK with it ?



+1



--
Brian Akins
Lead Systems Engineer
CNN Internet Technologies


Re: httpd-win.conf broken on trunk

2006-06-01 Thread William A. Rowe, Jr.

Garrett Rooney wrote:


One thing that seems odd, it looks like Makefile.win is still copying
docs/conf/httpd-win.conf to conf/httpd.conf.default, isn't the goal of
the previous changes to get a massaged version of httpd-std.conf.in
there?


future tense, yes.  These were just glaring things I tripped over and
streamlining the extras copy that were already committed.

I think the one issue with using httpd-std.conf.in is the list of modules
to load; netware has an awk script which accomplishes this - perhaps we
piggyback on the same script?

Bill


Re: svn commit: r410757 - /httpd/httpd/trunk/docs/conf/extra/httpd-info.conf.in

2006-06-01 Thread William A. Rowe, Jr.

Joe Orton wrote:

On Thu, Jun 01, 2006 at 09:17:27AM -0400, Joshua Slive wrote:


On 6/1/06, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:


Add example/default allow-from for localhost, please???



Location /server-status
   SetHandler server-status
   Require host .example.com
+Allow from 127
/Location


I think you are looking for Require ip 127 or something like that.  


Of course, thank you...


But I'm not sure why it is necessary.
We've always just used .example.com here, haven't we?


These were;

#Order deny,allow
#Deny from all
#Allow from .your_domain.com

which is slight different than saying .example.com to someone trying
to understand these modules.

I agree it would be better to omit it.  It's misleading to say that 
127.0.0.0/8 is sufficient to allow access from the local host anyway; on 
any IPv6-enabled host you need to allow access from ::1 too.


Require ip 127 ::1   or whatnot then?  Yes, this is a good point, perhaps
using localhost makes more sense (sure folks break that dns entry, but better
that, than to fight ip local syntaxes ;-)

If consensus says ignore this issue (default to allowing the admin to look at
their local machine's status) then I'll revert this commit rather than fix.

Bill


[PATCH] add notfound to mod_rewrite rule

2006-06-01 Thread Brian Akins
patch against 2.2.2 (but should work on most other versions).  Adds the 
notfound (NF) flag to rewriterule that will case it to return 
HTTP_NOT_FOUND for matched, similar to the gone and forbidden options.




--- mod_rewrite.c.bak   2006-06-01 15:25:48.0 -0400
+++ mod_rewrite.c   2006-06-01 15:29:18.0 -0400
@@ -3264,6 +3264,11 @@
 || !strcasecmp(key, ocase)) {/* nocase */
 cfg-flags |= RULEFLAG_NOCASE;
 }
+else if (((*key == 'F' || *key == 'f')  !key[1])
+ || !strcasecmp(key, otfound)) {   /* notfound */
+cfg-flags |= (RULEFLAG_STATUS | RULEFLAG_NOSUB);
+cfg-forced_responsecode = HTTP_NOT_FOUND;
+}
 else {
 ++error;
 }


--
Brian Akins
Lead Systems Engineer
CNN Internet Technologies


Re: [PATCH] add notfound to mod_rewrite rule

2006-06-01 Thread André Malo
* Brian Akins wrote:

 patch against 2.2.2 (but should work on most other versions).  Adds the
 notfound (NF) flag to rewriterule that will case it to return
 HTTP_NOT_FOUND for matched, similar to the gone and forbidden options.

We don't need this, because you can use [R=404]. F and G are already just 
syntactic sugar.

nd
-- 
Umfassendes Werk (auch fuer Umsteiger vom Apache 1.3)
  -- aus einer Rezension

http://pub.perlig.de/books.html#apache2


Re: [PATCH] add notfound to mod_rewrite rule

2006-06-01 Thread Brian Akins

André Malo wrote:

We don't need this, because you can use [R=404]. F and G are already just 
syntactic sugar.




From the docs:

'redirect|R  [=code]' (force redirect)
Prefix Substitution with http://thishost[:thisport]/ (which makes the 
new URL a URI) to force a external redirection. If no code is given, a 
HTTP response of 302 (MOVED TEMPORARILY) will be returned. If you want 
to use other response codes in the range 300-400, simply specify the 
appropriate number or use one of the following symbolic names: temp 
(default), permanent, seeother. Use this for rules to canonicalize the 
URL and return it to the client - to translate ``/~'' into ``/u/'', or 
to always append a slash to /u/user, etc.



So do the docs need to be updated to say: return arbitrary http code?

BTW, this does work.  patch withdrawn.  somebody needs to fix docs...

--
Brian Akins
Lead Systems Engineer
CNN Internet Technologies


Re: [PATCH] add notfound to mod_rewrite rule

2006-06-01 Thread André Malo
* Brian Akins wrote:

 'redirect|R  [=code]' (force redirect)
 Prefix Substitution with http://thishost[:thisport]/ (which makes the
 new URL a URI) to force a external redirection. If no code is given, a
 HTTP response of 302 (MOVED TEMPORARILY) will be returned. If you want
 to use other response codes in the range 300-400, simply specify the
 appropriate number or use one of the following symbolic names: temp
 (default), permanent, seeother. Use this for rules to canonicalize the
 URL and return it to the client - to translate ``/~'' into ``/u/'', or
 to always append a slash to /u/user, etc.


 So do the docs need to be updated to say: return arbitrary http code?

 BTW, this does work.  patch withdrawn.  somebody needs to fix docs...

Yeah, good point. This could really need some better wording. ;)

nd
-- 
Umfassendes Werk (auch fuer Umsteiger vom Apache 1.3)
  -- aus einer Rezension

http://pub.perlig.de/books.html#apache2