Luciano, Can you please open a JIRA and post your patch? :) +1 to making things simpler to extend in principle.
-- dims On Thu, Oct 14, 2010 at 12:40 AM, Luciano Resende <[email protected]> wrote: > Currently, embedding Wink requires extending multiple Wink Server > classes, particularly if you want to add special handlers etc. It > seems that some of this could be simplified, particularly in the case > of adding user specific handlers. I was wondering if, instead of > having to provide my own DeploymentConfiguration that needs to > override the specific handler initialization method, we could expose > the actual list of handlers in the Wink DeploymentConfiguration and > allow embedders to add new handlers to this list in the > getDeploymentConfiguration in the Servlet ? This might also require > some updates on how the handler chain is processed, basically delaying > the initialization of the handler chain or updating the > AbstractHandlersChain to process userHandlers and Handlers as two > separate lists. > > Thoughts ? In case people are ok with this approach, I have a patch in > hand that I can provide. > > -- > Luciano Resende > http://people.apache.org/~lresende > http://twitter.com/lresende1975 > http://lresende.blogspot.com/ > -- Davanum Srinivas :: http://davanum.wordpress.com
