Skip to content

Commit 290e888

Browse files
committed
Polish "Handle StepRegistry connect/read timeout deprecation"
Closes spring-projectsgh-14886
1 parent fe63342 commit 290e888

9 files changed

+103
-50
lines changed

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/appoptics/AppOpticsMetricsExportAutoConfiguration.java

+12-6
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,26 @@
5252
@EnableConfigurationProperties(AppOpticsProperties.class)
5353
public class AppOpticsMetricsExportAutoConfiguration {
5454

55+
private final AppOpticsProperties properties;
56+
57+
public AppOpticsMetricsExportAutoConfiguration(AppOpticsProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public AppOpticsConfig appOpticsConfig(AppOpticsProperties appOpticsProperties) {
58-
return new AppOpticsPropertiesConfigAdapter(appOpticsProperties);
63+
public AppOpticsConfig appOpticsConfig() {
64+
return new AppOpticsPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
6369
public AppOpticsMeterRegistry appOpticsMeterRegistry(AppOpticsConfig config,
64-
Clock clock, AppOpticsProperties appOpticsProperties) {
70+
Clock clock) {
6571
return AppOpticsMeterRegistry.builder(config).clock(clock)
66-
.httpClient(new HttpUrlConnectionSender(
67-
appOpticsProperties.getConnectTimeout(),
68-
appOpticsProperties.getReadTimeout()))
72+
.httpClient(
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076
}
7177

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/datadog/DatadogMetricsExportAutoConfiguration.java

+11-5
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,26 @@
5252
@EnableConfigurationProperties(DatadogProperties.class)
5353
public class DatadogMetricsExportAutoConfiguration {
5454

55+
private final DatadogProperties properties;
56+
57+
public DatadogMetricsExportAutoConfiguration(DatadogProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public DatadogConfig datadogConfig(DatadogProperties datadogProperties) {
58-
return new DatadogPropertiesConfigAdapter(datadogProperties);
63+
public DatadogConfig datadogConfig() {
64+
return new DatadogPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
6369
public DatadogMeterRegistry datadogMeterRegistry(DatadogConfig datadogConfig,
64-
Clock clock, DatadogProperties datadogProperties) {
70+
Clock clock) {
6571
return DatadogMeterRegistry.builder(datadogConfig).clock(clock)
6672
.httpClient(
67-
new HttpUrlConnectionSender(datadogProperties.getConnectTimeout(),
68-
datadogProperties.getReadTimeout()))
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076
}
7177

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/dynatrace/DynatraceMetricsExportAutoConfiguration.java

+12-6
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,26 @@
5252
@EnableConfigurationProperties(DynatraceProperties.class)
5353
public class DynatraceMetricsExportAutoConfiguration {
5454

55+
private final DynatraceProperties properties;
56+
57+
public DynatraceMetricsExportAutoConfiguration(DynatraceProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public DynatraceConfig dynatraceConfig(DynatraceProperties dynatraceProperties) {
58-
return new DynatracePropertiesConfigAdapter(dynatraceProperties);
63+
public DynatraceConfig dynatraceConfig() {
64+
return new DynatracePropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
6369
public DynatraceMeterRegistry dynatraceMeterRegistry(DynatraceConfig dynatraceConfig,
64-
Clock clock, DynatraceProperties dynatraceProperties) {
70+
Clock clock) {
6571
return DynatraceMeterRegistry.builder(dynatraceConfig).clock(clock)
66-
.httpClient(new HttpUrlConnectionSender(
67-
dynatraceProperties.getConnectTimeout(),
68-
dynatraceProperties.getReadTimeout()))
72+
.httpClient(
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076
}
7177

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/elastic/ElasticMetricsExportAutoConfiguration.java

+11-5
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,26 @@
5252
@EnableConfigurationProperties(ElasticProperties.class)
5353
public class ElasticMetricsExportAutoConfiguration {
5454

55+
private final ElasticProperties properties;
56+
57+
public ElasticMetricsExportAutoConfiguration(ElasticProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public ElasticConfig elasticConfig(ElasticProperties elasticProperties) {
58-
return new ElasticPropertiesConfigAdapter(elasticProperties);
63+
public ElasticConfig elasticConfig() {
64+
return new ElasticPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
6369
public ElasticMeterRegistry elasticMeterRegistry(ElasticConfig elasticConfig,
64-
Clock clock, ElasticProperties elasticProperties) {
70+
Clock clock) {
6571
return ElasticMeterRegistry.builder(elasticConfig).clock(clock)
6672
.httpClient(
67-
new HttpUrlConnectionSender(elasticProperties.getConnectTimeout(),
68-
elasticProperties.getReadTimeout()))
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076
}
7177

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/humio/HumioMetricsExportAutoConfiguration.java

+11-6
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,25 @@
5252
@EnableConfigurationProperties(HumioProperties.class)
5353
public class HumioMetricsExportAutoConfiguration {
5454

55+
private final HumioProperties properties;
56+
57+
public HumioMetricsExportAutoConfiguration(HumioProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public HumioConfig humioConfig(HumioProperties humioProperties) {
58-
return new HumioPropertiesConfigAdapter(humioProperties);
63+
public HumioConfig humioConfig() {
64+
return new HumioPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
63-
public HumioMeterRegistry humioMeterRegistry(HumioConfig humioConfig, Clock clock,
64-
HumioProperties humioProperties) {
69+
public HumioMeterRegistry humioMeterRegistry(HumioConfig humioConfig, Clock clock) {
6570
return HumioMeterRegistry.builder(humioConfig).clock(clock)
6671
.httpClient(
67-
new HttpUrlConnectionSender(humioProperties.getConnectTimeout(),
68-
humioProperties.getReadTimeout()))
72+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
73+
this.properties.getReadTimeout()))
6974
.build();
7075

7176
}

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/influx/InfluxMetricsExportAutoConfiguration.java

+10-4
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,16 @@
5252
@EnableConfigurationProperties(InfluxProperties.class)
5353
public class InfluxMetricsExportAutoConfiguration {
5454

55+
private final InfluxProperties properties;
56+
57+
public InfluxMetricsExportAutoConfiguration(InfluxProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public InfluxConfig influxConfig(InfluxProperties influxProperties) {
58-
return new InfluxPropertiesConfigAdapter(influxProperties);
63+
public InfluxConfig influxConfig() {
64+
return new InfluxPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
@@ -64,8 +70,8 @@ public InfluxMeterRegistry influxMeterRegistry(InfluxConfig influxConfig, Clock
6470
InfluxProperties influxProperties) {
6571
return InfluxMeterRegistry.builder(influxConfig).clock(clock)
6672
.httpClient(
67-
new HttpUrlConnectionSender(influxProperties.getConnectTimeout(),
68-
influxProperties.getReadTimeout()))
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076

7177
}

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/kairos/KairosMetricsExportAutoConfiguration.java

+12-6
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,26 @@
5252
@EnableConfigurationProperties(KairosProperties.class)
5353
public class KairosMetricsExportAutoConfiguration {
5454

55+
private final KairosProperties properties;
56+
57+
public KairosMetricsExportAutoConfiguration(KairosProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public KairosConfig kairosConfig(KairosProperties kairosProperties) {
58-
return new KairosPropertiesConfigAdapter(kairosProperties);
63+
public KairosConfig kairosConfig() {
64+
return new KairosPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
63-
public KairosMeterRegistry kairosMeterRegistry(KairosConfig kairosConfig, Clock clock,
64-
KairosProperties kairosProperties) {
69+
public KairosMeterRegistry kairosMeterRegistry(KairosConfig kairosConfig,
70+
Clock clock) {
6571
return KairosMeterRegistry.builder(kairosConfig).clock(clock)
6672
.httpClient(
67-
new HttpUrlConnectionSender(kairosProperties.getConnectTimeout(),
68-
kairosProperties.getReadTimeout()))
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076

7177
}

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/newrelic/NewRelicMetricsExportAutoConfiguration.java

+12-6
Original file line numberDiff line numberDiff line change
@@ -53,20 +53,26 @@
5353
@EnableConfigurationProperties(NewRelicProperties.class)
5454
public class NewRelicMetricsExportAutoConfiguration {
5555

56+
private final NewRelicProperties properties;
57+
58+
public NewRelicMetricsExportAutoConfiguration(NewRelicProperties properties) {
59+
this.properties = properties;
60+
}
61+
5662
@Bean
5763
@ConditionalOnMissingBean
58-
public NewRelicConfig newRelicConfig(NewRelicProperties props) {
59-
return new NewRelicPropertiesConfigAdapter(props);
64+
public NewRelicConfig newRelicConfig() {
65+
return new NewRelicPropertiesConfigAdapter(this.properties);
6066
}
6167

6268
@Bean
6369
@ConditionalOnMissingBean
6470
public NewRelicMeterRegistry newRelicMeterRegistry(NewRelicConfig newRelicConfig,
65-
Clock clock, NewRelicProperties newRelicProperties) {
71+
Clock clock) {
6672
return NewRelicMeterRegistry.builder(newRelicConfig).clock(clock)
67-
.httpClient(new HttpUrlConnectionSender(
68-
newRelicProperties.getConnectTimeout(),
69-
newRelicProperties.getReadTimeout()))
73+
.httpClient(
74+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
75+
this.properties.getReadTimeout()))
7076
.build();
7177

7278
}

spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/metrics/export/wavefront/WavefrontMetricsExportAutoConfiguration.java

+12-6
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,26 @@
5252
@EnableConfigurationProperties(WavefrontProperties.class)
5353
public class WavefrontMetricsExportAutoConfiguration {
5454

55+
private final WavefrontProperties properties;
56+
57+
public WavefrontMetricsExportAutoConfiguration(WavefrontProperties properties) {
58+
this.properties = properties;
59+
}
60+
5561
@Bean
5662
@ConditionalOnMissingBean
57-
public WavefrontConfig wavefrontConfig(WavefrontProperties props) {
58-
return new WavefrontPropertiesConfigAdapter(props);
63+
public WavefrontConfig wavefrontConfig() {
64+
return new WavefrontPropertiesConfigAdapter(this.properties);
5965
}
6066

6167
@Bean
6268
@ConditionalOnMissingBean
6369
public WavefrontMeterRegistry wavefrontMeterRegistry(WavefrontConfig wavefrontConfig,
64-
Clock clock, WavefrontProperties wavefrontProperties) {
70+
Clock clock) {
6571
return WavefrontMeterRegistry.builder(wavefrontConfig).clock(clock)
66-
.httpClient(new HttpUrlConnectionSender(
67-
wavefrontProperties.getConnectTimeout(),
68-
wavefrontProperties.getReadTimeout()))
72+
.httpClient(
73+
new HttpUrlConnectionSender(this.properties.getConnectTimeout(),
74+
this.properties.getReadTimeout()))
6975
.build();
7076
}
7177

0 commit comments

Comments
 (0)