On Thu, Aug 03, 2006 at 10:18:07AM +0200, Jakub Narebski wrote:
> <opublikowany i wys?any>
>
> Marc Singer wrote:
>
>
> > I did some debugging on the latest repo version. The lines
> >
> > our $project = ($cgi->param('p') || $ENV{'PATH_INFO'});
> > if (defined $project) {
> > ...
> >
> > are being executed even though the url is
> >
> > http://server/git
> >
> > I think that the problem is that Cherokee translates the request URL
> > into
> >
> > http://server/git/
> >
> > which means that the $ENV{'PATH_INFO'} is the string "/" insted of
> > being undefined.
>
> Try changing
>
> my $project = ($cgi->param('p') || $ENV{'PATH_INFO'});
> if (defined $project) {
> $project =~ s|^/||; $project =~ s|/$||;
>
> to
>
> my $project = ($cgi->param('p') || $ENV{'PATH_INFO'});
> $project =~ s|^/||; $project =~ s|/$||;
> if (defined $project && $project) {
>
> (and send patch if it works, please).
That isn't enough. I did something like that when I was exploring the
script. While the change *does* eliminate the 403 error, it doesn't
make the rest of the script work properly. All of the links return to
the same page that lists the projects.
>
> --
> Jakub Narebski
> Warsaw, Poland
> ShadeHawk on #git
>
>
> -
> : send the line "unsubscribe git" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
-
: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html