attempt to fix recursion hell as micronaut http client setup tries to

build the OtlpMetriceRequest that used the TS DB OauthRetriever that
itself causes the http client to try and do it's thing

Author: atimgraves

IoTDBJDBC/src/main/java/com/oracle/demo/timg/iot/iotdbjdbc/messagehandler/outputs/http/normalizeddata/timeseriesdb/otlp/OtlpMetricsRequestFilter.java

@@ -9,6 +9,7 @@
 import com.oracle.demo.timg.iot.iotdbjdbc.messagehandler.outputs.http.normalizeddata.timeseriesdb.oauth.TimeSeriesDBOAuthTokenRequestFilter;
 import com.oracle.demo.timg.iot.iotdbjdbc.messagehandler.outputs.http.normalizeddata.timeseriesdb.oauth.TimeSeriesDBOAuthTokenRetriever;
 
+import io.micronaut.context.BeanProvider;
 import io.micronaut.context.annotation.Requires;
 import io.micronaut.context.event.StartupEvent;
 import io.micronaut.http.MutableHttpRequest;
@@ -20,24 +21,38 @@
 
 //@ClientFilter(patterns = { "${" + TimeSeriesDBProperties.TIME_SERIES_PROPERTY_METRICS_PATH + ":/tel/v1/metrics}",
 //		"${" + TimeSeriesDBProperties.TIME_SERIES_PROPERTY_METRICS_PATH + ":/tel/v1/metrics}/**" })
-@ClientFilter(patterns = "/tel/v1/metrics")
+@ClientFilter(patterns = "/tel/v1/metrics", serviceId = OtlpProperties.METRICS_CLIENT_ID)
 @Requires(property = TimeSeriesDBProperties.TIME_SERIES_PROPERTY_ENABLED, value = "true", defaultValue = "false")
+@Requires(property = "micronaut.http.services.timeseriesmetrics.url")
+@Requires(property = "micronaut.http.services.timeseriesoauth.url")
 @Log
 public class OtlpMetricsRequestFilter {
-	private final TimeSeriesDBOAuthTokenRetriever tokenRetriever;
+	// private final TimeSeriesDBOAuthTokenRetriever tokenRetriever;
+	// need to do this using a provider as otherwise we hit a massive recursion that
+	// goes this then the retriever that itself uses a http client and so
+	// ends up in the http client framework that then tries to load this again on
+	// setting this up that ultimately results in stack overflow
+	// using the bean provider delays the retriever setup until this is constructed,
+	// and hopefully also the retriever and it's client
+	private final BeanProvider<TimeSeriesDBOAuthTokenRetriever> tokenRetriever;
 
 	@Inject
-	public OtlpMetricsRequestFilter(TimeSeriesDBOAuthTokenRetriever tokenRetriever) {
-		log.info("OtlpMetricsRequestFilter constructor");
+	public OtlpMetricsRequestFilter(BeanProvider<TimeSeriesDBOAuthTokenRetriever> tokenRetriever) {
 		this.tokenRetriever = tokenRetriever;
 	}
 
+//	@Inject
+//	public OtlpMetricsRequestFilter(TimeSeriesDBOAuthTokenRetriever tokenRetriever) {
+//		log.info("OtlpMetricsRequestFilter constructor");
+//		this.tokenRetriever = tokenRetriever;
+//	}
+
 	@RequestFilter
 	public void doFilter(MutableHttpRequest<?> request) {
 		log.info("Running metrics OTLP filtering");
 		try {
-			String token = tokenRetriever.getToken();
-			String tokenType = tokenRetriever.getTokenType();
+			String token = tokenRetriever.get().getToken();
+			String tokenType = tokenRetriever.get().getTokenType();
 			log.info("Got the token and tokenType");
 			request.getHeaders().add(AUTHORIZATION, (tokenType == null ? "Bearer" : tokenType) + " " + token);
 			request.getHeaders().add(TimeSeriesDBOAuthTokenRequestFilter.HEADER_REQUEST_ID,