httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marc Slemko <ma...@znep.com>
Subject Re: Agenda for 1.2b7
Date Fri, 07 Feb 1997 06:47:36 GMT
Ok, unless anyone complains I will be applying the following patch.  It
has been approved in concept, just the HTTP_MOVED_PERMANENTLY bit that is
different (from MOVED).

Index: mod_dir.c
===================================================================
RCS file: /export/home/cvs/apache/src/mod_dir.c,v
retrieving revision 1.20
diff -c -r1.20 mod_dir.c
*** mod_dir.c	1997/01/12 20:11:53	1.20
--- mod_dir.c	1997/02/07 06:45:38
***************
*** 783,790 ****
  			 "/", NULL);
  
  	table_set (r->headers_out, "Location",
! 		   construct_url(r->pool, ifile, r->server));
! 	return REDIRECT;
      }
  
      /* KLUDGE --- make the sub_req lookups happen in the right directory.
--- 783,790 ----
  			 "/", NULL);
  
  	table_set (r->headers_out, "Location",
! 		   construct_url(r->pool, ifile, r->server)); 
! 	return HTTP_MOVED_PERMANENTLY;
      }
  
      /* KLUDGE --- make the sub_req lookups happen in the right directory.


On Thu, 6 Feb 1997, Dean Gaudet wrote:

> On Wed, 5 Feb 1997, Marc Slemko wrote:
> 
> > > >   * directory redirect returns 302 not 301, see PR 77
> > > >        Status: easy to fix, but will require testing
> > > 
> > > I reposted the patch against current CVS.  I gave it +1, Marc gave it +1,
> > > Ed gave it +1.  I've been running this one on HotWired as well without
> > > problem.
> > 
> > I was all ready to commit this when I wondered if
> > we should be using HTTP_MOVED_PERMANENTLY instead of MOVED?
> > 
> > I think I think so.
> 
> Yup, MOVED is deprecated isn't it?  +1 on the HTTP_MOVED_PERMANENTLY
> change. 
> 
> Dean
> 


Mime
View raw message