Fix ENG-3650. Allow alphanumeric, underscore, dollar, period, and at …

…sign. DDLCompiler uses "([\\w.$]+)" to match. I allowed @ to allow users to use new system procedures without updating the client library.
VoltDB · Mar 9, 2013 · fa0f506 · fa0f506
1 parent 91bbcc6
commit fa0f506
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/VoltDB.Data.Client/Connections/ProcedureAccess.cs b/VoltDB.Data.Client/Connections/ProcedureAccess.cs
@@ -51,7 +51,7 @@ internal ProcedureAccess(VoltConnection executor)
         /// Regular expression used to validate tokens (procedure names only for now) (will make sure the name doesn't
         /// have any @ sign, or other invalid character for that matter).
         /// </summary>
-        private Regex TokenValidator = new Regex("^[a-zA-Z0-9_]*$");
+        private Regex TokenValidator = new Regex("^[a-zA-Z0-9_.$@]*$");
 
         /// <summary>
         /// Validate the procedure name (alpahnumeric only, no @)

diff --git a/VoltDB.Data.Client/Connections/ProcedureWrapper[T].tt b/VoltDB.Data.Client/Connections/ProcedureWrapper[T].tt
@@ -261,7 +261,7 @@ namespace VoltDB.Data.Client
         /// Regular expression used to validate tokens (procedure names only for now) (will make sure the name doesn't
         /// have any @ sign, or other invalid character for that matter).
         /// </summary>
-        private Regex TokenValidator = new Regex("^[a-zA-Z0-9_]*$");
+        private Regex TokenValidator = new Regex("^[a-zA-Z0-9_.$@]*$");
 
         /// <summary>
         /// Validate the procedure name (alpahnumeric only, no @)