[CAMEL-10771] CxfEndpoint shows WARNING altough endpoint-/port name is configured

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.17.0, 2.18.1
• Fix Version/s: 2.17.6, 2.18.2, 2.19.0
• Component/s: camel-cxf
• Estimated Complexity: Unknown
• Assignee: Andrea Cosentino
• Reporter: Jochen Riedlinger
• Created: 31/Jan/17 10:27
• Updated: 31/Jan/17 15:31
• Resolved: 31/Jan/17 12:36

Description

Although I have configured the endpoint-/port name I get this WARNING in my logs:

2017-01-31 09:27:10.662 | WARN | org.apache.camel.component.cxf.CxfEndpoint | http-0.0.0.0:8443-1 | The endpoint/port name of cxf://https://someURL?cxfEndpointConfigurer=%23myWSCxfEndpointConfigurer&dataFormat=PAYLOAD&defaultOperationName=defaultOperation&portName=%7Bhttp%3A%2F%2Fexample.com%2Fx%2y%2FGz%7DMyBinding_soap12&wrapped=true&wrappedStyle=true&wsdlURL=wsdl-reference%2Fmywsdl.wsdl is empty, cxf will try to load the first one in wsdl for you. | org.apache.camel.component.cxf.CxfEndpoint.checkName(CxfEndpoint.java:646)

The cause for this is that the "checkName" method checks the instance variable "portName" and not "portNameString" (see attached screenshot).
Therefore I think it's better if the lines

checkName(portName, "endpoint/port name");
checkName(serviceName, "service name");

get replaced by

checkName(getPortName(), "endpoint/port name");
checkName(getServiceName(), "service name");

I'll send a Pull Request soon....

Issue Links

• links to: GitHub Pull Request #1433