#308 If a colon is available it must be followed by a five digit number to represent a proper port definition.

Author: mmaeller

plugin/src/main/java/com/spotify/plugin/dockerfile/BuildMojo.java

@@ -53,7 +53,7 @@ public class BuildMojo extends AbstractDockerMojo {
   /**
    * Regex for a valid docker repository name.  Used in validateRepository().
    */
-  private static final String VALID_REPO_REGEX = "^([a-z0-9_.-]+[:]?)+(\\/[a-z0-9_.-]+)*$";
+  private static final String VALID_REPO_REGEX = "^([a-z0-9_.-])+(:[0-9]{1,5})?(\\/[a-z0-9_.-]+)*$";
 
   /**
    * Directory containing the the build context. This is typically the directory that contains

plugin/src/test/java/com/spotify/plugin/dockerfile/TestRepoNameValidation.java

@@ -61,5 +61,6 @@ public void testFailCases() {
     assertFalse("Starting slash didn't fail", BuildMojo.validateRepository("/atstart"));
     assertFalse("Ending slash didn't fail", BuildMojo.validateRepository("atend/"));
     assertFalse("Only port", BuildMojo.validateRepository(":443"));
+    assertFalse("Port exceeding range", BuildMojo.validateRepository("example.com:100000/myproject"));
   }
 }