Merge branch 'refs/heads/fix-caching-getOrComputeAllDestinations' into add-preLookup-Check

Author: Jonas-Isr

cloudplatform/connectivity-destination-service/src/main/java/com/sap/cloud/sdk/cloudplatform/connectivity/DestinationService.java

@@ -2,6 +2,7 @@
 
 import java.time.Duration;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.LinkedHashMap;
 import java.util.List;
@@ -907,11 +908,23 @@ private static Try<List<DestinationProperties>> getOrComputeAllDestinations(
                 return destinationDownloader.apply(options);
             }
 
+            if( isUsingExperimentalFeatures(options) ) {
+                log
+                    .warn(
+                        "Using cache/change detection together with either fragments, cross-level options, or custom headers is discouraged and might lead to unexpected behaviour.");
+            }
+
             return GetOrComputeAllDestinationsCommand
                 .prepareCommand(options, instanceAll(), isolationLocks(), destinationDownloader)
                 .execute();
         }
 
+        private static boolean isUsingExperimentalFeatures( @Nonnull final DestinationOptions options )
+        {
+            String[] featureNames = { "X-fragment-name", "crossLevelSetting", "customHeader" };
+            return options.getOptionKeys().stream().anyMatch(s -> Arrays.stream(featureNames).anyMatch(s::contains));
+        }
+
         private Cache()
         {
             throw new IllegalStateException("This static class must never be instantiated.");