Jack Berg

Results 759 comments of Jack Berg

I see. I think it probably makes sense to add support for setting an Authenticator specifically for use with the proxy. But really we ought to revisit Authenticator, which has...

`Authenticator` was removed in #6984, but I think it makes sense to add proxy overloads that allow additional headers to be provided. Something like: ``` OtlpHttpSpanExporter.builder() .setProxy( ProxyOptions.create(InetSocketAddress.createUnresolved("host", "port")), ()...

> I've taken a look at the declarative config being proposed. There's actually nothing in here related to declarative config. The getters in `HttpSenderConfig`, `GrpcSenderConfig` reflect the standard configuration options...

Ok so it seems like something close to this will work for quarkus. There are still some small tweaks ([example](https://github.com/open-telemetry/opentelemetry-java/pull/7782#discussion_r2546486262)), but its mostly correct. The next step will be to...

> From a quick investigation, looks like https://github.com/open-telemetry/opentelemetry-java/pull/6944 removes the incubator dependency from the classpath, but I see it's still being referenced in places like [ExtendedSdkSpanBuilder](https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk/trace/src/main/java/io/opentelemetry/sdk/trace/ExtendedSdkSpanBuilder.java#L107) This is a strange...

Closing since I haven't heard any strong strong push back against my comment

Hey @codeboten - I put together a branch with a little demo that shows how the code generation might work: https://github.com/jack-berg/opentelemetry-configuration/tree/generate-compliance-matrix It wrote a fast / dirty [script](https://github.com/jack-berg/opentelemetry-configuration/blob/generate-compliance-matrix/scripts/generate-markdown.js) to: 1....

Oops looks like I had some copy paste errors on the links. Fixed. This still needs a decent amount of work before its ready for PR, but I think its...

@marcalff I'll work on refining some ideas sometime in the next 7-10 days.