[Dancer-users] Conditional HTTP proposal

[Oleg A. Mamontov]

On Feb 12, 2011, at 8:00 AM, Flavio Poletti wrote:

> 
> 
> On Fri, Feb 11, 2011 at 10:57 PM, Oleg A. Mamontov <oleg at mamontov.net> wrote:
> 
> We can delegate serve static files to Plack or anything other.
> No problem, but it would be good to document this recommendation in manual
> and remove this code from Dancer core.
> But if Dancer offers its own built-in functionality, it seems to me
> that we need to do it with maximum quality. Isn't it?
> Poorly made functionality worse than missing functionality because it creates ambiguity.
> 
> This strategy seems a bit too aggressive. I don't think the functionality is poorly made, it just
> is a basic one and lacks some more advanced feature. If you want the advanced features,
> you can hop to more sophisticated solutions. This is how I see it.
> 
> I can agree in documenting it, but keep in mind that Dancer is a young project!
> 
>  
> BTW, see source code for Catalyst::Plugin::Static::Simple in _serve_static method.
> There is no conditional get but there is:
>    $c->res->headers->last_modified( $stat->mtime );
> What do you think, why is it done?
> Without this header many browsers will re-request all your static files when
> you navigate on your site with mouse clicks (no force page refresh).
> Just one simple header (Last-Modified) and you skip a lot of requests!
> 
> So maybe you agree with Alexis here: I don't know anything about Catalyst and its architecture, but
> the functionality you're talking about is in a "::Plugin::" module, which seems to point in
> the same direction.

Key moment: Catalyst::Plugin::Static::Simple is core module of Catalyst distribution.

> 
>  
> I did not expect that such simple and obvious things can generate as much debates :(
> 
> I see the debate as a winning point, because it is productive and not destructive. So far, I see
> the potential for a Plack middleware, a change to the API to add some hook and a related plugin,
> and some more documentation to make it all clearer. Just try to consider this thing a bit less
> obvious ;-)
> 
> Cheers,
> 
>     Flavio.
> _______________________________________________
> Dancer-users mailing list
> Dancer-users at perldancer.org
> http://www.backup-manager.org/cgi-bin/listinfo/dancer-users


-- 
Cheers,
Oleg A. Mamontov

mailto:  oleg at mamontov.net

jabber:  lonerr at charla.mamontov.net
icq uin: 79-521-617
cell:    +7-903-798-1352