Oliver - I think you're right, this was a mistake. It was the result of

   https://github.com/jonswar/perl-mason/issues/8

I'm not sure what the right behavior should be. For the sake of good SEO, you 
generally want canonical urls and it's suboptimal to allow /foo/bar and 
/foo/bar/ to map to the exact same thing. One of them should redirect to the 
other. In the old Apache world, /foo/bar would redirect to /foo/bar/, but these 
days it is much more common to have urls without trailing slashes.

Feedback welcome. I agree, though, the current situation is very un-DWIM-ish.

Jon

On Jan 4, 2013, at 11:36 AM, Oliver Paukstadt <pst...@sourcentral.org> wrote:

> Hi
> 
> while reading through latest Mason::Manual::RequestDispatch and
> Mason::Interp::_build_match_request_path to understand trailing slash
> handling I found that my installation was Mason-2.19 and latest was 2.20
> with the trailing slash change for directories.
> 
> So I updated and everything looks broken now. ;-)
> 
> As far as I can see an access to a directory with trailing slash
> (except / itself) will never match an index.mc until allow_path_info is
> set to 1.
> This is the only way to get the check  in _build_match_request_path for
> index.mc valid, because path_info now is '/' and it is not a dhandler.
> 
> This is really bad, what was the reason to do it this way?
> Maybe the simplest fix could be a check similar to "is_dhandler" like
> is_index to get index files delivered without allow_path_info set.
> 
>> From my point of view (is_index and path_info eq '') could lead to a
> redirect adding a slash to path if attribute force_directory_slash is
> set.
> 
> Regards,
> Oliver Paukstadt
> -- 
> Oliver Paukstadt <pst...@sourcentral.org>
> 
> 
> ------------------------------------------------------------------------------
> Master HTML5, CSS3, ASP.NET, MVC, AJAX, Knockout.js, Web API and
> much more. Get web development skills now with LearnDevNow -
> 350+ hours of step-by-step video tutorials by Microsoft MVPs and experts.
> SALE $99.99 this month only -- learn more at:
> http://p.sf.net/sfu/learnmore_122812
> _______________________________________________
> Mason-users mailing list
> Mason-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/mason-users


------------------------------------------------------------------------------
Master HTML5, CSS3, ASP.NET, MVC, AJAX, Knockout.js, Web API and
much more. Get web development skills now with LearnDevNow -
350+ hours of step-by-step video tutorials by Microsoft MVPs and experts.
SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122812
_______________________________________________
Mason-users mailing list
Mason-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mason-users

Reply via email to