summaryrefslogtreecommitdiffstats
path: root/config-model-api
diff options
context:
space:
mode:
authorØyvind Grønnesby <oyving@verizonmedia.com>2019-05-31 13:01:00 +0200
committerØyvind Grønnesby <oyving@verizonmedia.com>2019-05-31 13:01:00 +0200
commitd0b778a6301f52dc8859ef5938cf36baded302c1 (patch)
treec3e93a4f3c47ff94e5dda8822eb12776f4d8dff9 /config-model-api
parent6e8f2db53f9d0ef4c4fbbd92c064f3b873f803e9 (diff)
Throw an exception if a region is mentioned multiple times for an endpoint
Diffstat (limited to 'config-model-api')
-rw-r--r--config-model-api/src/main/java/com/yahoo/config/application/api/xml/DeploymentSpecXmlReader.java5
1 files changed, 4 insertions, 1 deletions
diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/xml/DeploymentSpecXmlReader.java b/config-model-api/src/main/java/com/yahoo/config/application/api/xml/DeploymentSpecXmlReader.java
index 1078317e0d2..58795f6ea9e 100644
--- a/config-model-api/src/main/java/com/yahoo/config/application/api/xml/DeploymentSpecXmlReader.java
+++ b/config-model-api/src/main/java/com/yahoo/config/application/api/xml/DeploymentSpecXmlReader.java
@@ -177,7 +177,10 @@ public class DeploymentSpecXmlReader {
if (region == null || region.isEmpty() || region.isBlank()) {
throw new IllegalArgumentException("Empty 'region' element in 'endpoint' tag.");
}
- regions.add(regionElement.getTextContent());
+ if (regions.contains(region)) {
+ throw new IllegalArgumentException("Duplicate 'region' element in 'endpoint' tag: " + region);
+ }
+ regions.add(region);
}
endpoints.add(new Endpoint(rotationId, containerId.get(), regions));