Merge pull request #343 from AmpersandTarski/feature/338-limit-allowe…

…d-chars-in-username-or-do-not-use-username-in-code Feature/338 limit allowed chars in username or do not use username in code
AmpersandTarski · Dec 21, 2023 · 4d7657a · 4d7657a
2 parents 6be6166 + 5960b42
commit 4d7657a
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/RAP4/customizations/bootstrap/files/ExecEngineFunctions.php b/RAP4/customizations/bootstrap/files/ExecEngineFunctions.php
@@ -320,7 +320,10 @@
          * - run kubectl apply -f "student-manifest-{{student}}.yaml"
         */
 
+        $pattern = '/[\W+]/';
+
         $studentName=strtolower($userName);
+        $studentName = preg_replace($pattern, '-', $studentName);
 
         $namespace=getenv('RAP_KUBERNETES_NAMESPACE');
         $containerImage=getenv('RAP_STUDENT_PROTO_IMAGE');