On Thu, Jul 11, 2013 at 2:10 PM, Dinusha Senanayaka <dinu...@wso2.com>wrote:
> > > On Thu, Jul 11, 2013 at 1:50 PM, Afkham Azeez <az...@wso2.com> wrote: > >> I took a look at the code, and the only problem I see is the order of >> registering valves. If the ordering is not a problem, then we can easily >> implement it. I will look into implementing this. >> > > Thanks Azeez.. yes, the problem that we had with carbon tomcat valve was, > only the requests are going through that valve but not the responses. And > we need to handle response as well through the valve. > Um are you sure that's how it works ?! I was under the impression that once you call; getNext().invoke(request, response); The next wave will be hit, and the response will also go through the same valve chain, check [1]. [1] - https://svn.wso2.org/repos/wso2/carbon/platform/trunk/components/data-agents/org.wso2.carbon.bam.webapp.stat.publisher/src/main/java/org/wso2/carbon/bam/webapp/stat/publisher/WebAppStatisticPublisherValve.java > > Regards, > Dinusha. > >> >> Azeez >> >> >> On Thu, Jul 11, 2013 at 1:37 PM, Afkham Azeez <az...@wso2.com> wrote: >> >>> Sorry, maintaining your own cartalina.xml is not acceptable. I spoke to >>> NuwanD and mentioned how the current carbon Tomcat valve has to be changed >>> to support the chaining model. you will need to implement a Carbon >>> tomcat valve, not a Catalina valve >>> >>> >>> On Thu, Jul 11, 2013 at 1:16 PM, Dinusha Senanayaka <dinu...@wso2.com>wrote: >>> >>>> Hi All, >>>> >>>> When embedding api-management functionalities into GReg, we had to >>>> wrote an tomcat valve as an interceptor for handle requests. For that >>>> we implemented a direct tomcat valve which extended from catalina. >>>> valves.ValveBase instead of CarbonTomcatValve. But currently we don't >>>> have a way to register this tomcat valve programmatically, we have to >>>> manually register this using catalina-server.xml. >>>> >>>> For Greg , we have to maintain a it's own catalina-server.xml file at >>>> the product level by including apimgt-valve into it and override the >>>> catalina-server.xml coming from the platform.. Instead of doing like >>>> that, is it possible to support the registering of tomcat valves >>>> programmatically in future ? >>>> >>>> Regards, >>>> Dinusha. >>>> >>>> >>>> -- >>>> Dinusha Dilrukshi >>>> Senior Software Engineer >>>> WSO2 Inc.: http://wso2.com/ >>>> Mobile: +94725255071 >>>> Blog: http://dinushasblog.blogspot.com/ >>>> >>> >>> >>> >>> -- >>> *Afkham Azeez* >>> Director of Architecture; WSO2, Inc.; http://wso2.com >>> Member; Apache Software Foundation; http://www.apache.org/ >>> * <http://www.apache.org/>** >>> email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 >>> blog: **http://blog.afkham.org* <http://blog.afkham.org>* >>> twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> >>> * >>> linked-in: **http://lk.linkedin.com/in/afkhamazeez* >>> * >>> * >>> *Lean . Enterprise . Middleware* >>> >> >> >> >> -- >> *Afkham Azeez* >> Director of Architecture; WSO2, Inc.; http://wso2.com >> Member; Apache Software Foundation; http://www.apache.org/ >> * <http://www.apache.org/>** >> email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 >> blog: **http://blog.afkham.org* <http://blog.afkham.org>* >> twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> >> * >> linked-in: **http://lk.linkedin.com/in/afkhamazeez* >> * >> * >> *Lean . Enterprise . Middleware* >> > > > > -- > Dinusha Dilrukshi > Senior Software Engineer > WSO2 Inc.: http://wso2.com/ > Mobile: +94725255071 > Blog: http://dinushasblog.blogspot.com/ > > _______________________________________________ > Dev mailing list > Dev@wso2.org > http://wso2.org/cgi-bin/mailman/listinfo/dev > > -- Thanks, Shariq. Phone: +94 777 202 225
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev