Remove obsolete endpoint logic code (#3680)

Author: normj

generator/.DevConfigs/B0756C5F-C2BC-4DBF-830C-91AC3F1359DE.json

@@ -0,0 +1,10 @@
+{
+    "core": {
+      "changeLogMessages": [
+        "Remove obsolete endpoint logic code",
+		"Remove obsolete global ClockSkew property"
+      ],
+      "type": "patch",
+      "updateMinimum": true
+    }
+  }

generator/ServiceClientGeneratorLib/Generators/Endpoints/EndpointResolver.partial.cs

@@ -14,7 +14,7 @@ private string GetValueSource(Parameter parameter)
         {
             switch (parameter.builtIn)
             {
-                case "AWS::Region": return "config.RegionEndpoint?.SystemName";
+                case "AWS::Region": return "requestContext.Request.AlternateEndpoint?.SystemName ?? config.RegionEndpoint?.SystemName;";
                 case "AWS::UseFIPS": return "config.UseFIPSEndpoint";
                 case "AWS::UseDualStack": return "config.UseDualstackEndpoint";
                 case "SDK::Endpoint": return "config.ServiceURL";

generator/ServiceClientGeneratorLib/Generators/SourceFiles/ServiceConfig.cs

@@ -135,6 +135,9 @@ namespace <#=this.Config.Namespace#>
                 ClientConfig = this,
                 OriginalRequest = parameters.Request,
                 Request = new DefaultRequest(parameters.Request, ServiceId)
+                {
+                    AlternateEndpoint = parameters.AlternateEndpoint
+                }
             };
 
             var executionContext = new Amazon.Runtime.Internal.ExecutionContext(requestContext, null);

generator/ServiceClientGeneratorLib/Generators/SourceFiles/ServiceConfig.tt

@@ -123,33 +123,13 @@ public static TimeSpan? ManualClockCorrection
         /// by determining the correct server time and reissuing the
         /// request with the correct time.
         /// Default value of this field is True.
-        /// <seealso cref="ClockOffset"/> will be updated with the calculated
-        /// offset even if this field is set to false, though requests
-        /// will not be corrected or retried.
-        /// Ignored if <seealso cref="ManualClockCorrection"/> is set.
         /// </summary>
         public static bool CorrectForClockSkew
         {
             get { return _rootConfig.CorrectForClockSkew; }
             set { _rootConfig.CorrectForClockSkew = value; }
         }
 
-        /// <summary>
-        /// The calculated clock skew correction, if there is one.
-        /// This field will be set if a service call resulted in an exception
-        /// and the SDK has determined that there is a difference between local
-        /// and server times.
-        /// 
-        /// If <seealso cref="CorrectForClockSkew"/> is set to true, this
-        /// value will be set to the correction, but it will not be used by the
-        /// SDK and clock skew errors will not be retried.
-        /// </summary>
-        [Obsolete("This value is deprecated in favor of IClientConfig.ClockOffset, use CorrectClockSkew.GetClockCorrectionForEndpoint(string endpoint) instead.")]
-        public static TimeSpan ClockOffset
-        {
-            get;
-            internal set;
-        }
         #endregion
 
         #region Region

sdk/src/Core/AWSConfigs.cs

@@ -20,7 +20,6 @@
 
 
 using Amazon.Util;
-using Amazon.Internal;
 using Amazon.Runtime.Endpoints;
 using Amazon.Runtime.Internal;
 using Amazon.Runtime.Internal.Util;
@@ -417,18 +416,6 @@ public bool UseHttp
             set { this.useHttp = value; }
         }
 
-        internal static string GetUrl(IClientConfig config, RegionEndpoint regionEndpoint)
-        {
-#pragma warning disable CS0612,CS0618
-            var endpoint = 
-                regionEndpoint.GetEndpointForService(
-                    config.RegionEndpointServiceName, 
-                    config.ToGetEndpointForServiceOptions());
-#pragma warning restore CS0612,CS0618
-            string url = new Uri(string.Format(CultureInfo.InvariantCulture, "{0}{1}", config.UseHttp ? "http://" : "https://", endpoint.Hostname)).AbsoluteUri;
-            return url;
-        }
-
         /// <summary>
         /// Gets and sets the AuthenticationRegion property.
         /// Used in AWS4 request signing, this is an optional property; 
@@ -958,18 +945,6 @@ public virtual void Validate()
 #endif
         }
 
