Fixed missing comma in nuget files for Transfer and Identity Management. Added china endpoint calculation to endpoints files.

Author: JonathanHenson

aws-cpp-sdk-identity-management/nuget/aws-cpp-sdk-identity-management.autopkg

@@ -48,7 +48,7 @@ nuget {
 
     dependencies {
        packages: {
-            AWSSDKCPP-Core/1.0
+            AWSSDKCPP-Core/1.0,
             AWSSDKCPP-CognitoIdentity/1.0.20140630
        }
     }

aws-cpp-sdk-transfer/nuget/aws-cpp-sdk-transfer.autopkg

@@ -48,7 +48,7 @@ nuget {
 
     dependencies {
        packages: {
-            AWSSDKCPP-Core/1.0
+            AWSSDKCPP-Core/1.0,
             AWSSDKCPP-S3/1.0.20060301
        }
     }

code-generation/generator/src/main/resources/com/amazonaws/util/awsclientgenerator/velocity/cpp/EndpointEnumSource.vm

@@ -15,18 +15,19 @@ namespace ${metadata.namespace}
 {
 namespace ${metadata.classNamePrefix}Endpoint
 {
-
+  static const int CN_REGION_HASH = Aws::Utils::HashingUtils::HashString("cn-north-1");
+  
 #foreach($regionEndpointPair in $endpointMapping.entrySet())
   static const int ${regionEndpointPair.key.toUpperCase().replace("-", "_")}_HASH = Aws::Utils::HashingUtils::HashString("${regionEndpointPair.key}");
 #end
 
   Aws::String ForRegion(const Aws::String& regionName, bool useDualStack)
   {
+    auto hash = Aws::Utils::HashingUtils::HashString(regionName.c_str());
+    
 #if($endpointMapping.size() > 0)
     if(!useDualStack)
-    {
-      auto hash = Aws::Utils::HashingUtils::HashString(regionName.c_str());
-
+    {      
 #set($elseText = "")
 #foreach($regionEndpointPair in $endpointMapping.entrySet())
       ${elseText}if(hash == ${regionEndpointPair.key.toUpperCase().replace("-", "_")}_HASH)
@@ -46,6 +47,12 @@ namespace ${metadata.classNamePrefix}Endpoint
     }
 
     ss << regionName << ".amazonaws.com";
+    
+    if(hash == CN_REGION_HASH)
+    {
+      ss << ".cn"; 
+    }
+    
     return ss.str();
   }