The two constants have different values?
On Tue, Aug 27, 2013 at 8:02 AM, Sameera Jayasoma <[email protected]> wrote: > This patch has broken the RESTful service invocations deployed in tenants > in AS. e.g. > > https://wso2.org/jira/browse/CARBON-14390 > > curl -v -k -X GET > http://localhost:9763/services/t/jayasoma.org/echo/echoInt?in=12 > curl -v -k -X POST -d "in=12" > http://localhost:9763/services/t/jayasoma.org/echo/echoInt > > Hence I am reverting this patch. Specially the following change. > > - String httpMethod = (String) mainInMsgContext.getProperty( > HTTPConstants.HTTP_METHOD); > + String httpMethod = (String) mainInMsgContext.getProperty( > Constants.Configuration.HTTP_METHOD); > > Thanks, > Sameera. > > > > > On Thu, Jul 4, 2013 at 11:45 AM, Amila Suriarachchi <[email protected]>wrote: > >> I have applied the patch in wrong way. Will fix that. >> >> thanks, >> Amila. >> >> >> On Thu, Jul 4, 2013 at 7:55 AM, Amila Suriarachchi <[email protected]>wrote: >> >>> >>> >>> >>> On Wed, Jul 3, 2013 at 8:33 AM, Afkham Azeez <[email protected]> wrote: >>> >>>> >>>> >>>> >>>> On Wed, Jul 3, 2013 at 6:43 PM, <[email protected]> wrote: >>>> >>>>> Author: [email protected] >>>>> Date: Wed Jul 3 18:43:39 2013 >>>>> New Revision: 177123 >>>>> URL: http://wso2.org/svn/browse/wso2?view=rev&revision=177123 >>>>> >>>>> Log: >>>>> applied the patch of CARBON-14087 >>>>> >>>>> Modified: >>>>> >>>>> >>>>> carbon/kernel/trunk/core/org.wso2.carbon.core/src/main/java/org/wso2/carbon/core/multitenancy/MultitenantMessageReceiver.java >>>>> >>>>> Modified: >>>>> carbon/kernel/trunk/core/org.wso2.carbon.core/src/main/java/org/wso2/carbon/core/multitenancy/MultitenantMessageReceiver.java >>>>> URL: >>>>> http://wso2.org/svn/browse/wso2/carbon/kernel/trunk/core/org.wso2.carbon.core/src/main/java/org/wso2/carbon/core/multitenancy/MultitenantMessageReceiver.java?rev=177123&r1=177122&r2=177123&view=diff >>>>> >>>>> ============================================================================== >>>>> --- >>>>> carbon/kernel/trunk/core/org.wso2.carbon.core/src/main/java/org/wso2/carbon/core/multitenancy/MultitenantMessageReceiver.java >>>>> (original) >>>>> +++ >>>>> carbon/kernel/trunk/core/org.wso2.carbon.core/src/main/java/org/wso2/carbon/core/multitenancy/MultitenantMessageReceiver.java >>>>> Wed Jul 3 18:43:39 2013 >>>>> @@ -339,7 +339,7 @@ >>>>> MultitenantRESTServlet restServlet = new >>>>> MultitenantRESTServlet( >>>>> tenantConfigCtx, requestUri, tenant); >>>>> >>>>> - String httpMethod = (String) >>>>> mainInMsgContext.getProperty(HTTPConstants.HTTP_METHOD); >>>>> + String httpMethod = (String) >>>>> mainInMsgContext.getProperty(Constants.Configuration.HTTP_METHOD); >>>>> try { >>>>> if >>>>> (httpMethod.equals(Constants.Configuration.HTTP_METHOD_GET)) { >>>>> restServlet.doGet(request, response); >>>>> @@ -347,7 +347,8 @@ >>>>> restServlet.doPost(request, response); >>>>> } else if >>>>> (httpMethod.equals(Constants.Configuration.HTTP_METHOD_PUT)) { >>>>> restServlet.doPut(request, response); >>>>> - } else if >>>>> (httpMethod.equals(Constants.Configuration.HTTP_METHOD_DELETE)) { >>>>> + } else if >>>>> (httpMethod.equals(Constants.Configuration.HTTP_METHOD_DELETE) || >>>>> + httpMethod.equals("OPTIONS")) { >>>>> restServlet.doDelete(request, response); >>>>> >>>> >>>> Why do we delete if OPTIONS is received? >>>> >>> >>> Sorry I did not look at the content but applied the patch Nuwan has send >>> in this thread "Patching the 4.1.0 Kernel". Will talk to him and do the >>> correct thing. >>> >>> thanks, >>> Amila. >>> >>>> >>>> >>>>> } else { >>>>> // TODO: throw exception: Invalid verb >>>>> _______________________________________________ >>>>> Commits mailing list >>>>> [email protected] >>>>> http://wso2.org/cgi-bin/mailman/listinfo/commits >>>>> >>>> >>>> >>>> >>>> -- >>>> *Afkham Azeez* >>>> Director of Architecture; WSO2, Inc.; http://wso2.com >>>> Member; Apache Software Foundation; http://www.apache.org/ >>>> * <http://www.apache.org/>** >>>> email: **[email protected]* <[email protected]>* 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* >>>> >>> >>> >>> >>> -- >>> *Amila Suriarachchi* >>> >>> Software Architect >>> WSO2 Inc. ; http://wso2.com >>> lean . enterprise . middleware >>> >>> phone : +94 71 3082805 >>> >> >> >> >> -- >> *Amila Suriarachchi* >> >> Software Architect >> WSO2 Inc. ; http://wso2.com >> lean . enterprise . middleware >> >> phone : +94 71 3082805 >> >> _______________________________________________ >> Dev mailing list >> [email protected] >> http://wso2.org/cgi-bin/mailman/listinfo/dev >> >> > > > -- > Sameera Jayasoma, > Architect, > > WSO2, Inc. (http://wso2.com) > email: [email protected] > blog: http://sameera.adahas.org > twitter: https://twitter.com/sameerajayasoma > flickr: http://www.flickr.com/photos/sameera-jayasoma/collections > > > 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: **[email protected]* <[email protected]>* 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*
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
