Commit 290e8886 authored by Andy Wilkinson's avatar Andy Wilkinson

Polish "Handle StepRegistry connect/read timeout deprecation"

Closes gh-14886
parent fe63342f
...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(AppOpticsProperties.class) @EnableConfigurationProperties(AppOpticsProperties.class)
public class AppOpticsMetricsExportAutoConfiguration { public class AppOpticsMetricsExportAutoConfiguration {
private final AppOpticsProperties properties;
public AppOpticsMetricsExportAutoConfiguration(AppOpticsProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public AppOpticsConfig appOpticsConfig(AppOpticsProperties appOpticsProperties) { public AppOpticsConfig appOpticsConfig() {
return new AppOpticsPropertiesConfigAdapter(appOpticsProperties); return new AppOpticsPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public AppOpticsMeterRegistry appOpticsMeterRegistry(AppOpticsConfig config, public AppOpticsMeterRegistry appOpticsMeterRegistry(AppOpticsConfig config,
Clock clock, AppOpticsProperties appOpticsProperties) { Clock clock) {
return AppOpticsMeterRegistry.builder(config).clock(clock) return AppOpticsMeterRegistry.builder(config).clock(clock)
.httpClient(new HttpUrlConnectionSender( .httpClient(
appOpticsProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
appOpticsProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(DatadogProperties.class) @EnableConfigurationProperties(DatadogProperties.class)
public class DatadogMetricsExportAutoConfiguration { public class DatadogMetricsExportAutoConfiguration {
private final DatadogProperties properties;
public DatadogMetricsExportAutoConfiguration(DatadogProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public DatadogConfig datadogConfig(DatadogProperties datadogProperties) { public DatadogConfig datadogConfig() {
return new DatadogPropertiesConfigAdapter(datadogProperties); return new DatadogPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public DatadogMeterRegistry datadogMeterRegistry(DatadogConfig datadogConfig, public DatadogMeterRegistry datadogMeterRegistry(DatadogConfig datadogConfig,
Clock clock, DatadogProperties datadogProperties) { Clock clock) {
return DatadogMeterRegistry.builder(datadogConfig).clock(clock) return DatadogMeterRegistry.builder(datadogConfig).clock(clock)
.httpClient( .httpClient(
new HttpUrlConnectionSender(datadogProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
datadogProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(DynatraceProperties.class) @EnableConfigurationProperties(DynatraceProperties.class)
public class DynatraceMetricsExportAutoConfiguration { public class DynatraceMetricsExportAutoConfiguration {
private final DynatraceProperties properties;
public DynatraceMetricsExportAutoConfiguration(DynatraceProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public DynatraceConfig dynatraceConfig(DynatraceProperties dynatraceProperties) { public DynatraceConfig dynatraceConfig() {
return new DynatracePropertiesConfigAdapter(dynatraceProperties); return new DynatracePropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public DynatraceMeterRegistry dynatraceMeterRegistry(DynatraceConfig dynatraceConfig, public DynatraceMeterRegistry dynatraceMeterRegistry(DynatraceConfig dynatraceConfig,
Clock clock, DynatraceProperties dynatraceProperties) { Clock clock) {
return DynatraceMeterRegistry.builder(dynatraceConfig).clock(clock) return DynatraceMeterRegistry.builder(dynatraceConfig).clock(clock)
.httpClient(new HttpUrlConnectionSender( .httpClient(
dynatraceProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
dynatraceProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(ElasticProperties.class) @EnableConfigurationProperties(ElasticProperties.class)
public class ElasticMetricsExportAutoConfiguration { public class ElasticMetricsExportAutoConfiguration {
private final ElasticProperties properties;
public ElasticMetricsExportAutoConfiguration(ElasticProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public ElasticConfig elasticConfig(ElasticProperties elasticProperties) { public ElasticConfig elasticConfig() {
return new ElasticPropertiesConfigAdapter(elasticProperties); return new ElasticPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public ElasticMeterRegistry elasticMeterRegistry(ElasticConfig elasticConfig, public ElasticMeterRegistry elasticMeterRegistry(ElasticConfig elasticConfig,
Clock clock, ElasticProperties elasticProperties) { Clock clock) {
return ElasticMeterRegistry.builder(elasticConfig).clock(clock) return ElasticMeterRegistry.builder(elasticConfig).clock(clock)
.httpClient( .httpClient(
new HttpUrlConnectionSender(elasticProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
elasticProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,20 +52,25 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,25 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(HumioProperties.class) @EnableConfigurationProperties(HumioProperties.class)
public class HumioMetricsExportAutoConfiguration { public class HumioMetricsExportAutoConfiguration {
private final HumioProperties properties;
public HumioMetricsExportAutoConfiguration(HumioProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public HumioConfig humioConfig(HumioProperties humioProperties) { public HumioConfig humioConfig() {
return new HumioPropertiesConfigAdapter(humioProperties); return new HumioPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public HumioMeterRegistry humioMeterRegistry(HumioConfig humioConfig, Clock clock, public HumioMeterRegistry humioMeterRegistry(HumioConfig humioConfig, Clock clock) {
HumioProperties humioProperties) {
return HumioMeterRegistry.builder(humioConfig).clock(clock) return HumioMeterRegistry.builder(humioConfig).clock(clock)
.httpClient( .httpClient(
new HttpUrlConnectionSender(humioProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
humioProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,10 +52,16 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,10 +52,16 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(InfluxProperties.class) @EnableConfigurationProperties(InfluxProperties.class)
public class InfluxMetricsExportAutoConfiguration { public class InfluxMetricsExportAutoConfiguration {
private final InfluxProperties properties;
public InfluxMetricsExportAutoConfiguration(InfluxProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public InfluxConfig influxConfig(InfluxProperties influxProperties) { public InfluxConfig influxConfig() {
return new InfluxPropertiesConfigAdapter(influxProperties); return new InfluxPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
...@@ -64,8 +70,8 @@ public class InfluxMetricsExportAutoConfiguration { ...@@ -64,8 +70,8 @@ public class InfluxMetricsExportAutoConfiguration {
InfluxProperties influxProperties) { InfluxProperties influxProperties) {
return InfluxMeterRegistry.builder(influxConfig).clock(clock) return InfluxMeterRegistry.builder(influxConfig).clock(clock)
.httpClient( .httpClient(
new HttpUrlConnectionSender(influxProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
influxProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(KairosProperties.class) @EnableConfigurationProperties(KairosProperties.class)
public class KairosMetricsExportAutoConfiguration { public class KairosMetricsExportAutoConfiguration {
private final KairosProperties properties;
public KairosMetricsExportAutoConfiguration(KairosProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public KairosConfig kairosConfig(KairosProperties kairosProperties) { public KairosConfig kairosConfig() {
return new KairosPropertiesConfigAdapter(kairosProperties); return new KairosPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public KairosMeterRegistry kairosMeterRegistry(KairosConfig kairosConfig, Clock clock, public KairosMeterRegistry kairosMeterRegistry(KairosConfig kairosConfig,
KairosProperties kairosProperties) { Clock clock) {
return KairosMeterRegistry.builder(kairosConfig).clock(clock) return KairosMeterRegistry.builder(kairosConfig).clock(clock)
.httpClient( .httpClient(
new HttpUrlConnectionSender(kairosProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
kairosProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -53,20 +53,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -53,20 +53,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(NewRelicProperties.class) @EnableConfigurationProperties(NewRelicProperties.class)
public class NewRelicMetricsExportAutoConfiguration { public class NewRelicMetricsExportAutoConfiguration {
private final NewRelicProperties properties;
public NewRelicMetricsExportAutoConfiguration(NewRelicProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public NewRelicConfig newRelicConfig(NewRelicProperties props) { public NewRelicConfig newRelicConfig() {
return new NewRelicPropertiesConfigAdapter(props); return new NewRelicPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public NewRelicMeterRegistry newRelicMeterRegistry(NewRelicConfig newRelicConfig, public NewRelicMeterRegistry newRelicMeterRegistry(NewRelicConfig newRelicConfig,
Clock clock, NewRelicProperties newRelicProperties) { Clock clock) {
return NewRelicMeterRegistry.builder(newRelicConfig).clock(clock) return NewRelicMeterRegistry.builder(newRelicConfig).clock(clock)
.httpClient(new HttpUrlConnectionSender( .httpClient(
newRelicProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
newRelicProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration; ...@@ -52,20 +52,26 @@ import org.springframework.context.annotation.Configuration;
@EnableConfigurationProperties(WavefrontProperties.class) @EnableConfigurationProperties(WavefrontProperties.class)
public class WavefrontMetricsExportAutoConfiguration { public class WavefrontMetricsExportAutoConfiguration {
private final WavefrontProperties properties;
public WavefrontMetricsExportAutoConfiguration(WavefrontProperties properties) {
this.properties = properties;
}
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public WavefrontConfig wavefrontConfig(WavefrontProperties props) { public WavefrontConfig wavefrontConfig() {
return new WavefrontPropertiesConfigAdapter(props); return new WavefrontPropertiesConfigAdapter(this.properties);
} }
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public WavefrontMeterRegistry wavefrontMeterRegistry(WavefrontConfig wavefrontConfig, public WavefrontMeterRegistry wavefrontMeterRegistry(WavefrontConfig wavefrontConfig,
Clock clock, WavefrontProperties wavefrontProperties) { Clock clock) {
return WavefrontMeterRegistry.builder(wavefrontConfig).clock(clock) return WavefrontMeterRegistry.builder(wavefrontConfig).clock(clock)
.httpClient(new HttpUrlConnectionSender( .httpClient(
wavefrontProperties.getConnectTimeout(), new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
wavefrontProperties.getReadTimeout())) this.properties.getReadTimeout()))
.build(); .build();
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment