From 9cab6670b9cd1ed01e896cc28468d0fe787ab9dd Mon Sep 17 00:00:00 2001
From: Alexander Zaft <a.zaft@fz-juelich.de>
Date: Fri, 17 Mar 2023 10:51:44 +0100
Subject: [PATCH] check configured names for spaces

reject equipment_ids and module names which contain spaces

Change-Id: I0ffe8fee13122ddfff6715863d0a82f72b5cb9f6
Reviewed-on: https://forge.frm2.tum.de/review/c/secop/frappy/+/30711
Tested-by: Jenkins Automated Tests <pedersen+jenkins@frm2.tum.de>
Reviewed-by: Alexander Zaft <a.zaft@fz-juelich.de>
---
 frappy/config.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/frappy/config.py b/frappy/config.py
index e16f56e..90a67ad 100644
--- a/frappy/config.py
+++ b/frappy/config.py
@@ -20,6 +20,7 @@
 #
 # *****************************************************************************
 import os
+import re
 
 from frappy.errors import ConfigError
 from frappy.lib import generalConfig
@@ -65,6 +66,12 @@ class Mod(dict):
             cls=cls,
             description=description
         )
+
+        # matches name from spec
+        if not re.match(r'^[a-zA-Z]\w{0,62}$', name, re.ASCII):
+            raise ConfigError('Not a valid SECoP Module name: "%s". '
+                              'Does it only contain letters, numbers and underscores?'
+                              % (name))
         # Make parameters out of all keywords
         groups = {}
         for key, val in kwds.items():