-        /// <summary>
-        /// Wrapper around <seealso cref="AWSConfigs.ManualClockCorrection"/>
-        /// </summary>
-        [Obsolete("Please use CorrectClockSkew.GetClockCorrectionForEndpoint(string endpoint) instead.", false)]
-        public TimeSpan ClockOffset
-        {
-            get
-            {
-                return AWSConfigs.ManualClockCorrection.GetValueOrDefault();
-            }
-        }
-
         /// <summary>
         /// Gets and sets the DisableHostPrefixInjection flag. If true, host prefix injection will be disabled for this client, the default value of this flag is false. 
         /// Host prefix injection prefixes the service endpoint with request members from APIs which use this feature. 

sdk/src/Core/Amazon.Runtime/ClientConfig.cs

@@ -127,9 +127,6 @@ internal static void SetClockCorrectionForEndpoint(string endpoint, TimeSpan cor
             try
             {
                 clockCorrectionDictionary[endpoint] = correction;
-#pragma warning disable CS0618 // Type or member is obsolete
-                AWSConfigs.ClockOffset = correction;
-#pragma warning restore CS0618 // Type or member is obsolete
             }
             finally
             {

sdk/src/Core/Amazon.Runtime/CorrectClockSkew.cs

@@ -51,28 +51,6 @@ protected static string GetContents(Uri uri, IWebProxy proxy, Dictionary<string,
             }
         }
 
-        [Obsolete("This method is not compatible with Native AOT builds. The GetObjectFromResponse overload using the generic parameter taking in a JsonSerializerContext should be used instead.")]
-        [RequiresUnreferencedCode("GetObjectFromResponse overload using the generic parameter taking in a JsonSerializerContext should be used instead.")]
-        protected static T GetObjectFromResponse<T>(Uri uri)
-        {
-            return GetObjectFromResponse<T>(uri, null, null);
-        }
-
-        [Obsolete("This method is not compatible with Native AOT builds. The GetObjectFromResponse overload using the generic parameter taking in a JsonSerializerContext should be used instead.")]
-        [RequiresUnreferencedCode("GetObjectFromResponse overload using the generic parameter taking in a JsonSerializerContext should be used instead.")]
-        protected static T GetObjectFromResponse<T>(Uri uri, IWebProxy proxy)
-        {
-            return GetObjectFromResponse<T>(uri, proxy, null);
-        }
-
-        [Obsolete("This method is not compatible with Native AOT builds. The GetObjectFromResponse overload using the generic parameter taking in a JsonSerializerContext should be used instead.")]
-        [RequiresUnreferencedCode("GetObjectFromResponse overload using the generic parameter taking in a JsonSerializerContext should be used instead.")]
-        protected static T GetObjectFromResponse<T>(Uri uri, IWebProxy proxy, Dictionary<string, string> headers)
-        {
-            string json = GetContents(uri, proxy, headers);
-            return JsonSerializer.Deserialize<T>(json);
-        }
-
         protected static T GetObjectFromResponse<T, TC>(Uri uri, IWebProxy proxy, Dictionary<string, string> headers)
             where TC :
 #if NET8_0_OR_GREATER

sdk/src/Core/Amazon.Runtime/Credentials/URIBasedRefreshingCredentialHelper.cs

@@ -14,6 +14,7 @@
  */
 using System;
 using System.Collections.Generic;
+using System.Text.RegularExpressions;
 
 namespace Amazon.Runtime.Endpoints
 {
@@ -27,9 +28,20 @@ public ServiceOperationEndpointParameters(AmazonWebServiceRequest request)
             Request = request;
         }
 
+        public ServiceOperationEndpointParameters(AmazonWebServiceRequest request, RegionEndpoint alternateEndpoint)
+            : this(request)
+        {
+            AlternateEndpoint = alternateEndpoint;
+        }
+
         /// <summary>
         /// Request for the desired service operation.
         /// </summary>
         public AmazonWebServiceRequest Request { get; }
+
+        /// <summary>
+        /// Alternate endpoint that overrides the endpoint configured for service client.
+        /// </summary>
+        public RegionEndpoint AlternateEndpoint { get; }
     }
 }

sdk/src/Core/Amazon.Runtime/Endpoints/ServiceOperationEndpointParameters.cs

@@ -319,11 +319,6 @@ public partial interface IClientConfig
         /// <exception cref="Amazon.Runtime.AmazonClientException">The timeout specified is null.</exception>
         void Validate();
 
-        /// <summary>
-        /// Returns the calculated clock skew value for this config's service endpoint. If AWSConfigs.CorrectForClockSkew is false,
-        /// this value won't be used to construct service requests.
-        /// </summary>
-        TimeSpan ClockOffset { get; }
         /// <summary>
         /// Gets the DisableHostPrefixInjection flag. If true, host prefix injection will be disabled for this client, the default value of this flag is false. 
         /// Host prefix injection prefixes the service endpoint with request members from APIs which use this feature.