On Feb 11, 2011, at 5:10 PM, Alexis Sukrieh wrote:
Le 11/02/2011 15:07, franck a écrit :
>> I create small patch to enhance static files serving functionality. >> During development process its important to refresh pages quickly. >> Conditional HTTP requests is the good choise. >> Seems like working, but needs more testing :) > > I'm not sure we should do that in Dancer's core. This can be very easily achieved with a Plack Middleware. >
This is clearly a Plack solution. If you really need, you can maybe create a plugin for this, as soon as we add a new hook.
I would indeed not be opposed to a plugin, but I fear we open a pandora box: reimplementing any Plack middleware as Dancer plugin (which is clearly something we don't want to do).
Anyways, I don't think this should go in the core of Dancer. Dancer should remain as lightweight as possible, it's code should be very specific to what it does: dispatching request to the appropriate route handler.
I hope you understand my point of view ;)
I understand you clearly. But few lines of code (with no extra dependencies) which make developers' lives easier couldn't break lightweight of framework. It's my opinion only, sorry. For future i will have to patch new versions of Dancer myself :)
Regards,
-- Alexis Sukrieh
-- Cheers, Oleg A. Mamontov mailto: oleg@mamontov.net jabber: lonerr@charla.mamontov.net icq uin: 79-521-617 cell: +7-903-798-1352