Re: svn commit: r1448003 - /cxf/trunk/rt/transports/http/pom.xml

2013-02-20 Thread Daniel Kulp
Freeman, Can I ask why this is necessary? The HTTP module does not reference any of these classes. Thus, it shouldn't be importing them. What are the symptoms/stack traces? This sounds like more of a problem someplace else. I'm -1 on this commit until we can understand why it's

Re: svn commit: r1448003 - /cxf/trunk/rt/transports/http/pom.xml

2013-02-20 Thread Freeman Fang
Hi Dan, In AbstractHTTPDestination, we have code like inMessage.put(SecurityContext.class, new SecurityContext() { public Principal getUserPrincipal() { return pp; } public boolean isUserInRole(String role) { return

Re: svn commit: r1448003 - /cxf/trunk/rt/transports/http/pom.xml

2013-02-20 Thread Daniel Kulp
This is a bug in the jetty JAAS bundles then, not CXF. Get a bug logged there. We should not be importing things we don't actually use. We should be importing the Principal and SecurityContext things and such, but not any specific implementations. This still needs to be reverted. Dan

Re: svn commit: r1448003 - /cxf/trunk/rt/transports/http/pom.xml

2013-02-20 Thread Daniel Kulp
OK. I think I see the real problem… In ServletController, we're setting the ContextClassloader in the case where a destination can be found based on the path, but we're apparently not setting it in the the other cases. This should be deferred down to the application level bundles via the

RE: [VOTE] - Release Apache Fediz 1.0.3

2013-02-20 Thread Oliver Wulff
+1 Oli From: Colm O hEigeartaigh [cohei...@apache.org] Sent: 19 February 2013 18:05 To: dev@cxf.apache.org Subject: [VOTE] - Release Apache Fediz 1.0.3 This is a vote to release Apache Fediz 1.0.3. This is a bug-fix release with an upgraded CXF 2.6.6

Re: [VOTE] - Release Apache Fediz 1.0.3

2013-02-20 Thread Daniel Kulp
+1 Dan On Feb 20, 2013, at 9:35 AM, Oliver Wulff owu...@talend.com wrote: +1 Oli From: Colm O hEigeartaigh [cohei...@apache.org] Sent: 19 February 2013 18:05 To: dev@cxf.apache.org Subject: [VOTE] - Release Apache Fediz 1.0.3 This is a

Re: [VOTE] - Release Apache Fediz 1.0.3

2013-02-20 Thread Sergey Beryozkin
+1 Sergey On 20/02/13 14:35, Oliver Wulff wrote: +1 Oli From: Colm O hEigeartaigh [cohei...@apache.org] Sent: 19 February 2013 18:05 To: dev@cxf.apache.org Subject: [VOTE] - Release Apache Fediz 1.0.3 This is a vote to release Apache Fediz 1.0.3.

Re: svn commit: r1448003 - /cxf/trunk/rt/transports/http/pom.xml

2013-02-20 Thread Freeman Fang
Thanks Dan, it works perfectly. - Freeman(Yue) Fang Red Hat, Inc. FuseSource is now part of Red Hat Web: http://fusesource.com | http://www.redhat.com/ Twitter: freemanfang Blog: http://freemanfang.blogspot.com http://blog.sina.com.cn/u/1473905042 weibo: @Freeman小屋 On 2013-2-20, at