fix: use character class regex without lookahead for Python/Rust compatibility

Author: jaya6400

openmetadata-spec/src/main/resources/json/schema/entity/data/pipeline.json

@@ -109,7 +109,7 @@
           "description": "Name that identifies this task instance uniquely.",
           "type": "string",
           "minLength": 1,
-          "pattern": "^[^><\":|\\x00-\\x1f]*$"
+          "pattern": "^[^><\"|\\x00-\\x1f]*$"
         },
         "displayName": {
           "description": "Display Name that identifies this Task. It could be title or label from the pipeline services.",

openmetadata-spec/src/main/resources/json/schema/entity/data/searchIndex.json

@@ -97,7 +97,7 @@
       "type": "string",
       "minLength": 1,
       "maxLength": 256,
-      "pattern": "^[^><\":|\\x00-\\x1f]*$"
+      "pattern": "^[^><\"|\\x00-\\x1f]*$"
     },
     "searchIndexField": {
       "type": "object",

openmetadata-spec/src/main/resources/json/schema/entity/data/table.json

@@ -209,7 +209,7 @@
           "type": "array",
           "items": {
             "type": "string",
-            "pattern": "^[^><\":|\\x00-\\x1f]*$"
+            "pattern": "^[^><\"|\\x00-\\x1f]*$"
           }
         },
         "referredColumns": {
@@ -236,7 +236,7 @@
       "description": "Local name (not fully qualified name) of the column. ColumnName is `-` when the column is not named in struct dataType. For example, BigQuery supports struct with unnamed fields.",
       "type": "string",
       "minLength": 1,
-      "pattern": "^[^><\":|\\x00-\\x1f]*$"
+      "pattern": "^[^><\"|\\x00-\\x1f]*$"
     },
     "partitionIntervalTypes": {
       "javaType": "org.openmetadata.schema.type.PartitionIntervalTypes",
@@ -275,7 +275,7 @@
         "columnName": {
           "description": "List of column names corresponding to the partition.",
           "type": "string",
-          "pattern": "^[^><\":|\\x00-\\x1f]*$"
+          "pattern": "^[^><\"|\\x00-\\x1f]*$"
         },
         "intervalType": {
           "$ref": "#/definitions/partitionIntervalTypes"

openmetadata-spec/src/main/resources/json/schema/type/basic.json

@@ -124,13 +124,13 @@
       "type": "string",
       "minLength": 1,
       "maxLength": 256,
-      "pattern": "^[^><\":|\\x00-\\x1f]*$"
+      "pattern": "^[^><\"|\\x00-\\x1f]*$"
     },
     "testCaseEntityName": {
       "description": "Name that identifies a test definition and test case.",
       "type": "string",
       "minLength": 1,
-      "pattern": "^[^><\":|\\x00-\\x1f]*$"
+      "pattern": "^[^><\"|\\x00-\\x1f]*$"
     },
     "fullyQualifiedEntityName": {
       "description": "A unique name that identifies an entity. Example for table 'DatabaseService.Database.Schema.Table'.",

openmetadata-spec/src/main/resources/json/schema/type/schema.json

@@ -67,7 +67,7 @@
       "type": "string",
       "minLength": 1,
       "maxLength": 128,
-      "pattern": "^[^><\":|\\x00-\\x1f]*$"
+      "pattern": "^[^><\"|\\x00-\\x1f]*$"
     },
     "field": {
       "type": "object",

openmetadata-ui/src/main/resources/ui/src/constants/regex.constants.test.ts

@@ -160,13 +160,11 @@ describe('Test Regex', () => {
   });
 
   it('EntityName regex should fail for the invalid entity name', () => {
-    // conatines :: in the name should fail
-    expect(ENTITY_NAME_REGEX.test('Hello::World')).toEqual(false);
-    // single colon also blocked as FQN separator
-    expect(ENTITY_NAME_REGEX.test('name:bad')).toEqual(false);
+    // contains reserved FQN characters - should fail
     expect(ENTITY_NAME_REGEX.test('name>bad')).toEqual(false);
     expect(ENTITY_NAME_REGEX.test('name<bad')).toEqual(false);
     expect(ENTITY_NAME_REGEX.test('name"bad')).toEqual(false);
+    expect(ENTITY_NAME_REGEX.test('name|bad')).toEqual(false);
     expect(ENTITY_NAME_REGEX.test('name\nbad')).toEqual(false);
     expect(ENTITY_NAME_REGEX.test('name\rbad')).toEqual(false);
     expect(ENTITY_NAME_REGEX.test('name\x00bad')).toEqual(false);

openmetadata-ui/src/main/resources/ui/src/constants/regex.constants.ts

@@ -21,7 +21,7 @@ export const EMAIL_REG_EX = /^\S+@\S+\.\S+$/;
  * and ASCII control characters. Supports Unicode characters.
  */
 // eslint-disable-next-line no-control-regex
-export const ENTITY_NAME_REGEX = /^[^><"|:\u0000-\u001f]*$/;
+export const ENTITY_NAME_REGEX = /^[^><"|\u0000-\u001f]*$/;
 
 /**
  * Matches any string that does NOT contain the following:
@@ -53,16 +53,27 @@ export const TIMESTAMP_UNIX_IN_MILLISECONDS_REGEX = /^\d{13}$/;
 
 export const ALL_ASTERISKS_REGEX = /^\*+$/;
 
+// Split the input into pairs using `;` and handle quoted strings properly
 export const SEMICOLON_SPLITTER = /;(?=(?:(?:[^"]*"){2})*[^"]*$)/;
 
+// Use regex to check if the string starts and ends with escape characters
 export const VALIDATE_ESCAPE_START_END_REGEX = /^(\\+|"+)([\s\S]*?)(\\+|"+)$/;
 
+// Validates decimal numbers between 0 and 1 (inclusive)
+// Matches: 0, 0.5, 0.123, 1, 1.0, 1.00, etc.
 export const DECIMAL_ZERO_TO_ONE_REGEX = /^(0(\.\d+)?|1(\.0+)?)$/;
 
+// Validates integers between 0 and 100 (inclusive)
+// Matches: 0, 1, 9, 10, 50, 99, 100
 export const INTEGER_ZERO_TO_HUNDRED_REGEX = /^(100|[1-9]?\d)$/;
 
+// Validates locale/language codes (ISO 639-1 format)
+// Matches: en, fr, de, en-US, fr-CA, de-DE, etc.
+// Format: two lowercase letters optionally followed by hyphen and two uppercase letters
 export const LOCALE_CODE_REGEX = /^[a-z]{2}(-[A-Z]{2})?$/;
 
+// Matches URLs (http/https with content, absolute paths with content, data URIs) OR filenames with image extensions
+// Filenames restricted to alphanumeric, hyphens, underscores, and dots for security
 export const IMAGE_URL_PATTERN =
   /^(https?:\/\/.+|\/[^\s]+|data:image\/.+)|^[\w\-.]+\.(png|jpg|jpeg|gif|svg|webp|bmp|ico)$/i;