Set region to nothing for Route53 endpoint when making service URL

Project.toml

@@ -1,6 +1,6 @@
 name = "AWSCore"
 uuid = "4f1ea46c-232b-54a6-9b17-cc2d0f3e6598"
-version = "0.6.7"
+version = "0.6.8"
 
 [deps]
 Base64 = "2a0f44e3-6c83-55bd-87e4-b1978d98bd5f"

src/AWSCore.jl

@@ -202,13 +202,20 @@ Service endpoint URL for `request`.
 """
 function service_url(aws, request)
     endpoint = get(request, :endpoint, request[:service])
-    region = "." * aws[:region]
-    if endpoint == "iam" || (endpoint == "sdb" && region == ".us-east-1") || region=="."
+    region = aws[:region]
+    regionless_endpoints = ("iam", "route53")
+
+    if endpoint in regionless_endpoints || (endpoint == "sdb" && region == "us-east-1")
         region = ""
     end
-    service_host = get(aws,:service_host,"amazonaws.com")
-    string("https://", endpoint, region, ".", service_host,
-      request[:resource])
+
+    service_host = get(aws, :service_host, "amazonaws.com")
+
+    if !isempty(region)
+        return string("https://", endpoint, ".", region, ".", service_host, request[:resource])
+    else
+        return string("https://", endpoint, ".", service_host, request[:resource])
+    end
 end
 
 

test/endpoints.jl

@@ -0,0 +1,12 @@
+using AWSCore.Services: route53
+
+@testset "Endpoints" begin
+    @testset "Route53" begin
+        try
+            route53("GET", "/2013-04-01/accountlimit/MAX_HEALTH_CHECKS_BY_OWNER")
+            @test true
+        catch
+            @test false
+        end
+    end
+end

test/runtests.jl

@@ -25,6 +25,7 @@ aws = aws_config()
     include("arn.jl")
     include("credentials.jl")
     include("exceptions.jl")
+    include("endpoints.jl")
     include("localhost.jl")
     include("signaturev4.jl")
     include("xml.jl")