cross
New Foswiki release 2.1.6 is available with important security fixes.
Sourceforge foswiki email lists being discontinued. Subscribe to the new Foswiki announce and discuss lists at MailingLists
You are here: Foswiki>Tasks Web>Item908 (23 Feb 2009, KennethLavrsen)Edit Attach

Item908: adding a script suffix seems to confuse the url parsing code

pencil
Priority: Urgent
Current State: Closed
Released In:
Target Release: patch
Applies To: Engine
Component:
Branches:
Reported By: SvenDowideit
Waiting For:
Last Change By: KennethLavrsen
if you add .pl and goto http://yourfoswiki/bin/view.pl the parsing code will tell you it can't fine the Pl topic

will be a pretty trivial unit test to add and then fix - related to the other bug fix i did in this area, and the feature proposal Kenneth kindly made me smile

-- SvenDowideit - 29 Jan 2009

Foswiki core expects PATH_INFO to be something like /Web/Subweb/Topic. IIS adjusts it to /foswiki/bin/view.pl/Web/Subweb/Topic. Foswiki::Engine::CGI::preparePath() already handle this case:

    # SMELL: "The Microsoft Internet Information Server is broken with
    # respect to additional path information. If you use the Perl DLL
    # library, the IIS server will attempt to execute the additional
    # path information as a Perl script. If you use the ordinary file
    # associations mapping, the path information will be present in the
    # environment, but incorrect. The best thing to do is to avoid using
    # additional path information."

    # (...)

    my $cgiScriptPath = $ENV{SCRIPT_NAME};
    $pathInfo =~ s{$cgiScriptPath/}{/}i;

This workaround works to /foswiki/bin/view.pl/ but not to /foswiki/bin/view.pl (notice the trailing slash).

So this workaround should be improved.

-- GilmarSantosJr - 30 Jan 2009

ItemTemplate edit

Summary adding a script suffix seems to confuse the url parsing code
ReportedBy SvenDowideit
Codebase 1.0.0, trunk
SVN Range Foswiki-1.0.0, Thu, 08 Jan 2009, build 1878
AppliesTo Engine
Component
Priority Urgent
CurrentState Closed
WaitingFor
Checkins distro:03af67c2197f
TargetRelease patch
ReleasedIn
Topic revision: r6 - 23 Feb 2009, KennethLavrsen - This page was cached on 24 May 2018 - 07:59.

The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License