diff options
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 |
commit | d0b778a6301f52dc8859ef5938cf36baded302c1 (patch) | |
tree | c3e93a4f3c47ff94e5dda8822eb12776f4d8dff9 | |
parent | 6e8f2db53f9d0ef4c4fbbd92c064f3b873f803e9 (diff) |
Throw an exception if a region is mentioned multiple times for an endpoint
-rw-r--r-- | config-model-api/src/main/java/com/yahoo/config/application/api/xml/DeploymentSpecXmlReader.java | 5 |
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)); |