Remove support for URLConnectionSender

[mhalbritter]

In #42589, we make ZipkinHttpClientSender the default sender. The conditions for that bean are:

@ConditionalOnClass(HttpClient.class)
@ConditionalOnMissingBean(BytesMessageSender.class)

that means on every JDK which has HttpClient in it (which should be most of them, unless you built a custom one with jlink), ZipkinHttpClientSender will be used.

Given that, I think we can remove URLConnectionSender, which needs the zipkin-sender-urlconnection dependency on the classpath.

[thecooldrop]

Hi @mhalbritter , this seems like completed issue. Is there some work outstanding? If so, what would it be? I may be interessted in pushing the issue over into the "complete" category.

[nosan]

Hi @thecooldrop,

This issue is still open.

Is there some work outstanding? If so, what would it be?

The SenderConfiguration in ZipkinConfigurations still imports UrlConnectionSenderConfiguration. The goal of this task is to remove this import and completely delete UrlConnectionSenderConfiguration. Additionally, update ZipkinConfigurationsSenderConfigurationTests and remove the io.zipkin.reporter2:zipkin-sender-urlconnection dependency from all locations.

[thecooldrop]

Hi @thecooldrop,

This issue is still open.

Is there some work outstanding? If so, what would it be?

The SenderConfiguration in ZipkinConfigurations still imports UrlConnectionSenderConfiguration. The goal of this task is to remove this import and completely delete UrlConnectionSenderConfiguration. Additionally, update ZipkinConfigurationsSenderConfigurationTests and remove the io.zipkin.reporter2:zipkin-sender-urlconnection dependency from all locations.

Like this #44500 ? What I did is suspiciously simple to me.

[mhalbritter]

Superseded by #44500.