[ 
https://issues.apache.org/jira/browse/CAMEL-12827?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624237#comment-16624237
 ] 

ASF GitHub Bot commented on CAMEL-12827:
----------------------------------------

bobpaulin opened a new pull request #2528: CAMEL-12827 - HttpSendDynamicAware 
setting port to -1 when not defined.
URL: https://github.com/apache/camel/pull/2528
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> camel-http-common HttpSendDynamicAware setting port to -1 when port is not 
> defined in uri
> -----------------------------------------------------------------------------------------
>
>                 Key: CAMEL-12827
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12827
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-http-common
>    Affects Versions: 2.22.0, 2.22.1, 2.23.0
>            Reporter: Bob Paulin
>            Priority: Major
>
> If the route qualifies for SendDyanamicAware processing within the camel-http 
> or camel-http4 component and the route does not specify a port it is getting 
> set to -1.  This should simply assume port 80 or 443 based on http vs https.  
> Logic exists to trim those ports if they are specified however if you omit 
> them your get the following exception:
>  
> {code:java}
> 2018-09-21T16:20:32,797 | ERROR | Camel (camel-2) thread #10 - 
> NettyEventExecutorGroup | AbstractErrorListProcessor       | 26 - 
> com.somebundle - 1.0.76 | Error Handled for route: 
> http://0.0.0.0:8880/%7BsiteId%7D/%7Bcountry%7D/somepath?httpMethodRestrict=GET
> org.apache.camel.ResolveEndpointFailedException: Failed to resolve endpoint: 
> https4://somedomain.com:-1?bridgeEndpoint=true due to: Expected 
> scheme-specific part at index 6: https:
>     at 
> org.apache.camel.impl.DefaultCamelContext.getEndpoint(DefaultCamelContext.java:758)
>  ~[91:org.apache.camel.camel-core:2.22.0]
>     at 
> org.apache.camel.util.CamelContextHelper.getMandatoryEndpoint(CamelContextHelper.java:80)
>  ~[91:org.apache.camel.camel-core:2.22.0]
>     at 
> org.apache.camel.util.ExchangeHelper.resolveEndpoint(ExchangeHelper.java:91) 
> ~[91:org.apache.camel.camel-core:2.22.0]
>     at 
> org.apache.camel.processor.SendDynamicProcessor.resolveEndpoint(SendDynamicProcessor.java:239)
>  ~[91:org.apache.camel.camel-core:2.22.0]
>     at 
> org.apache.camel.processor.SendDynamicProcessor.process(SendDynamicProcessor.java:132)
>  ~[91:org.apache.camel.camel-core:2.22.0]
>     at 
> org.apache.camel.processor.interceptor.TraceInterceptor.process(TraceInterceptor.java:181)
>  ~[91:org.apache.camel.camel-core:2.22.0]
>     at 
> org.apache.camel.processor.InterceptorToAsyncProcessorBridge.process(InterceptorToAsyncProcessorBridge.java:67)
>  ~[91:org.apache.camel.camel-core:2.22.0]{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to