coheigea commented on code in PR #3552:
URL: https://github.com/apache/cxf/pull/3552#discussion_r4216173848


##########
core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java:
##########


Review Comment:
   @reta Yes good catch thanks!



##########
core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java:
##########


Review Comment:
   I tried it, but it breaks WS-RM: Proxy.RMClient passes an endpoint whose 
WrappedService.getEndpoints() is an immutable singletonMap, so the put throws 
UnsupportedOperationException (RM10PolicyWsdlTest, RM12PolicyWsdlTest, 
RMPolicyTest). Since callers of the Endpoint constructors create the endpoint 
and own its Service, I've left registration to them and kept the fix to the 
constructor that creates the endpoint itself.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to