set implicit https port to 443.

Bardo1 · Jul 20, 2017 · b520fd7 · b520fd7
1 parent ec74ad5
commit b520fd7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/ribbon-loadbalancer/src/main/java/com/netflix/loadbalancer/Server.java b/ribbon-loadbalancer/src/main/java/com/netflix/loadbalancer/Server.java
@@ -151,8 +151,10 @@ static Pair<String, Integer> getHostPort(String id) {
 
             if (id.toLowerCase().startsWith("http://")) {
                 id = id.substring(7);
+                port = 80;
             } else if (id.toLowerCase().startsWith("https://")) {
                 id = id.substring(8);
+                port = 443;
             }
 
             if (id.contains("/")) {

diff --git a/ribbon-loadbalancer/src/test/java/com/netflix/loadbalancer/ServerTest.java b/ribbon-loadbalancer/src/test/java/com/netflix/loadbalancer/ServerTest.java
@@ -28,7 +28,7 @@ public void createSecureSchemeHost() {
         Server server = new Server("https://netflix.com");
         assertEquals("https", server.getScheme());
         assertEquals("netflix.com", server.getHost());
-        assertEquals(80, server.getPort());
+        assertEquals(443, server.getPort());
     }
 
     @Test