Require scheme and host as well

1 files changed, 4 insertions, 1 deletions

diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java
index 685eed22699..816d647dfbb 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java
@@ -51,7 +51,10 @@ public class ApplicationVersion implements Comparable<ApplicationVersion> {
         if (commit.isPresent() && commit.get().length() > 128)
             throw new IllegalArgumentException("Commit may not be longer than 128 characters");
 
-        sourceUrl.map(URI::create);
+        sourceUrl.map(URI::create).ifPresent(url -> {
+            if (url.getHost() == null || url.getScheme() == null)
+                throw new IllegalArgumentException("Source URL must include scheme and host");
+        });
 
         if (authorEmail.isPresent() && ! authorEmail.get().matches("[^@]+@[^@]+"))
             throw new IllegalArgumentException("Invalid author email '" + authorEmail.get() + "'.");