From ce0cc6bd1f2e7af525d1ac243d3296b784ef72ef Mon Sep 17 00:00:00 2001 From: Jonas Israel Date: Thu, 18 Dec 2025 14:32:26 +0100 Subject: [PATCH 1/2] propagate destination options for Lookup check --- .../sdk/cloudplatform/connectivity/DestinationService.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java b/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java index 2d91b987c..fb0bb5a6d 100644 --- a/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java +++ b/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java @@ -120,7 +120,7 @@ static ResilienceConfiguration createResilienceConfiguration( Try tryGetDestination( @Nonnull final String destinationName, @Nonnull final DestinationOptions options ) { - if( Cache.preLookupCheckEnabled && !preLookupCheckSuccessful(destinationName) ) { + if( Cache.preLookupCheckEnabled && !preLookupCheckSuccessful(destinationName, options) ) { final String msg = "Destination %s was not found among the destinations of the current tenant."; return Try.failure(new DestinationNotFoundException(String.format(msg, destinationName))); } @@ -388,9 +388,10 @@ private static boolean hasCauseAssignableFrom( @Nonnull final Throwable t, @Nonn return ExceptionUtils.getThrowableList(t).stream().map(Throwable::getClass).anyMatch(cls::isAssignableFrom); } - private boolean preLookupCheckSuccessful( final String destinationName ) + private boolean preLookupCheckSuccessful( final String destinationName, final DestinationOptions options ) { - return getAllDestinationProperties() + return new ArrayList<>( + Cache.getOrComputeAllDestinations(options, this::getAllDestinationsByRetrievalStrategy).get()) .stream() .anyMatch(properties -> properties.get(DestinationProperty.NAME).contains(destinationName)); } From 13fda769939170ca75cdd644ef1d7bf584b43c8e Mon Sep 17 00:00:00 2001 From: Jonas Israel Date: Thu, 18 Dec 2025 16:56:54 +0100 Subject: [PATCH 2/2] codestyle --- .../sdk/cloudplatform/connectivity/DestinationService.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java b/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java index fb0bb5a6d..72f6b208c 100644 --- a/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java +++ b/cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java @@ -390,8 +390,8 @@ private static boolean hasCauseAssignableFrom( @Nonnull final Throwable t, @Nonn private boolean preLookupCheckSuccessful( final String destinationName, final DestinationOptions options ) { - return new ArrayList<>( - Cache.getOrComputeAllDestinations(options, this::getAllDestinationsByRetrievalStrategy).get()) + return new ArrayList<>( + Cache.getOrComputeAllDestinations(options, this::getAllDestinationsByRetrievalStrategy).get()) .stream() .anyMatch(properties -> properties.get(DestinationProperty.NAME).contains(destinationName)); }