Avoid upper case characters into VO names

iam-voms-aa/src/main/java/it/infn/mw/voms/properties/VomsProperties.java

@@ -19,6 +19,7 @@
 
 import javax.validation.Valid;
 import javax.validation.constraints.NotBlank;
+import javax.validation.constraints.Pattern;
 import javax.validation.constraints.Positive;
 
 import org.springframework.boot.context.properties.ConfigurationProperties;
@@ -103,6 +104,7 @@ public void setRefreshIntervalSec(long refreshIntervalSec) {
   public static class VOMSAAProperties {
 
     @NotBlank
+    @Pattern(regexp = "^[a-z][a-z0-9\\-]*(\\.[a-z][a-z0-9\\-]*)*$")
     private String voName;
 
     @NotBlank

iam-voms-aa/src/test/java/it/infn/mw/voms/properties/VomsPropertiesValidatorTests.java

@@ -0,0 +1,73 @@
+/**
+ * Copyright (c) Istituto Nazionale di Fisica Nucleare (INFN). 2016-2021
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package it.infn.mw.voms.properties;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import java.util.List;
+import java.util.Set;
+
+import javax.validation.ConstraintViolation;
+import javax.validation.Validation;
+import javax.validation.Validator;
+import javax.validation.ValidatorFactory;
+
+import org.junit.AfterClass;
+import org.junit.BeforeClass;
+import org.junit.Test;
+
+import it.infn.mw.voms.properties.VomsProperties.VOMSAAProperties;
+
+public class VomsPropertiesValidatorTests {
+  private static ValidatorFactory validatorFactory;
+  private static Validator validator;
+  private static final List<String> invalidExampleVONames = List.of("VO-name.test", "vo_name.test", "1vo-name.test",
+      "_vo-name.test", "vo_name.test", "vo_name.test1");
+  private static final String validVOName = "vo1-name.test";
+  private static final String validVOHost = "vo-host";
+
+  @BeforeClass
+  public static void createValidator() {
+    validatorFactory = Validation.buildDefaultValidatorFactory();
+    validator = validatorFactory.getValidator();
+  }
+
+  @AfterClass
+  public static void close() {
+    validatorFactory.close();
+  }
+
+  @Test
+  public void invalidVoName() {
+    VOMSAAProperties vomsaa = new VOMSAAProperties();
+    vomsaa.setHost(validVOHost);
+    for (String voName : invalidExampleVONames) {
+      vomsaa.setVoName(voName);
+      Set<ConstraintViolation<VOMSAAProperties>> violations = validator.validate(vomsaa);
+      assertFalse(violations.isEmpty());
+    }
+  }
+
+  @Test
+  public void validVoName() {
+    VOMSAAProperties vomsaa = new VOMSAAProperties();
+    vomsaa.setHost(validVOHost);
+    vomsaa.setVoName(validVOName);
+    Set<ConstraintViolation<VOMSAAProperties>> violations = validator.validate(vomsaa);
+    assertTrue(violations.isEmpty());
+  }
+}