[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-31 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14191803#comment-14191803
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user asoldano commented on the pull request:

https://github.com/apache/cxf/pull/25#issuecomment-61260523
  
@kylape just for the record... I fixed JBossWS to avoid calling 
setHandlerChain(..) at every invocation :-) see 
https://source.jboss.org/changelog/JBossWS?cs=19036


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape
Assignee: Daniel Kulp
 Fix For: 3.1.0, 3.0.3, 2.7.14


 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-31 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14191965#comment-14191965
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user kylape commented on the pull request:

https://github.com/apache/cxf/pull/25#issuecomment-61281462
  
Ah, nice.  Thanks, Alessio!


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape
Assignee: Daniel Kulp
 Fix For: 3.1.0, 3.0.3, 2.7.14


 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-30 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14190881#comment-14190881
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user kylape commented on the pull request:

https://github.com/apache/cxf/pull/25#issuecomment-61172618
  
I've verified the fix on CXF 2.7.12 (the version in JBoss EAP) already, but 
I'll work to verify on CXF master.


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape
Assignee: Daniel Kulp
 Fix For: 3.1.0, 3.0.3, 2.7.14


 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-30 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14191390#comment-14191390
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user kylape commented on the pull request:

https://github.com/apache/cxf/pull/25#issuecomment-61220541
  
Okay, I reproduced the issue with CXF 3.0.2 and verified it is resolved in 
my build of the master branch today.  

Just FYI -- in JBoss I only had to add this code:

```
final Service service = Service.create(wsdl, ns);
service.setHandlerResolver(new HandlerResolver() {
  public ListHandler getHandlerChain(PortInfo info) {
return handlerChain;
  }
});
```

because the JBossWS integration calls `setHandlerChain` for every 
invocation to ensure handler order (perhaps that could be improved upon as 
well?).  When using CXF without JBossWS integration, I had to call 
`((BindingProvider)port).getBinding().setHandlerChain(handlerChain)` for every 
invocation to reproduce the issue.


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape
Assignee: Daniel Kulp
 Fix For: 3.1.0, 3.0.3, 2.7.14


 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-30 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14191391#comment-14191391
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user kylape closed the pull request at:

https://github.com/apache/cxf/pull/25


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape
Assignee: Daniel Kulp
 Fix For: 3.1.0, 3.0.3, 2.7.14


 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-29 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14188984#comment-14188984
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user dkulp commented on the pull request:

https://github.com/apache/cxf/pull/25#issuecomment-61003841
  
This has been merged.   Can you please verify and close?   Thanks!


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape
Assignee: Daniel Kulp
 Fix For: 3.1.0, 3.0.3, 2.7.14


 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-21 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14178999#comment-14178999
 ] 

ASF GitHub Bot commented on CXF-6038:
-

Github user asfgit closed the pull request at:

https://github.com/apache/cxf/pull/26


 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape

 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-10 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14167558#comment-14167558
 ] 

ASF GitHub Bot commented on CXF-6038:
-

GitHub user kylape opened a pull request:

https://github.com/apache/cxf/pull/25

[CXF-6038] Only add the handler interceptors once



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kylape/cxf cxf-6038

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cxf/pull/25.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #25


commit a4d64d349832bcd413222fddd37d613eb7e78112
Author: Kyle Lape kyle.l...@redhat.com
Date:   2014-10-10T21:34:07Z

[CXF-6038] Only add the handler interceptors once




 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape

 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-10 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14167599#comment-14167599
 ] 

ASF GitHub Bot commented on CXF-6038:
-

GitHub user kylape opened a pull request:

https://github.com/apache/cxf/pull/26

[CXF-6038] Only add the handler interceptors once



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kylape/cxf cxf-6038--2.7-fixes

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cxf/pull/26.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #26


commit 69c001d075fa819622600897d057168e65a1d330
Author: Kyle Lape kyle.l...@redhat.com
Date:   2014-10-10T21:34:07Z

[CXF-6038] Only add the handler interceptors once




 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape

 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (CXF-6038) Repeatedly invoking setHandlerChain() can cause a build up of handler interceptors on the chain

2014-10-10 Thread Kyle Lape (JIRA)

[ 
https://issues.apache.org/jira/browse/CXF-6038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14167602#comment-14167602
 ] 

Kyle Lape commented on CXF-6038:


Submitted pull requests on github:

master: https://github.com/apache/cxf/pull/25
2.7-fixes: https://github.com/apache/cxf/pull/26

 Repeatedly invoking setHandlerChain() can cause a build up of handler 
 interceptors on the chain
 ---

 Key: CXF-6038
 URL: https://issues.apache.org/jira/browse/CXF-6038
 Project: CXF
  Issue Type: Bug
  Components: JAX-WS Runtime
Affects Versions: 3.0.1
Reporter: Kyle Lape

 If an interceptor chain is cached between invocations and a user calls 
 {{setHandlerChain()}} before every invocation, the handler chain interceptors 
 get added to the interceptor chain over and over indefinitely.  
 I'm guessing that {{JaxWsEndpointImpl.addHandlerInterceptors()}} would need 
 to check for the presence of the handler interceptors on the chain already.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)