From e5cbf948a1d4acb73c697cbb85184f1bb15635d7 Mon Sep 17 00:00:00 2001
From: Franco Fichtner <franco@opnsense.org>
Date: Wed, 20 Sep 2023 20:22:58 +0200
Subject: [PATCH] ipsec: require pool name, otherwise empty is allowed
 (multiple times)

The required property gives clarity to the unique constraint
which allowe this only once before but now a number of times.

Also update the mask to make it more obvious.
---
 src/opnsense/mvc/app/models/OPNsense/IPsec/Swanctl.xml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/opnsense/mvc/app/models/OPNsense/IPsec/Swanctl.xml b/src/opnsense/mvc/app/models/OPNsense/IPsec/Swanctl.xml
index 6c22a580b..1328bde0c 100644
--- a/src/opnsense/mvc/app/models/OPNsense/IPsec/Swanctl.xml
+++ b/src/opnsense/mvc/app/models/OPNsense/IPsec/Swanctl.xml
@@ -314,7 +314,8 @@
                     <Required>Y</Required>
                 </enabled>
                 <name type="TextField">
-                    <mask>/^([0-9a-zA-Z\.,_\-:]){0,1024}$/u</mask>
+                    <Required>Y</Required>
+                    <mask>/^([0-9a-zA-Z\.,_\-:]){1,1024}$/u</mask>
                     <Constraints>
                         <check001>
                             <ValidationMessage>Pool name must be unique.</ValidationMessage>