Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Opentracing extension: java.lang.reflect.InvocationTargetException: Illegal character in authority #4077

Closed
jamesfalkner opened this issue Sep 18, 2019 · 2 comments
Labels
area/config kind/bug Something isn't working
Milestone

Comments

@jamesfalkner
Copy link

Describe the bug
When building an app with the OpenTracing extension, and specifying an environment variable in the definition of an application property in application.properties:

quarkus.jaeger.endpoint=http://${JAEGER_ENDPOINT:jaeger-tracing}:14268/api/traces

The application fails to build (jvm or native) with

[ERROR] Failed to execute goal io.quarkus:quarkus-maven-plugin:0.22.0:build (default) on project getting-started: Failed to build a runnable JAR: Failed to build a runner jar: Failed to augment application classes: java.lang.reflect.InvocationTargetException: Illegal character in authority at index 7: http://${JAEGER_ENDPOINT:jaeger-tracing}:14268/api/traces -> [Help 1]

Expected behavior
App builds and runs with no errors and the property is settable via environment variable at runtime.

Actual behavior
Unable to build (See above). If I rename the property to anything else, e.g.

quarkus.foobar.endpoint=http://${JAEGER_ENDPOINT:jaeger-tracing}:14268/api/traces

it works. If I set it to a value not using an environment variable, it also works:

quarkus.jaeger.endpoint=foo

To Reproduce
Steps to reproduce the behavior:
1.

mvn io.quarkus:quarkus-maven-plugin:0.21.1:create \
    -DprojectGroupId=org.acme \
    -DprojectArtifactId=getting-started \
    -DclassName="org.acme.quickstart.GreetingResource" \
    -Dpath="/hello" -Dextensions="opentracing"

Add variable definition to src/main/resources/application.properties

  1. mvn clean package -DskipTests

Configuration

# Add your application.properties here, if applicable.
quarkus.jaeger.endpoint=http://${JAEGER_ENDPOINT:jaeger-tracing}:14268/api/traces

Environment (please complete the following information):

  • Output of uname -a or ver:
    Darwin jfalkner-OSX 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64

  • Output of java -version:

java version "1.8.0_201"
Java(TM) SE Runtime Environment (build 1.8.0_201-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.201-b09, mixed mode)
  • GraalVM version (if different from Java):

graalvm-ce-19.1.1

  • Quarkus version or git rev:

0.22.0

@jamesfalkner jamesfalkner added the kind/bug Something isn't working label Sep 18, 2019
@gsmet
Copy link
Member

gsmet commented Sep 18, 2019

There's a good change you're hitting this issue: #3516 and that the env variable is not expanded and Jaeger refuses either $ or { in the URL.

@dmlloyd what's the status of this one? It's an issue that often gets reported.

@dmlloyd
Copy link
Member

dmlloyd commented Sep 18, 2019

It's in progress, but keeps getting bumped down due to one-off issues... sorry. I'll try to get it done ASAP.

@gsmet gsmet added this to the 1.1.0 milestone Nov 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/config kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants