You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Claus Ibsen (JIRA)" <ji...@apache.org> on 2017/12/21 10:34:02 UTC

[jira] [Updated] (CAMEL-12098) URISyntaxException in OpenTracingTracer for endpoints with path parameters

     [ https://issues.apache.org/jira/browse/CAMEL-12098?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen updated CAMEL-12098:
--------------------------------
    Fix Version/s: 2.21.0
                   2.20.2

> URISyntaxException in OpenTracingTracer for endpoints with path parameters
> --------------------------------------------------------------------------
>
>                 Key: CAMEL-12098
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12098
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-opentracing, camel-restlet
>    Affects Versions: 2.20.1
>            Reporter: Anton Koscejev
>            Assignee: Claus Ibsen
>             Fix For: 2.20.2, 2.21.0
>
>
> We're using a lot of endpoints with path parameters, such as: {{restlet://https://localhost:58530//people/{personId\}?restletMethod=GET}} 
> The endpoint URI contains curly brackets, which is invalid and doesn't pass validation in {{URI.create}}. Unfortunately, that is exactly what happens in OpenTracingTracer:
> {code}
> SpanDecorator sd = decorators.get(URI.create(endpoint.getEndpointUri()).getScheme());
> {code}
> Naturally, this results in OpenTracing not working and lots of warnings like this: {code}
> WARN  o.a.c.opentracing.OpenTracingTracer - OpenTracing: Failed to capture tracing data
> java.lang.IllegalArgumentException: Illegal character in path at index 42: restlet://https://localhost:58530//people/{personId}?restletMethod=GET
> 	at java.net.URI.create(URI.java:852)
> 	at org.apache.camel.opentracing.OpenTracingTracer.getSpanDecorator(OpenTracingTracer.java:192)
> 	at org.apache.camel.opentracing.OpenTracingTracer$OpenTracingEventNotifier.notify(OpenTracingTracer.java:218)
> 	at org.apache.camel.util.EventHelper.doNotifyEvent(EventHelper.java:1096)
> 	at org.apache.camel.util.EventHelper.notifyExchangeSending(EventHelper.java:833)
> 	at org.apache.camel.processor.SendProcessor.process(SendProcessor.java:137)
> 	at org.apache.camel.processor.DelegateAsyncProcessor.process(DelegateAsyncProcessor.java:110)
> 	at org.apache.camel.processor.RedeliveryErrorHandler.process(RedeliveryErrorHandler.java:548)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:138)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:101)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.component.direct.DirectProducer.process(DirectProducer.java:62)
> 	at org.apache.camel.processor.SendProcessor.process(SendProcessor.java:148)
> 	at org.apache.camel.processor.DelegateAsyncProcessor.process(DelegateAsyncProcessor.java:110)
> 	at org.apache.camel.processor.RedeliveryErrorHandler.process(RedeliveryErrorHandler.java:548)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:138)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:101)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.component.direct.DirectProducer.process(DirectProducer.java:62)
> 	at org.apache.camel.processor.SendProcessor.process(SendProcessor.java:148)
> 	at org.apache.camel.processor.DelegateAsyncProcessor.process(DelegateAsyncProcessor.java:110)
> 	at org.apache.camel.processor.RedeliveryErrorHandler.process(RedeliveryErrorHandler.java:548)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:138)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:101)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.component.direct.DirectProducer.process(DirectProducer.java:62)
> 	at org.apache.camel.processor.SendProcessor.process(SendProcessor.java:148)
> 	at org.apache.camel.processor.DelegateAsyncProcessor.process(DelegateAsyncProcessor.java:110)
> 	at org.apache.camel.processor.RedeliveryErrorHandler.process(RedeliveryErrorHandler.java:548)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:138)
> 	at org.apache.camel.processor.Pipeline.process(Pipeline.java:101)
> 	at org.apache.camel.processor.CamelInternalProcessor.process(CamelInternalProcessor.java:201)
> 	at org.apache.camel.component.timer.TimerConsumer.sendTimerExchange(TimerConsumer.java:197)
> 	at org.apache.camel.component.timer.TimerConsumer$2.run(TimerConsumer.java:110)
> 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
> 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
> 	at java.lang.Thread.run(Thread.java:748)
> Caused by: java.net.URISyntaxException: Illegal character in path at index 42: restlet://https://localhost:58530//people/{personId}?restletMethod=GET
> 	at java.net.URI$Parser.fail(URI.java:2848)
> 	at java.net.URI$Parser.checkChars(URI.java:3021)
> 	at java.net.URI$Parser.parseHierarchical(URI.java:3105)
> 	at java.net.URI$Parser.parse(URI.java:3053)
> 	at java.net.URI.<init>(URI.java:588)
> 	at java.net.URI.create(URI.java:850)
> 	... 40 common frames omitted
> {code}
> Previously Restlet endpoint URI used normal brackets instead of curly brackets, so this wasn't an issue.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)